[U-Boot] [PATCH 10/25] SPEAr: Change the default environment variables

[Amit Virdi]

Hello Stefan,

>> Signifacance of CONFIG_EXTRA_ENV_UNLOCK:
>> This env variable is read by the cfi driver to unlock all flash sectors.
>> This is necessary because the Parallel NOR flash connected on the spear
>> boards, M28W64, has all its sectors in locked state at reset and these
>> have to be unlocked explicitly before being erased or written.
>
> But this is not needed for boards not using the common CFI flash driver. As
> the SPEAr600 based ones. So please only define it for board using the CFI
> driver.
>

Ok. I would undefine it from the spear-common file and define it for 
SPEAr310 board file only.

>> Signed-off-by: Vipin Kumar<vipin.kumar at st.com>
>> Signed-off-by: Shiraz Hashim<shiraz.hashim at st.com>
>> Signed-off-by: Amit Virdi<amit.virdi at st.com>
>
> Other than my comment above:
>
> Acked-by: Stefan Roese<sr at denx.de>
>

Thanks.

Regards
Amit Virdi