Hi Tim
On Thu, 2021-08-26 at 07:57 -0700, Tim Harvey wrote:
... Marcel,
This would break imx8mm-venice.
Sure, that's why I made it an RFC. I was just missing some context.
The of-list, @fdt-SEQ and @config-SEQ are required to support automatic generation of fdt and config nodes when you have multiple device-trees. See tools/binman/entries.rst for details.
Thanks for that pointer, that explains a lot.
Currently, imx8mm-venice is the only board that uses multiple dtbs in CONFIG_OF_LIST so this would be the only board broken by your patch. I'm curious if just using the of-list, @fdt-SEQ and @config-SEQ as above works fine for boards with a single dtb? Otherwise I suppose boards like mine can include your common "imx8mm-binman.dtsi" but will have to leave the &binman node override.
Yes, let me investigate further and find a suitable solution. Expect a v2 with that shortly.
Best regards,
Tim
Cheers
Marcel