On 01/09/2021 07:51, Cai Huoqing wrote: > Use the devm_platform_ioremap_resource() helper instead of > calling platform_get_resource() and devm_ioremap_resource() > separately > > Signed-off-by: Cai Huoqing <caihuoqing@xxxxxxxxx> > --- > drivers/media/platform/meson/ge2d/ge2d.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/media/platform/meson/ge2d/ge2d.c b/drivers/media/platform/meson/ge2d/ge2d.c > index 9b1e973e78da..ccda18e5a377 100644 > --- a/drivers/media/platform/meson/ge2d/ge2d.c > +++ b/drivers/media/platform/meson/ge2d/ge2d.c > @@ -922,7 +922,6 @@ static int ge2d_probe(struct platform_device *pdev) > struct reset_control *rst; > struct video_device *vfd; > struct meson_ge2d *ge2d; > - struct resource *res; > void __iomem *regs; > int ret = 0; > int irq; > @@ -937,8 +936,7 @@ static int ge2d_probe(struct platform_device *pdev) > ge2d->dev = &pdev->dev; > mutex_init(&ge2d->mutex); > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - regs = devm_ioremap_resource(ge2d->dev, res); > + regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(regs)) > return PTR_ERR(regs); > > Reviewed-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>