[U-Boot] [PATCH v8 1/5] mtd: nand: omap: enable BCH ECC scheme using ELM for generic platform

Scott Wood scottwood at freescale.com
Wed Oct 16 21:37:14 CEST 2013


On Wed, 2013-10-16 at 18:44 +0000, Gupta, Pekon wrote:
> Hi,
> 
> > From: Scott Wood [scottwood at freescale.com]
> > > > From: Scott Wood [mailto:scottwood at freescale.com]
> > > On Thu, 2013-10-10 at 16:30 +0530, Pekon Gupta wrote:
> > [snip]
> > > > -#else
> > > > + } else {
> > > >   /*
> > > >    * This ecc_size_config setting is for BCH sw library.
> > > >    *
> > > > @@ -333,7 +330,7 @@ static void omap_hwecc_init_bch(struct nand_chip
> > > *chip, int32_t mode)
> > > >    *  size1 = 32 (skip 32 nibbles = 16 bytes per sector in spare area)
> > > >    */
> > > >   val = (32 << 22) | (0 << 12);
> > > > -#endif
> > > > + }
> > >
> > > Either indentation or braces are wrong above.  If it's just the
> > > indentation that's wrong, let me know and I can fix up when applying.
> > >
> Please let me know if you can pick this series, Or if you want me to fix indentation.
> I would be sending another series which also has some of your feedbacks.

I'll take it, but I probably won't have a chance to apply U-Boot patches
until at least the week after next (I'm attending conferences next week,
and this week I need to process Linux patches since those have an
earlier deadline for making the next release).

-Scott





More information about the U-Boot mailing list