| Message ID | 20230216090610.130860-1-yang.lee@linux.alibaba.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [-next] can: ctucanfd: Use devm_platform_ioremap_resource() | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Series ignored based on subject |
On Thursday 16 of February 2023 10:06:10 Yang Li wrote: > Convert platform_get_resource(), devm_ioremap_resource() to a single > call to Use devm_platform_ioremap_resource(), as this is exactly > what this function does. > > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> Acked-by: Pavel Pisa <pisa@cmp.felk.cvut.cz> > --- > drivers/net/can/ctucanfd/ctucanfd_platform.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/net/can/ctucanfd/ctucanfd_platform.c > b/drivers/net/can/ctucanfd/ctucanfd_platform.c index > f83684f006ea..a17561d97192 100644 > --- a/drivers/net/can/ctucanfd/ctucanfd_platform.c > +++ b/drivers/net/can/ctucanfd/ctucanfd_platform.c > @@ -47,7 +47,6 @@ static void ctucan_platform_set_drvdata(struct device > *dev, */ > static int ctucan_platform_probe(struct platform_device *pdev) > { > - struct resource *res; /* IO mem resources */ > struct device *dev = &pdev->dev; > void __iomem *addr; > int ret; > @@ -55,8 +54,7 @@ static int ctucan_platform_probe(struct platform_device > *pdev) int irq; > > /* Get the virtual base address for the device */ > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - addr = devm_ioremap_resource(dev, res); > + addr = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(addr)) { > ret = PTR_ERR(addr); > goto err;
From: Yang Li <yang.lee@linux.alibaba.com> Date: Thu, 16 Feb 2023 17:06:10 +0800 > Convert platform_get_resource(), devm_ioremap_resource() to a single > call to Use devm_platform_ioremap_resource(), as this is exactly > what this function does. > > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> Reviewed-by: Alexander Lobakin <aleksander.lobakin@intel.com> > --- > drivers/net/can/ctucanfd/ctucanfd_platform.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/net/can/ctucanfd/ctucanfd_platform.c b/drivers/net/can/ctucanfd/ctucanfd_platform.c > index f83684f006ea..a17561d97192 100644 > --- a/drivers/net/can/ctucanfd/ctucanfd_platform.c > +++ b/drivers/net/can/ctucanfd/ctucanfd_platform.c > @@ -47,7 +47,6 @@ static void ctucan_platform_set_drvdata(struct device *dev, > */ > static int ctucan_platform_probe(struct platform_device *pdev) > { > - struct resource *res; /* IO mem resources */ > struct device *dev = &pdev->dev; > void __iomem *addr; > int ret; > @@ -55,8 +54,7 @@ static int ctucan_platform_probe(struct platform_device *pdev) > int irq; > > /* Get the virtual base address for the device */ > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - addr = devm_ioremap_resource(dev, res); > + addr = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(addr)) { > ret = PTR_ERR(addr); > goto err; Thanks, Olek
On 16.02.2023 17:06:10, Yang Li wrote: > Convert platform_get_resource(), devm_ioremap_resource() to a single > call to Use devm_platform_ioremap_resource(), as this is exactly > what this function does. > > Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> Applied to linux-can-next/testing. Thanks, Marc
diff --git a/drivers/net/can/ctucanfd/ctucanfd_platform.c b/drivers/net/can/ctucanfd/ctucanfd_platform.c index f83684f006ea..a17561d97192 100644 --- a/drivers/net/can/ctucanfd/ctucanfd_platform.c +++ b/drivers/net/can/ctucanfd/ctucanfd_platform.c @@ -47,7 +47,6 @@ static void ctucan_platform_set_drvdata(struct device *dev, */ static int ctucan_platform_probe(struct platform_device *pdev) { - struct resource *res; /* IO mem resources */ struct device *dev = &pdev->dev; void __iomem *addr; int ret; @@ -55,8 +54,7 @@ static int ctucan_platform_probe(struct platform_device *pdev) int irq; /* Get the virtual base address for the device */ - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); - addr = devm_ioremap_resource(dev, res); + addr = devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(addr)) { ret = PTR_ERR(addr); goto err;
Convert platform_get_resource(), devm_ioremap_resource() to a single call to Use devm_platform_ioremap_resource(), as this is exactly what this function does. Signed-off-by: Yang Li <yang.lee@linux.alibaba.com> --- drivers/net/can/ctucanfd/ctucanfd_platform.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)