2.6.20 : BUG in nand_do_read_oobwith MTD_OOB_PLACE ?

[Matthieu CASTET]

Matthieu CASTET <matthieu.castet <at> parrot.fr> writes:

> 
> Hi,
> 
> 2.6.20 nand mtd seems to read wrongly oob in MTD_OOB_PLACE.
> If I dump a page with uboot I get [1] and if I dump it with nanddump I get [2].
> As you could can see OOB doesn't match. Moreover the 2 snd 8 bytes OOB data
> given by nanddump are the 1st 8 bytes OOB of the next page.
> 
> If we look in nand_base.c, we saw that in case of MTD_OOB_PLACE, the len of oob
> is set to chip->ecc.layout->oobavail. This seems wrong as user expect to be
> mtd->oobsize.
> 
> Does this bug is know ?
> 
Any news on that ?

I needed to patch nand_do_read_oob to use jffs2 with 2.6.20 [1].


Matthieu
[1]
if (ops->mode == MTD_OOB_RAW || ops->mode == MTD_OOB_PLACE)
                len = mtd->oobsize;
        else
                len = chip->ecc.layout->oobavail;