[U-Boot] [PATCH] [OneNAND] bad block aware read/write support

[Kyungmin Park]

Hi,

On Wed, Nov 5, 2008 at 5:40 PM, Stefan Roese <sr at denx.de> wrote:
> Hi Kyungmin,
>
> On Tuesday 04 November 2008, Stefan Roese wrote:
>> > Update OneNAND command to support bad block awareness
>> > Also change the OneNAND command styel like NAND
>>
>> I'm starting with OneNAND support for a MIPS platform right now and wasn't
>> ware that the onenand commands were not bad block aware. So thanks for this
>> patch. But I have some comments.
>
> Some further comment below.
>
> <snip>
>
>> > +           ret = mtd->read_oob(mtd, ofs, &ops);
>> > +           if (ret) {
>> > +                   printk("Read failed 0x%x, %d", (unsigned int) ofs, ret);
>> > +                   mtd->block_markbad(mtd, ofs);
>
> You are marking all blocks as bad whenever a failure occurs. I'm not sure if
> we really should do it this way. Failures could have other reasons as well.
> I'm inclined to remove this marking in my patch version.
>
> Any comments?

Right, current implementation make a bad on all errors, but actually
it need to try again in case read/write error. It will patch it later.

>
> <snip>
>
>> I'm currently working on a version of this "bad block aware" OneNAND
>> command support which resembles the NAND command style even more. I hope to
>> have something ready till tomorrow that I can send to the list for review.
>> I would really like to see some comments from you on this since you are
>> much more experienced with the OneNAND stuff.
>
> Do you already have some comments to the patch version I posted yesterday?

Looks good to me. No problem to commit your version.

Acked-by: Kyungmin Park <kyungmin.park at samsung.com>

Thank you,
Kyungmin Park