You still have missed several instances. I am dropping this set. On Fri, Sep 06, 2024 at 07:34:05PM +0800, Zhang Zekun wrote: > platform_get_resource_byname() and devm_ioremap_resource() can be > replaced by devm_platform_ioremap_resource_byname(), which can > simplify the code logic a bit, No functional change here. > > Signed-off-by: Zhang Zekun <zhangzekun11@xxxxxxxxxx> > --- > drivers/remoteproc/ingenic_rproc.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/remoteproc/ingenic_rproc.c b/drivers/remoteproc/ingenic_rproc.c > index 9902cce28692..1b78d8ddeacf 100644 > --- a/drivers/remoteproc/ingenic_rproc.c > +++ b/drivers/remoteproc/ingenic_rproc.c > @@ -183,8 +183,7 @@ static int ingenic_rproc_probe(struct platform_device *pdev) > vpu->dev = &pdev->dev; > platform_set_drvdata(pdev, vpu); > > - mem = platform_get_resource_byname(pdev, IORESOURCE_MEM, "aux"); > - vpu->aux_base = devm_ioremap_resource(dev, mem); > + vpu->aux_base = devm_platform_ioremap_resource_byname(pdev, "aux"); > if (IS_ERR(vpu->aux_base)) { > dev_err(dev, "Failed to ioremap\n"); > return PTR_ERR(vpu->aux_base); > -- > 2.17.1 >