Messages in this thread | | | Date | Tue, 22 Nov 2022 11:02:39 +0100 | From | Miquel Raynal <> | Subject | Re: [PATCH] mtd: rawnand: orion: use devm_platform_get_and_ioremap_resource() |
| |
Hello,
ye.xingchen@zte.com.cn wrote on Tue, 22 Nov 2022 09:33:27 +0800 (CST):
> From: Minghao Chi <chi.minghao@zte.com.cn> >
Double space in the subject
> Convert platform_get_resource(), devm_ioremap_resource() to a single > call to devm_platform_get_and_ioremap_resource(), as this is exactly > what this function does. > > Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn> > --- > drivers/mtd/nand/raw/orion_nand.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/mtd/nand/raw/orion_nand.c b/drivers/mtd/nand/raw/orion_nand.c > index 1bfecf502216..00bd7be76e07 100644 > --- a/drivers/mtd/nand/raw/orion_nand.c > +++ b/drivers/mtd/nand/raw/orion_nand.c > @@ -102,7 +102,6 @@ static int __init orion_nand_probe(struct platform_device *pdev) > struct mtd_info *mtd; > struct nand_chip *nc; > struct orion_nand_data *board; > - struct resource *res; > void __iomem *io_base; > int ret = 0; > u32 val = 0; > @@ -119,8 +118,7 @@ static int __init orion_nand_probe(struct platform_device *pdev) > info->controller.ops = &orion_nand_ops; > nc->controller = &info->controller; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - io_base = devm_ioremap_resource(&pdev->dev, res); > + io_base = devm_platform_get_and_ioremap_resource(pdev, 0, NULL);
If we no longer use res, why not just devm_platform_ioremap_resource() ?
> > if (IS_ERR(io_base)) > return PTR_ERR(io_base);
Thanks, Miquèl
| |