Hi Yue, YueHaibing <yuehaib...@huawei.com> wrote on Wed, 4 Sep 2019 20:29:25 +0800:
> Use devm_platform_ioremap_resource() to simplify the code a bit. > This is detected by coccinelle. > > Reported-by: Hulk Robot <hul...@huawei.com> > Signed-off-by: YueHaibing <yuehaib...@huawei.com> > --- > drivers/thermal/armada_thermal.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/thermal/armada_thermal.c > b/drivers/thermal/armada_thermal.c > index 709a22f..70fe9c6 100644 > --- a/drivers/thermal/armada_thermal.c > +++ b/drivers/thermal/armada_thermal.c > @@ -708,12 +708,10 @@ static int armada_thermal_probe_legacy(struct > platform_device *pdev, > struct armada_thermal_priv *priv) > { > struct armada_thermal_data *data = priv->data; > - struct resource *res; > void __iomem *base; > > /* First memory region points towards the status register */ > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - base = devm_ioremap_resource(&pdev->dev, res); > + base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(base)) > return PTR_ERR(base); > Reviewed-by: Miquel Raynal <miquel.ray...@bootlin.com> Thanks, Miquèl