[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