[PATCH -next] test/py/test_mmc: wrap multi-argument printf-style strings

Bryan Brattlof bb at ti.com
Thu Jun 19 19:15:29 CEST 2025


On June 19, 2025 thus sayeth Tom Rini:
> On Thu, Jun 19, 2025 at 06:45:05AM -0500, Bryan Brattlof wrote:
> 
> > Newer versions of python will emit a TypeError about not enough
> > arguments for a format string:
> > 
> >     FAILED ub/test/py/tests/test_mmc.py::test_mmc_dev - TypeError: not enough arguments for format string
> >     FAILED ub/test/py/tests/test_mmc.py::test_mmcinfo - TypeError: not enough arguments for format string
> >     FAILED ub/test/py/tests/test_mmc.py::test_mmc_info - TypeError: not enough arguments for format string
> >     FAILED ub/test/py/tests/test_mmc.py::test_mmc_rescan - TypeError: not enough arguments for format string
> >     FAILED ub/test/py/tests/test_mmc.py::test_mmc_part - TypeError: not enough arguments for format string
> > 
> > Add parentheses around all multi argument format strings so all
> > arguments will be passed to the format string
> > 
> > Signed-off-by: Bryan Brattlof <bb at ti.com>
> 
> Nice! Does this mean you're also running these on hardware somewhere?

Yep I've got the all of the AM62* variants currently working in labgrid. 
I'm still trying to flow flush the u-boot-test-hooks scripts in the 
larger farm TI uses as a more permanent solution. But things are getting 
closer to fully automated :)

~Bryan

> 
> Reviewed-by: Tom Rini <trini at konsulko.com>
> 
> -- 
> Tom




More information about the U-Boot mailing list