On Thu, Mar 16, 2017 at 09:02:40AM +0100, Boris Brezillon wrote: > diff --git a/drivers/mtd/nand/atmel/nand-controller.c b/drivers/mtd/nand/atmel/nand-controller.c > new file mode 100644 > index 000000000000..f71b9e5d7d9d > --- /dev/null > +++ b/drivers/mtd/nand/atmel/nand-controller.c > @@ -0,0 +1,2198 @@ [...] > +static int > +atmel_hsmc_nand_controller_legacy_init(struct atmel_hsmc_nand_controller *nc) > +{ > + struct regmap_config regmap_conf = { > + .reg_bits = 32, > + .val_bits = 32, > + .reg_stride = 4, > + .val_bits = 32, You assigned val_bits twice. Is that just a harmless mistake, or did you mean to set something else? (sparse and other tools complain about this) > + }; Brian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html