On Fri, 20 Jul 2018 17:15:05 +0200 Miquel Raynal <miquel.raynal at bootlin.com> wrote: > > - ret = nand_scan_tail(mtd); > + chip->controller->ops = &marvell_nand_controller_ops; Assigning ->ops to &marvell_nand_controller_ops should be done only once in the probe function (here [1]). With that fixed, you can add Reviewed-by: Boris Brezillon <boris.brezillon at bootlin.com> > + ret = nand_scan(mtd, marvell_nand->nsels); > if (ret) { > - dev_err(dev, "nand_scan_tail failed: %d\n", ret); > + dev_err(dev, "could not scan the nand chip\n"); > return ret; > } >