[PATCH 1/5] mtd: nand: move layout structure into nand_ecc_ctrl

Troy Kisky troy.kisky at boundarydevices.com
Thu May 14 13:53:19 EDT 2009


David Brownell wrote:
> On Wednesday 13 May 2009, Troy Kisky wrote:
>> +static const struct nand_ecclayout atmel_oobinfo_large __initdata = {
> 
> Doesn't this give you section warnings?  __initconst is what
> I usually end up needing in such cases...

Only if the same file has a non const __initdata variable.
Is __initconst the preferred method ?

> 
> Also, for the davinci_nand driver this is going to need to
> switch away from the ecclayout struct in the driver-private
> area ... that's in a patch that (still) hasn't merged into
> the MTD tree, but was stored there for the same reason that
> you're submitting this patch:  the original code, using the
> annoying ECC_HW_SYNDROME support, computed the layout.
> 
> - Dave

Is ECC_HW_SYNDROME only needed because the eccpos array is too small??

I'll have to review that prepad  and ECC_HW_SYNDROME
Thanks

Troy




More information about the linux-mtd mailing list