[PATCH 4/5] mtd: nand: cafe_nand: use default ecc layout
Troy Kisky
troy.kisky at boundarydevices.com
Thu May 14 17:18:38 EDT 2009
David Brownell wrote:
> On Wednesday 13 May 2009, Troy Kisky wrote:
>> The bad block marker is at 14, but oobfree also
>> starts at 14. This doesn't make sense to me,
>
> On-chip bad-block tables stick the "Bbt0" (etc)
> tags into the OOB data area, and don't need to
> share that area with anything else. It's OK.
>
>
>> so I don't know if this patch is good or not.
>
> Isn't this one of the drivers using NAND_ECC_HW_SYNDROME?
> So that the ecclayout is fully defined by prepad==0?
>
> This is a case where you *could* safely generate the
> current ecclayout structs using simple rules.
>
>
Actually, cafe_nand overrides most everything and doesn't use
prepad or postpad or ecc.layout. But I think it still relies
on ecc.layout.oobfree to setup chip->oob_poi. So, I'm no closer
to understanding why oobfree and bad block marker start at 14.
Troy
More information about the linux-mtd
mailing list