sh-pfc: Use devm_ioremap_nocache()
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Sat, 15 Dec 2012 22:50:49 +0000 (23:50 +0100)
committerSimon Horman <horms+renesas@verge.net.au>
Fri, 25 Jan 2013 00:24:21 +0000 (09:24 +0900)
Replace probe-time ioremap_nocache() call with devm_ioremap_nocache()
and get rid of the corresponding iounmap() call.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
drivers/sh/pfc/core.c

index 54919026ac12a6d4dab9409373f01d6e0c0a4b76..8ec57bd5b1f9b9284bb6762882ff04ac2960f2f0 100644 (file)
 
 #include "core.h"
 
-static void pfc_iounmap(struct sh_pfc *pfc)
-{
-       int k;
-
-       for (k = 0; k < pfc->pdata->num_resources; k++)
-               if (pfc->window[k].virt)
-                       iounmap(pfc->window[k].virt);
-}
-
 static int pfc_ioremap(struct sh_pfc *pfc)
 {
        struct resource *res;
@@ -53,12 +44,10 @@ static int pfc_ioremap(struct sh_pfc *pfc)
                WARN_ON(resource_type(res) != IORESOURCE_MEM);
                pfc->window[k].phys = res->start;
                pfc->window[k].size = resource_size(res);
-               pfc->window[k].virt = ioremap_nocache(res->start,
-                                                        resource_size(res));
-               if (!pfc->window[k].virt) {
-                       pfc_iounmap(pfc);
+               pfc->window[k].virt = devm_ioremap_nocache(pfc->dev, res->start,
+                                                          resource_size(res));
+               if (!pfc->window[k].virt)
                        return -ENOMEM;
-               }
        }
 
        return 0;
@@ -524,7 +513,7 @@ static int sh_pfc_probe(struct platform_device *pdev)
         */
        ret = sh_pfc_register_pinctrl(pfc);
        if (unlikely(ret != 0))
-               goto err;
+               return ret;
 
 #ifdef CONFIG_GPIO_SH_PFC
        /*
@@ -546,10 +535,6 @@ static int sh_pfc_probe(struct platform_device *pdev)
        pr_info("%s support registered\n", pdata->name);
 
        return 0;
-
-err:
-       pfc_iounmap(pfc);
-       return ret;
 }
 
 static int sh_pfc_remove(struct platform_device *pdev)
@@ -561,8 +546,6 @@ static int sh_pfc_remove(struct platform_device *pdev)
 #endif
        sh_pfc_unregister_pinctrl(pfc);
 
-       pfc_iounmap(pfc);
-
        platform_set_drvdata(pdev, NULL);
 
        return 0;