On Thu, 30 Mar 2017 15:45:59 +0900 Masahiro Yamada <yamada.masah...@socionext.com> wrote:
> > diff --git a/drivers/mtd/nand/denali_dt.c b/drivers/mtd/nand/denali_dt.c > index 1681a30..c3bc333 100644 > --- a/drivers/mtd/nand/denali_dt.c > +++ b/drivers/mtd/nand/denali_dt.c > @@ -31,10 +31,12 @@ struct denali_dt { > > struct denali_dt_data { > unsigned int revision; > + unsigned long ecc_strength_avail; Shouldn't you directly make sure that your driver will support strengths > BITS_PER_LONG? I would store the supported strength in a table to make it more future-proof: int *ecc_strength_avail; int num_ecc_strengths; > unsigned int caps; > }; >