Skip to content

Commit

Permalink
sh-pfc: Use devm_ioremap_nocache()
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Laurent Pinchart authored and Simon Horman committed Jan 25, 2013
1 parent 1724acf commit c9fa88e
Showing 1 changed file with 4 additions and 21 deletions.
25 changes: 4 additions & 21 deletions drivers/sh/pfc/core.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,6 @@

#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;
Expand All @@ -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;
Expand Down Expand Up @@ -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
/*
Expand All @@ -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)
Expand All @@ -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;
Expand Down

0 comments on commit c9fa88e

Please sign in to comment.