Re: [PATCH v5 09/11] mtd: parse out the ECC info for the full-id nand chips

[Vikram Narayanan <vikram186@gmail.com>]

On 5/15/2013 2:10 PM, Huang Shijie wrote:
> Parse out the ECC information for the full-id nand chips.
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
>   drivers/mtd/nand/nand_base.c |    2 ++
>   1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> index 41e5c07..359e105 100644
> --- a/drivers/mtd/nand/nand_base.c
> +++ b/drivers/mtd/nand/nand_base.c
> @@ -3243,6 +3243,8 @@ static bool find_full_id_nand(struct mtd_info *mtd, struct nand_chip *chip,
>   		chip->cellinfo = id_data[2];
>   		chip->chipsize = (uint64_t)type->chipsize << 20;
>   		chip->options |= type->options;
> +		chip->ecc_strength = NAND_ECC_STRENGTH(type);
> +		chip->ecc_step = NAND_ECC_STEP(type);

Does all NAND chips define this? I mean the patch 10/11 "mtd: add the 
ecc info for some full-id nand chips". What does the above result for 
the other chips? Does it get populated at runtime?

~Vikram