[U-Boot] [PATCH 08/25] SPEAr: Add configuration options for spear3xx and spear6xx boards

[Amit Virdi]

Stefan,

>>>
>>> Can't you just remove this config.mk file completely?
>>> CONFIG_SYS_TEXT_BASE can be moved to the config header instead. What
>>> else is this file needed for?
>>
>> Could you please explain which config header you are referring to here?
>> Do you mean include/configs/spear..... ?
>
> Yes.
>
>> For SPEAr310 and SPEAr320, config.mk is needed to make image (by
>> specifying FLASH=PNOR during compile time) that can access CFI compliant
>> PNOR flash.
>
> In the example above (spear300), only CONFIG_SYS_TEXT_BASE is left in
> config.mk. This can be moved to the config header (include/configs/spear-xxx-
> h).
>
> You should be able to move the other remaining options (PNOR...) to
> boards.cfg. Or is this not possible?
>

It is possible and it adds to the clarity, thanks for this nice 
suggestion. I'll remove all the config.mk files from the 
board/spear/spearxx/ directories and add targets in boards.cfg for 
making u-boot build with environment variables placed in PNOR flash.

Best Regards
Amit Virdi