ASoC: Convert to devm_ioremap_resource()
authorThierry Reding <thierry.reding@avionic-design.de>
Mon, 21 Jan 2013 10:09:26 +0000 (11:09 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 22 Jan 2013 19:41:58 +0000 (11:41 -0800)
Convert all uses of devm_request_and_ioremap() to the newly introduced
devm_ioremap_resource() which provides more consistent error handling.

devm_ioremap_resource() provides its own error messages so all explicit
error messages can be removed from the failure code paths.

Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Liam Girdwood <lrg@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
sound/soc/cirrus/ep93xx-ac97.c
sound/soc/cirrus/ep93xx-i2s.c
sound/soc/codecs/jz4740.c
sound/soc/fsl/imx-audmux.c
sound/soc/fsl/imx-ssi.c
sound/soc/kirkwood/kirkwood-i2s.c
sound/soc/mxs/mxs-saif.c
sound/soc/pxa/mmp-sspa.c

index f3f50e6fd6eb29218f622664b37ba7065946b57c..1738d28fb04f586876c6e84c59fa3c7991d95ed4 100644 (file)
@@ -11,6 +11,7 @@
  */
 
 #include <linux/delay.h>
+#include <linux/err.h>
 #include <linux/io.h>
 #include <linux/init.h>
 #include <linux/module.h>
@@ -367,9 +368,9 @@ static int ep93xx_ac97_probe(struct platform_device *pdev)
        if (!res)
                return -ENODEV;
 
-       info->regs = devm_request_and_ioremap(&pdev->dev, res);
-       if (!info->regs)
-               return -ENXIO;
+       info->regs = devm_ioremap_resource(&pdev->dev, res);
+       if (IS_ERR(info->regs))
+               return PTR_ERR(info->regs);
 
        irq = platform_get_irq(pdev, 0);
        if (!irq)
index 3365d4e843b7382a663380bd9311bdd762828be0..323ed69b797559906c2f0f5b8c51200c491fca0d 100644 (file)
@@ -380,9 +380,9 @@ static int ep93xx_i2s_probe(struct platform_device *pdev)
        if (!res)
                return -ENODEV;
 
-       info->regs = devm_request_and_ioremap(&pdev->dev, res);
-       if (!info->regs)
-               return -ENXIO;
+       info->regs = devm_ioremap_resource(&pdev->dev, res);
+       if (IS_ERR(info->regs))
+               return PTR_ERR(info->regs);
 
        info->mclk = clk_get(&pdev->dev, "mclk");
        if (IS_ERR(info->mclk)) {
index d991529e1aff1a021210b8887a35b2b609601130..5f607b35b68b4c46aa6e3f8337730d66647d877b 100644 (file)
@@ -361,9 +361,9 @@ static int jz4740_codec_probe(struct platform_device *pdev)
                return -ENOMEM;
 
        mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       base = devm_request_and_ioremap(&pdev->dev, mem);
-       if (!base)
-               return -EBUSY;
+       base = devm_ioremap_resource(&pdev->dev, mem);
+       if (IS_ERR(base))
+               return PTR_ERR(base);
 
        jz4740_codec->regmap = devm_regmap_init_mmio(&pdev->dev, base,
                                            &jz4740_codec_regmap_config);
index 251f4d981e0c6e8882c6f0d9c23efad821e87e05..c21ac9c7081e076ab2fe49aaee2d681012eb41d1 100644 (file)
@@ -252,9 +252,9 @@ static int imx_audmux_probe(struct platform_device *pdev)
                        of_match_device(imx_audmux_dt_ids, &pdev->dev);
 
        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       audmux_base = devm_request_and_ioremap(&pdev->dev, res);
-       if (!audmux_base)
-               return -EADDRNOTAVAIL;
+       audmux_base = devm_ioremap_resource(&pdev->dev, res);
+       if (IS_ERR(audmux_base))
+               return PTR_ERR(audmux_base);
 
        pinctrl = devm_pinctrl_get_select_default(&pdev->dev);
        if (IS_ERR(pinctrl)) {
index 3b480423747fc3d00f70978173a676736057c562..55464a5b0706161fffd7c94bc3c524f4b3e40224 100644 (file)
@@ -550,10 +550,9 @@ static int imx_ssi_probe(struct platform_device *pdev)
                goto failed_get_resource;
        }
 
-       ssi->base = devm_request_and_ioremap(&pdev->dev, res);
-       if (!ssi->base) {
-               dev_err(&pdev->dev, "ioremap failed\n");
-               ret = -ENODEV;
+       ssi->base = devm_ioremap_resource(&pdev->dev, res);
+       if (IS_ERR(ssi->base)) {
+               ret = PTR_ERR(ssi->base);
                goto failed_register;
        }
 
index 282d8b1163ba9583afd6e38e38400ce7efea29e9..c74c89065493cec9e90963bf1162be0ed7e5662d 100644 (file)
@@ -472,11 +472,9 @@ static int kirkwood_i2s_dev_probe(struct platform_device *pdev)
                return -ENXIO;
        }
 
-       priv->io = devm_request_and_ioremap(&pdev->dev, mem);
-       if (!priv->io) {
-               dev_err(&pdev->dev, "devm_request_and_ioremap failed\n");
-               return -ENOMEM;
-       }
+       priv->io = devm_ioremap_resource(&pdev->dev, mem);
+       if (IS_ERR(priv->io))
+               return PTR_ERR(priv->io);
 
        priv->irq = platform_get_irq(pdev, 0);
        if (priv->irq <= 0) {
index 365d9d27a3216b2c8969451c83b0e49e0bbc9520..b327709eb1927d457021754b80d8068dc2a75fdc 100644 (file)
@@ -724,11 +724,9 @@ static int mxs_saif_probe(struct platform_device *pdev)
 
        iores = platform_get_resource(pdev, IORESOURCE_MEM, 0);
 
-       saif->base = devm_request_and_ioremap(&pdev->dev, iores);
-       if (!saif->base) {
-               dev_err(&pdev->dev, "ioremap failed\n");
-               return -ENODEV;
-       }
+       saif->base = devm_ioremap_resource(&pdev->dev, iores);
+       if (IS_ERR(saif->base))
+               return PTR_ERR(saif->base);
 
        dmares = platform_get_resource(pdev, IORESOURCE_DMA, 0);
        if (!dmares) {
index 41c3a09b53eaab72824ffe36862407b18c56d32b..9140c4abafbc98f4ea5a464f26cefba7286c258e 100644 (file)
@@ -429,9 +429,9 @@ static int asoc_mmp_sspa_probe(struct platform_device *pdev)
        if (res == NULL)
                return -ENOMEM;
 
-       priv->sspa->mmio_base = devm_request_and_ioremap(&pdev->dev, res);
-       if (priv->sspa->mmio_base == NULL)
-               return -ENODEV;
+       priv->sspa->mmio_base = devm_ioremap_resource(&pdev->dev, res);
+       if (IS_ERR(priv->sspa->mmio_base))
+               return PTR_ERR(priv->sspa->mmio_base);
 
        priv->sspa->clk = devm_clk_get(&pdev->dev, NULL);
        if (IS_ERR(priv->sspa->clk))