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

Scott Wood scottwood at freescale.com
Tue Dec 7 01:30:50 CET 2010


On Mon, 6 Dec 2010 11:13:22 -0600
Scott Wood <scottwood at freescale.com> wrote:

> 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.

Applied to u-boot-nand-flash.

-Scott



More information about the U-Boot mailing list