[U-Boot-Users] [PATCH]: Fix for bug: U-boot environment corrupt by reading uninitialized flash memory instead of RAM.

Remy Bohmer linux at bohmer.net
Wed May 7 09:52:26 CEST 2008


Hello Haavard, Wolfgang

>  Hmm...maybe this is a stupid question, but why can't it _always_ use
>  the RAM area to look up the environment? env_init() is one of the first
>  things to be called during bootstrap...
>  Maybe it's wishful thinking, but if we could get rid of this extra
>  logic, u-boot would become simpler, smaller and faster, right?

There are no stupid questions, just stupid answers... ;-)

I think it is nicer too, but that appears to me as a good thing to
investigate for the next release of U-boot, not changing this on the
deadline of the current release.

I think the patch I posted yesterday should be integrated in the
current release, because it is limited to fixing a very annoying bug
(bogus environment settings on at least the Atmel at91 boards using
dataflash as environment settings storage, maybe also other boards) In
fact, If it was up to me, I would classify such a bug as a
showstopper...

Wolfgang, have you looked at this patch already?


Kind Regards,

Remy




More information about the U-Boot mailing list