ASoC: xlnx: Use devm_platform_ioremap_resource() in xlnx_formatter_pcm_probe()
authorMarkus Elfring <elfring@users.sourceforge.net>
Thu, 19 Sep 2019 15:27:57 +0000 (17:27 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 1 Oct 2019 11:09:17 +0000 (12:09 +0100)
Simplify this function implementation by using a known wrapper function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Link: https://lore.kernel.org/r/8f7cf483-6ab3-d00f-5606-863e9f5b31fc@web.de
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/xilinx/xlnx_formatter_pcm.c

index 48970ef..fb652e7 100644 (file)
@@ -564,7 +564,6 @@ static int xlnx_formatter_pcm_probe(struct platform_device *pdev)
        int ret;
        u32 val;
        struct xlnx_pcm_drv_data *aud_drv_data;
-       struct resource *res;
        struct device *dev = &pdev->dev;
 
        aud_drv_data = devm_kzalloc(dev, sizeof(*aud_drv_data), GFP_KERNEL);
@@ -584,13 +583,7 @@ static int xlnx_formatter_pcm_probe(struct platform_device *pdev)
                return ret;
        }
 
-       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!res) {
-               dev_err(dev, "audio formatter node:addr to resource failed\n");
-               ret = -ENXIO;
-               goto clk_err;
-       }
-       aud_drv_data->mmio = devm_ioremap_resource(dev, res);
+       aud_drv_data->mmio = devm_platform_ioremap_resource(pdev, 0);
        if (IS_ERR(aud_drv_data->mmio)) {
                dev_err(dev, "audio formatter ioremap failed\n");
                ret = PTR_ERR(aud_drv_data->mmio);