diff options
author | Thierry Reding <thierry.reding@avionic-design.de> | 2013-01-21 05:08:55 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-25 15:21:45 -0500 |
commit | 5857bd98dbd0080e6b27b51087cc9ec24f426e8b (patch) | |
tree | 5527eeca02ac9c36b280ebfb054e090728174003 | |
parent | 84dbf809fbae0591d319a7ea76e6032ff434824c (diff) |
ARM: Convert to devm_ioremap_resource()
Convert all uses of devm_request_and_ioremap() to the newly introduced
devm_ioremap_resource() which provides more consistent error handling.
Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Cc: Russell King <linux@arm.linux.org.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | arch/arm/mach-omap2/gpmc.c | 8 | ||||
-rw-r--r-- | arch/arm/mach-tegra/tegra2_emc.c | 8 | ||||
-rw-r--r-- | arch/arm/plat-omap/dmtimer.c | 8 | ||||
-rw-r--r-- | arch/arm/plat-samsung/adc.c | 8 |
4 files changed, 12 insertions, 20 deletions
diff --git a/arch/arm/mach-omap2/gpmc.c b/arch/arm/mach-omap2/gpmc.c index 8033cb747c86..64bac53da0e8 100644 --- a/arch/arm/mach-omap2/gpmc.c +++ b/arch/arm/mach-omap2/gpmc.c | |||
@@ -1134,11 +1134,9 @@ static int gpmc_probe(struct platform_device *pdev) | |||
1134 | phys_base = res->start; | 1134 | phys_base = res->start; |
1135 | mem_size = resource_size(res); | 1135 | mem_size = resource_size(res); |
1136 | 1136 | ||
1137 | gpmc_base = devm_request_and_ioremap(&pdev->dev, res); | 1137 | gpmc_base = devm_ioremap_resource(&pdev->dev, res); |
1138 | if (!gpmc_base) { | 1138 | if (IS_ERR(gpmc_base)) |
1139 | dev_err(&pdev->dev, "error: request memory / ioremap\n"); | 1139 | return PTR_ERR(gpmc_base); |
1140 | return -EADDRNOTAVAIL; | ||
1141 | } | ||
1142 | 1140 | ||
1143 | res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); | 1141 | res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); |
1144 | if (res == NULL) | 1142 | if (res == NULL) |
diff --git a/arch/arm/mach-tegra/tegra2_emc.c b/arch/arm/mach-tegra/tegra2_emc.c index e18aa2f83ebf..ce7ce42a1ac9 100644 --- a/arch/arm/mach-tegra/tegra2_emc.c +++ b/arch/arm/mach-tegra/tegra2_emc.c | |||
@@ -312,11 +312,9 @@ static int tegra_emc_probe(struct platform_device *pdev) | |||
312 | return -ENOMEM; | 312 | return -ENOMEM; |
313 | } | 313 | } |
314 | 314 | ||
315 | emc_regbase = devm_request_and_ioremap(&pdev->dev, res); | 315 | emc_regbase = devm_ioremap_resource(&pdev->dev, res); |
316 | if (!emc_regbase) { | 316 | if (IS_ERR(emc_regbase)) |
317 | dev_err(&pdev->dev, "failed to remap registers\n"); | 317 | return PTR_ERR(emc_regbase); |
318 | return -ENOMEM; | ||
319 | } | ||
320 | 318 | ||
321 | pdata = pdev->dev.platform_data; | 319 | pdata = pdev->dev.platform_data; |
322 | 320 | ||
diff --git a/arch/arm/plat-omap/dmtimer.c b/arch/arm/plat-omap/dmtimer.c index 7b433f3bddca..a0daa2fb5de6 100644 --- a/arch/arm/plat-omap/dmtimer.c +++ b/arch/arm/plat-omap/dmtimer.c | |||
@@ -808,11 +808,9 @@ static int omap_dm_timer_probe(struct platform_device *pdev) | |||
808 | return -ENOMEM; | 808 | return -ENOMEM; |
809 | } | 809 | } |
810 | 810 | ||
811 | timer->io_base = devm_request_and_ioremap(dev, mem); | 811 | timer->io_base = devm_ioremap_resource(dev, mem); |
812 | if (!timer->io_base) { | 812 | if (IS_ERR(timer->io_base)) |
813 | dev_err(dev, "%s: region already claimed.\n", __func__); | 813 | return PTR_ERR(timer->io_base); |
814 | return -ENOMEM; | ||
815 | } | ||
816 | 814 | ||
817 | if (dev->of_node) { | 815 | if (dev->of_node) { |
818 | if (of_find_property(dev->of_node, "ti,timer-alwon", NULL)) | 816 | if (of_find_property(dev->of_node, "ti,timer-alwon", NULL)) |
diff --git a/arch/arm/plat-samsung/adc.c b/arch/arm/plat-samsung/adc.c index 2d676ab50f73..ca07cb1b155a 100644 --- a/arch/arm/plat-samsung/adc.c +++ b/arch/arm/plat-samsung/adc.c | |||
@@ -386,11 +386,9 @@ static int s3c_adc_probe(struct platform_device *pdev) | |||
386 | return -ENXIO; | 386 | return -ENXIO; |
387 | } | 387 | } |
388 | 388 | ||
389 | adc->regs = devm_request_and_ioremap(dev, regs); | 389 | adc->regs = devm_ioremap_resource(dev, regs); |
390 | if (!adc->regs) { | 390 | if (IS_ERR(adc->regs)) |
391 | dev_err(dev, "failed to map registers\n"); | 391 | return PTR_ERR(adc->regs); |
392 | return -ENXIO; | ||
393 | } | ||
394 | 392 | ||
395 | ret = regulator_enable(adc->vdd); | 393 | ret = regulator_enable(adc->vdd); |
396 | if (ret) | 394 | if (ret) |