[U-Boot] [U-Boot,v2,11/23] spl: Add a comment to spl_set_bd()

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Mon Feb 11 21:00:06 UTC 2019


Am 09.11.2018 um 19:43 schrieb Tom Rini:
> On Tue, Oct 02, 2018 at 05:22:41AM -0600, Simon Glass wrote:
> 
>> There is a strange feature to set global_data to a data-section variable
>> early in SPL. This only works if SPL actually has access to SRAM which is
>> not the case on x86, for eaxmple. Add a comment to this effect.

Seems like I missed that one back in October.

Does anyone have an idea why this variable ('static bd_t bdata') is 
hard-coded into section ".data"? To me this seems pretty unportable...

For example, for a specific feature on socfpga (warm reboot CRC check), 
I would prefer to have the ".data" section empty and put 'bdata' into 
bss. This is currently not possible.

But before sending a patch that somehow changes this behaviour, I'd like 
to know why this variable is put into ".data" instead of ".bss"

Thanks,
Simon

>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
> 
> Reviewed-by: Tom Rini <trini at konsulko.com>


More information about the U-Boot mailing list