On 13/06/2022 05:31, Peng Fan (OSS) wrote:
在 2022/6/11 0:47, Alper Nebi Yasak 写道:
Looks like I have misunderstood things here a bit. CONFIG_BINMAN enables you to declare and use symbols. CONFIG_SPL/TPL_BINMAN_SYMBOLS declares certain symbols ('u_boot_any'). The name is a bit misleading, as if it enables support for using symbols, and that confused me.
I am not sure, but I think CONFIG_SPL/TPL_BINMAN_SYMBOLS are only for certain symbols saying u_boot_any.
I mean I not think BINMAN_SYMBOLS are only for certtain symbols.
They are right now, your patch changes that.
I wasn't sure about changing it at first, but it makes more sense to me now. I'll update my series and split things out:
- BINMAN: to run binman after build - BINMAN_SYMBOLS: to enable binman symbols, binman_sym.h - BINMAN_UBOOT_SYMBOLS: new config to declare u_boot_any symbols