On Wednesday 07 March 2012 13:03:59 Amit Virdi wrote:
> From: Vipin KUMAR <vipin.ku...@st.com>
> 
> This patch modifies the default environment variables as:
> 1. Default bootargs:
>  - console=ttyAMA0,115200
>  - For environment present in NOR flash
>      root=/dev/mtdblock3
>  - For environment present in NAND flash
>      root=/dev/mtdblock7
>  - Removes "mem=" option
> 2. Introduces CONFIG_EXTRA_ENV_USBTTY as default usbtty env var even when
> usbtty is not selected
> 3. Add default definitions for nfsboot and ramboot
> 4. Add a new default environment variable(CONFIG_EXTRA_ENV_UNLOCK)
> 
> Signifacance of CONFIG_EXTRA_ENV_USBTTY:
> This environment variable is important for flashing utility to work. So if
> somebody accidently erases the env sector then also this variable must be
> preserved so that flashing utility functions properly.
> 
> 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.

> Signed-off-by: Vipin Kumar <vipin.ku...@st.com>
> Signed-off-by: Shiraz Hashim <shiraz.has...@st.com>
> Signed-off-by: Amit Virdi <amit.vi...@st.com>

Other than my comment above:

Acked-by: Stefan Roese <s...@denx.de>

Thanks,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: off...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to