Minor comment: $SUBJECT should say it adds this "for DaVinci". ;)
To recap other discussion (I hope), this needs to be reissued since: On Monday 18 May 2009, [email protected] wrote: > +static struct nand_ecclayout hwecc4_2048 __initconst = { > + .eccbytes = 40, > + .eccpos = { 0, 1, 2, 3, 4, Byte 0 -- for 8-bit large-page parts which need this patch -- is the manufacturer bad block marker, and the idea was to *not* clobber it. Those bytes need to be skipped. Will the 40 bytes of ECC data now be stored at offsets 24..63, matching Troy's patches? 0..1 manufacturer badblock markers 8..11 BBT marker ("free" oob data) 16..23 JFFS2 stuff ("free" oob data) 24..63 ECC data And for 4KB pages, it would be the same -- except that there would be 80 bytes of ECC data, from 48..127 (too much to fit into the eccpos array). > + /* offset 5 holds the badblock marker */ > + 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, > + /* 8 bytes at offset 16 hold JFFS2 clean markers */ > + 24, 25, 26, 27, 28, > + 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, > + 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, }, > + .oobfree = { > + {.offset = 16, .length = 8, }, > + {.offset = 49, }, > + }, > +}; _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
