Bernard Blackham wrote: > [Cc'ing Sergey who has been here before and might know more...] > > Troy Kisky wrote: >> Not only are the ECC bytes stored in different locations in the spare >> area, but the ECC value stored is also different. (davinci-git vs U-boot) >> >> I've modified both the linux kernel, and u-boot to store the ecc at the >> end of the spare bytes. But I haven't pushed this anywhere. >> >> It's pretty trivial to make U-Boot use the same ECC algorithm as the >> davinci-git tree. > > Maybe I don't understand it fully, but the current u-boot ECC > implementation for large page NAND devices looks rather broken to me - > it appears to generate a 12 byte ECC from the four NANDFnECC registers > for a 2048 byte page read. But the NANDFnECC registers are for NAND > chips on four different chip selects, not four 512-byte pages of one > request. The data sheet says the ECC hardware will only work on 512-byte > reads or writes.
You are 100% correct. The davinci u-boot implementation for 2048 byte pages is broken. /Thomas _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
