> -----Original Message----- > From: Scott Wood [mailto:[EMAIL PROTECTED] > Sent: Tuesday, October 07, 2008 12:30 PM > To: Nishanth Menon > Cc: Dirk Behme; u-boot@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot