On Wed, 3 Jun 2020 15:58:02 +0200 Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote: > Boris Brezillon <boris.brezillon@xxxxxxxxxxxxx> wrote on Wed, 3 Jun > 2020 15:49:17 +0200: > > > Replace the of_address_to_resource() + devm_ioremap() calls by > > platform_get_resource() + devm_ioremap_resource() ones which allows us > > to get rid of one error message since devm_ioremap_resource() already > > takes care of that. > > > > Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxx> > > --- > > drivers/mtd/nand/raw/fsl_upm.c | 23 +++++++---------------- > > 1 file changed, 7 insertions(+), 16 deletions(-) > > > > diff --git a/drivers/mtd/nand/raw/fsl_upm.c b/drivers/mtd/nand/raw/fsl_upm.c > > index a3e3a968891d..54851e9ea784 100644 > > --- a/drivers/mtd/nand/raw/fsl_upm.c > > +++ b/drivers/mtd/nand/raw/fsl_upm.c > > @@ -14,7 +14,6 @@ > > #include <linux/mtd/nand_ecc.h> > > #include <linux/mtd/partitions.h> > > #include <linux/mtd/mtd.h> > > -#include <linux/of_address.h> > > #include <linux/of_platform.h> > > #include <linux/of_gpio.h> > > #include <linux/io.h> > > @@ -197,7 +196,7 @@ static int fun_chip_init(struct fsl_upm_nand *fun, > > static int fun_probe(struct platform_device *ofdev) > > { > > struct fsl_upm_nand *fun; > > - struct resource io_res; > > + struct resource *io_res; > > const __be32 *prop; > > int rnb_gpio; > > int ret; > > @@ -208,13 +207,12 @@ static int fun_probe(struct platform_device *ofdev) > > if (!fun) > > return -ENOMEM; > > > > - ret = of_address_to_resource(ofdev->dev.of_node, 0, &io_res); > > - if (ret) { > > - dev_err(&ofdev->dev, "can't get IO base\n"); > > - return ret; > > - } > > + io_res = platform_get_resource(ofdev, IORESOURCE_MEM, 0); > > + fun->io_base = devm_ioremap_resource(&ofdev->dev, io_res); > > Why not even using devm_platform_ioremap_resource() resource directly? Because I need to pass the resource to fsl_upm_find(). ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/