Hi Ye, Thanks for your work. On 2023-02-08 16:59:40 +0800, ye.xingchen@xxxxxxxxxx wrote: > From: Ye Xingchen <ye.xingchen@xxxxxxxxxx> > > 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: Ye Xingchen <ye.xingchen@xxxxxxxxxx> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > --- > drivers/media/platform/renesas/rcar-isp.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/media/platform/renesas/rcar-isp.c b/drivers/media/platform/renesas/rcar-isp.c > index 10b3474f93a4..a3cfd21f8aae 100644 > --- a/drivers/media/platform/renesas/rcar-isp.c > +++ b/drivers/media/platform/renesas/rcar-isp.c > @@ -419,10 +419,7 @@ static const struct media_entity_operations risp_entity_ops = { > static int risp_probe_resources(struct rcar_isp *isp, > struct platform_device *pdev) > { > - struct resource *res; > - > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - isp->base = devm_ioremap_resource(&pdev->dev, res); > + isp->base = devm_platform_get_and_ioremap_resource(pdev, 0, NULL); > if (IS_ERR(isp->base)) > return PTR_ERR(isp->base); > > -- > 2.25.1 -- Kind Regards, Niklas Söderlund