[U-Boot] [PATCH 07/12 v2] ARM: OMAP3: Add memory and syslib common files, add NAND support

[Menon, Nishanth]

> -----Original Message-----
> From: Scott Wood [mailto:scottwood at freescale.com]
> Sent: Tuesday, October 07, 2008 12:30 PM
> To: Nishanth Menon
> Cc: Dirk Behme; u-boot at lists.denx.de; Kamat, Nishant; Menon, Nishanth
> Subject: Re: [U-Boot] [PATCH 07/12 v2] ARM: OMAP3: Add memory and syslib
> common files, add NAND support
> > >>> +    .eccbytes = 12,
> > >>> +    .eccpos = {
> > >>> +           2, 3, 4, 5,
> > >>> +           6, 7, 8, 9,
> > >>> +           10, 11, 12, 13},
> > >>> +    .oobfree = { {20, 50} }    /* don't care */
> > >>
> > >>
> > >> Bytes 64-69 of a 64-byte OOB are free?
> > >> What don't we care about?
> > > +static struct nand_ecclayout hw_nand_oob_64 = {
> > >
> > > Don't know (or understand?).
> > >
> > > All: Any help?
> > I do not get it either.. ECCPOS is in offset bytes, and oobfree should
> > be {.offset=20,.length=44} /*I always hated struct initialization done
> > as above..*/, but then,
> 
> Why not offset 14, length 50?
How about this part being used by ubi/jffs2 or some fs.. I cant think off-hand if there an implication, but yep, I suspect there is no reason why not..

Regards,
Nishanth Menon