[U-Boot-Users] [PATCH V3] ARM: Add support for Lyrtech SFF-SDR board (ARM926EJS)

[Wolfgang Denk]

In message <1211473413-27392-1-git-send-email-hugo.villeneuve at lyrtech.com> you wrote:
> ARM: This patch adds support for the Lyrtech SFF-SDR
> board, based on the TI DaVinci architecture (ARM926EJS).
> 
> Signed-off-by: Hugo Villeneuve <hugo.villeneuve at lyrtech.com>
> Signed-off-by: Philip Balister <philip at balister.org>
...
> +#define CONFIG_BOOTCOMMAND	"setenv setboot;run setboot"

That's even worse. The first part ("setenv setboot" will  delete  the
"setboot"  variable,  so  the second part ("run setboot") will always
just raise an error message.

Please review what exactly you are tyring to do here...

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Pray to God, but keep rowing to shore. - Russian Proverb