media: staging: tegra-vde: Use devm_platform_ioremap_resource_byname()

There is a new devm_platform_ioremap_resource_byname() helper in the
kernel now, which helps to make code cleaner a tad by replacing few
"boilerplate" lines of code with a single line. Let's utilize that
new helper in the VDE driver.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
This commit is contained in:
Dmitry Osipenko 2020-03-02 18:29:04 +01:00 committed by Mauro Carvalho Chehab
parent 1db56284b9
commit 7af80a697d

View file

@ -949,7 +949,6 @@ static int tegra_vde_runtime_resume(struct device *dev)
static int tegra_vde_probe(struct platform_device *pdev)
{
struct device *dev = &pdev->dev;
struct resource *regs;
struct tegra_vde *vde;
int irq, err;
@ -959,75 +958,39 @@ static int tegra_vde_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, vde);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "sxe");
if (!regs)
return -ENODEV;
vde->sxe = devm_ioremap_resource(dev, regs);
vde->sxe = devm_platform_ioremap_resource_byname(pdev, "sxe");
if (IS_ERR(vde->sxe))
return PTR_ERR(vde->sxe);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "bsev");
if (!regs)
return -ENODEV;
vde->bsev = devm_ioremap_resource(dev, regs);
vde->bsev = devm_platform_ioremap_resource_byname(pdev, "bsev");
if (IS_ERR(vde->bsev))
return PTR_ERR(vde->bsev);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mbe");
if (!regs)
return -ENODEV;
vde->mbe = devm_ioremap_resource(dev, regs);
vde->mbe = devm_platform_ioremap_resource_byname(pdev, "mbe");
if (IS_ERR(vde->mbe))
return PTR_ERR(vde->mbe);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "ppe");
if (!regs)
return -ENODEV;
vde->ppe = devm_ioremap_resource(dev, regs);
vde->ppe = devm_platform_ioremap_resource_byname(pdev, "ppe");
if (IS_ERR(vde->ppe))
return PTR_ERR(vde->ppe);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mce");
if (!regs)
return -ENODEV;
vde->mce = devm_ioremap_resource(dev, regs);
vde->mce = devm_platform_ioremap_resource_byname(pdev, "mce");
if (IS_ERR(vde->mce))
return PTR_ERR(vde->mce);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "tfe");
if (!regs)
return -ENODEV;
vde->tfe = devm_ioremap_resource(dev, regs);
vde->tfe = devm_platform_ioremap_resource_byname(pdev, "tfe");
if (IS_ERR(vde->tfe))
return PTR_ERR(vde->tfe);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "ppb");
if (!regs)
return -ENODEV;
vde->ppb = devm_ioremap_resource(dev, regs);
vde->ppb = devm_platform_ioremap_resource_byname(pdev, "ppb");
if (IS_ERR(vde->ppb))
return PTR_ERR(vde->ppb);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "vdma");
if (!regs)
return -ENODEV;
vde->vdma = devm_ioremap_resource(dev, regs);
vde->vdma = devm_platform_ioremap_resource_byname(pdev, "vdma");
if (IS_ERR(vde->vdma))
return PTR_ERR(vde->vdma);
regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "frameid");
if (!regs)
return -ENODEV;
vde->frameid = devm_ioremap_resource(dev, regs);
vde->frameid = devm_platform_ioremap_resource_byname(pdev, "frameid");
if (IS_ERR(vde->frameid))
return PTR_ERR(vde->frameid);