Hi Francis,
I was able to have the CI passes but I had to not use CONFIG_HUSH_2021_PARSER for two boards:
- u-boot.img file size for the rcar3_ulcb, when built with
CONFIG_HUSH_PARSER_2021, exceeds the 1MB file limit of 890 bytes. It should be possible to reduce the image size by not compiling some hush features (e.g. loop) for this particular board. 2. The keymile board family is the only set of boards to call get_local_var(), set_local_var() and unset_local_var(). Sadly, these functions are static in this contribution. I could have change all of them to introduce code like this: *_local_var(/*...*/) { if (gd->flags & GD_FLG_HUSH_OLD_PARSER) return *_local_var_old(/*...*/); if (gd->flags & GD_FLG_HUSH_2021_PARSER) return *_local_var_2021(/*...*/); } But this would have mean renaming all old hush functions calls and I did not want to change the old hush particularly to avoid breaking things. Instead, I change the keymile board to use environment variable instead of local ones. I think this particularities can be addressed in future works.
you can for your rework simply replace the calls for our boards to use environment variables as you proposed in your initial patch. I will put it on my list and we have to find a different solution for us in the future.
Best regards Holger