[RFC] nand/davinci: Fix comment to match the code
Artem Bityutskiy
Artem.Bityutskiy at nokia.com
Wed Aug 25 08:55:06 EDT 2010
On Wed, 2010-08-25 at 17:57 +0530, Sudhakar Rajashekhara wrote:
> > --- a/drivers/mtd/nand/davinci_nand.c
> > +++ b/drivers/mtd/nand/davinci_nand.c
> > @@ -369,8 +369,9 @@ compare:
> > * after setting the 4BITECC_ADD_CALC_START bit. So if you immediately
> > * begin trying to poll for the state, you may fall right out of your
> > * loop without any of the correction calculations having taken place.
> > - * The recommendation from the hardware team is to wait till ECC_STATE
> > - * reads less than 4, which means ECC HW has entered correction state.
> > + * The recommendation from the hardware team is to initially delay as
> > + * long as ECC_STATE reads less than 4. After that, ECC HW has entered
> > + * correction state.
> > */
> > do {
> > ecc_state = (davinci_nand_readl(info,
> > --
>
> Thanks for pointing this out, but I have already corrected this and submitted
> the patch. Artem Bityutskity has pushed this patch to his l2-mtd-2.6 / master
> tree.
If you mean
commit 1c3275b656045aff9a75bb2c9f3251af1043ebb3
Author: Sudhakar Rajashekhara <sudhakar.raj at ti.com>
Date: Tue Jul 20 15:24:01 2010 -0700
mtd: nand: davinci: correct 4-bit error correction
On TI's DA830/OMAP-L137, DA850/OMAP-L138 and DM365, after setting the
4BITECC_ADD_CALC_START bit in the NAND Flash control register to 1 and
before waiting for the NAND Flash status register to be equal to 1, 2 or
3, we have to wait till the ECC HW goes to correction state. Without this
wait, ECC correction calculations will not be proper.
This has been tested on DA830/OMAP-L137, DA850/OMAP-L138, DM355 and DM365
EVMs.
Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj at ti.com>
Acked-by: Sneha Narnakaje <nsnehaprabha at ti.com>
Signed-off-by: Andrew Morton <akpm at linux-foundation.org>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy at nokia.com>
Signed-off-by: David Woodhouse <David.Woodhouse at intel.com>
then it is already in upstream in Linus' tree. I do not have anything else from you.
--
Best Regards,
Artem Bityutskiy (Артём Битюцкий)
More information about the linux-mtd
mailing list