[PATCH v2 0/8] spl: binman: Fixes for BINMAN_SYMBOLS

[Alper Nebi Yasak]

On 23/06/2022 06:00, Peng Fan (OSS) wrote:
> Hi Alper,
> 
> 在 2022/6/18 20:13, Alper Nebi Yasak 写道:
>> There's some trouble with an i.MX8M series [1] trying to use binman
>> symbols. The crux of it is the 'u_boot_any' symbols BINMAN_SYMBOLS
>> configs declare, and the boards creating partial binman images including
>> an SPL without a U-Boot the symbol is referring to.
>>
>> Normally this should be easy to resolve by disabling BINMAN_SYMBOLS
>> configs, but that causes a build error. Apparently some parts of the SPL
>> code (RAW_IMAGE_SUPPORT, RAM_DEVICE) use the symbols directly without
>> guarding them by BINMAN_SYMBOLS, implicitly requiring it.
>>
>> The first patch fixes the issue above, the rest are related things I
>> tinkered with while trying to understand the issue and the i.MX8M use
>> case. Part of this is splitting binman symbols support from enabling
>> binman and from the u-boot-any symbols declarations. Another is to add a
>> new macro people can use to check if they can use binman symbols safely.
>>
>> These apply onto u-boot/next. I have also triggered an Azure CI run [2]
>> via a Github pull request.
> 
> I have tested your patchset with branch : imx-ddr-binman-symbols
> 
> Tested-by: Peng Fan <peng.fan at nxp.com> #i.MX8MP-EVK
> 
> Would you send out the i.MX patches? or you need me to send a V7 version 
> based on your patchset?

It would be better if you sent those.

Thanks.