On Mon, 26 Oct 2015 19:31:07 -0700 Brian Norris <computersforpe...@gmail.com> wrote:
> All of these drivers set up a parser data struct just to communicate DT > partition data. This field has been deprecated and is instead supported > by telling nand_scan_ident() about the 'flash_node'. > > This patch: > * sets chip->flash_node for those drivers that didn't already (but used > OF partitioning) As mentioned in patch 1, I think we should get rid of the ->flash_node field and directly set mtd->dev.of_node instead. If we want to hide MTD internals we could provide this kind of helper: static inline void mtd_set_of_node(struct mtd_device *mtd, struct device_node *np) { mtd->dev.of_node = np; } > * drops the parser data > * switches to the simpler mtd_device_register() where possible, now > that we've eliminated one of the auxiliary parameters > > Now that we've assigned chip->flash_node for these drivers, we can > probably rely on nand_dt_init() to do more of the DT parsing for us, but > for now, I don't want to fiddle with each of these drivers. The parsing > is done in duplicate for now on some drivers. I don't think this should > break things. (Famous last words.) > > Signed-off-by: Brian Norris <computersforpe...@gmail.com> For the sunxi_nand driver Acked-by: Boris Brezillon <boris.brezil...@free-electrons.com> Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/