NAND Flash Framework for review

[Luiz Otavio O Souza]

On Mar 12, 2010, at 10:02 AM, Andrew Turner wrote:

> On Tue, 9 Mar 2010 00:28:53 +0100
> Rafal Jaworowski <raj at semihalf.com> wrote:
>> We are looking for review, comments and any other feedback.
> 
> I've been reviewing the driver and should have some notes in the next
> couple of days on areas I would like to see changes.
> 
> Andrew

Rafal, first of all, i'd like to thank you, semihalf and Grzegorz for the contribution, the code is very well written, clear and easy to understand.

That said, the code isn't ready to use, there are a few small things to fix (ecc byte positions, bad block table for non onfi compliant devices, there is no support to read status from ready/busy nand pin and gnand interface doesn't work with disk->d_maxsize == disk->d_sectorsize == device->blocksize).

I've tested the code with two Hynix devices, one small page type (512/16) with 64MB of capacity and the other is a large page type (2048/64) with 512MB.

The small page device works really fine, i can read the raw data and verify ecc (the device has a YAFFS1 file system).

Now i'm looking at ecc positions for the large page device which looks like have a YAFFS2 FS on it, no luck with ecc data but raw data looks fine.

I'll wait for Andrew notes and then i gonna put some more ideas and patches together.

Luiz