On 2/13/26 9:20 PM, Simon Glass wrote:

Hello Simon,

This should be documented somewhere, likely in the comment for this
entry type, so it ends up visible in the docs.

I will send a V2 with documentation.

Also, this code path will need a test, please.
How do you propose to test this ?

Extend or copy one of the existing imx8 tests, perhaps with a new .dts
- and I suppose set the env var and see that it works too?

Note that imx8 has basic test coverage but there is quite a bit missing:

tools/binman/btool/cst.py                                  18      4    78%
tools/binman/etype/nxp_imx8mcst.py                         89     69    22%
tools/binman/etype/nxp_imx8mimage.py                       38      6    84%

I shed a small tear whenever I see that :-)
See [PATCH] binman: Fill in most of the imx8mimage and imx8mcst tests .

Reply via email to