login
Header Space

 
 

Re: [PATCH try #2] Blackfin on-chip NAND Flash Controller driver

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <bryan.wu@...>
Cc: David Woodhouse <dwmw2@...>, Thomas Gleixner <tglx@...>, Mike Frysinger <vapier.adi@...>, Robin Getz <rgetz@...>, <linux-mtd@...>, <linux-kernel@...>, <akpm@...>
Date: Tuesday, September 11, 2007 - 4:35 am

Hi,

On Mon, 2007-09-10 at 17:50 +0800, Bryan Wu wrote:


This means that MTD did not return an error when the test tried to read
the last NAND page's OOB. Here is the code:

/* Attempt to write off end of device */
ops.mode      = MTD_OOB_AUTO;
ops.ooblen    = mtd->ecclayout->oobavail + 1;
ops.oobbuf    = writebuf;
printk(PRINT_PREF "Attempting to write past end of device\n");
printk(PRINT_PREF "An error is expected...\n");
err = mtd->write_oob(mtd, mtd->size - mtd->writesize, &ops);

So we read OOB size + 1 byte from the last NAND page and get no error.

This test passes for nandsim, so I'm not sure the problem is in the
generic code. On the other hand, it is generic code which should check
rages.


Similar.


We are planning to clean up the tests and submit them for kernel
inclusion some day, when we have time.

-- 
Best regards,
Artem Bityutskiy (Битюцкий Артём)

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH try #2] Blackfin on-chip NAND Flash Controller dr..., Artem Bityutskiy, (Tue Sep 11, 4:35 am)
speck-geostationary