[U-Boot] [PATCH] env_nand: Use nand_read_skip_bad instead of nand_read

Scott Wood scottwood at freescale.com
Mon Dec 6 18:13:22 CET 2010


On Sun, 5 Dec 2010 15:22:25 -0600
"Paulraj, Sandeep" <s-paulraj at ti.com> wrote:

> 
> 
> > The nand-read function returns an error code if correctable errors have
> > occurred.
> > This is not desirable, since the errors have been corrected!
> > 
> > This patch switches to the nand_read_skip_bad function which does not
> > return an error code if the errors are correctable.
> > 
> > Signed-off-by: Steve Sakoman <steve.sakoman at linaro.org>
> > ---
> > 
> > diff --git a/common/env_nand.c b/common/env_nand.c
> > index 4e8307a..7f6c917 100644
> > --- a/common/env_nand.c
> > +++ b/common/env_nand.c
> > @@ -297,7 +297,7 @@ int readenv(size_t offset, u_char * buf)
> >  			offset += blocksize;
> >  		} else {
> >  			char_ptr = &buf[amount_loaded];
> > -			if (nand_read(&nand_info[0], offset, &len, char_ptr))
> > +			if (nand_read_skip_bad(&nand_info[0], offset, &len,
> > char_ptr))
> >  				return 1;
> >  			offset += blocksize;
> >  			amount_loaded += len;
> > 
> 
> Scott,
> 
> Can I add this to u-boot-ti?

Acked-by: Scott Wood <scottwood at freescale.com>

Sorry about the delay, I'll try to get to the NAND patches soon.  I've
been busy with other things.

-Scott



More information about the U-Boot mailing list