diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-03-04 04:06:27 -0500 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2013-04-15 00:21:18 -0400 |
commit | bcc7fa9547e7fa3b2f9b54f83ff10cd59f0f9fc6 (patch) | |
tree | 4d186505637bf81f31da3bb223cfb622de8148ae /drivers/dma/pl330.c | |
parent | d86b2f298e6de124984f5d5817ed1e6e759b3ada (diff) |
dma: pl330: Convert to devm_ioremap_resource()
Use the newly introduced devm_ioremap_resource() instead of
devm_request_and_ioremap() which provides more consistent error handling.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Reviewed-by: Thierry Reding <thierry.reding@avionic-design.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma/pl330.c')
-rw-r--r-- | drivers/dma/pl330.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c index 31619c08a760..a17553f7c028 100644 --- a/drivers/dma/pl330.c +++ b/drivers/dma/pl330.c | |||
@@ -26,6 +26,7 @@ | |||
26 | #include <linux/scatterlist.h> | 26 | #include <linux/scatterlist.h> |
27 | #include <linux/of.h> | 27 | #include <linux/of.h> |
28 | #include <linux/of_dma.h> | 28 | #include <linux/of_dma.h> |
29 | #include <linux/err.h> | ||
29 | 30 | ||
30 | #include "dmaengine.h" | 31 | #include "dmaengine.h" |
31 | #define PL330_MAX_CHAN 8 | 32 | #define PL330_MAX_CHAN 8 |
@@ -2903,9 +2904,9 @@ pl330_probe(struct amba_device *adev, const struct amba_id *id) | |||
2903 | pi->mcbufsz = pdat ? pdat->mcbuf_sz : 0; | 2904 | pi->mcbufsz = pdat ? pdat->mcbuf_sz : 0; |
2904 | 2905 | ||
2905 | res = &adev->res; | 2906 | res = &adev->res; |
2906 | pi->base = devm_request_and_ioremap(&adev->dev, res); | 2907 | pi->base = devm_ioremap_resource(&adev->dev, res); |
2907 | if (!pi->base) | 2908 | if (IS_ERR(pi->base)) |
2908 | return -ENXIO; | 2909 | return PTR_ERR(pi->base); |
2909 | 2910 | ||
2910 | amba_set_drvdata(adev, pdmac); | 2911 | amba_set_drvdata(adev, pdmac); |
2911 | 2912 | ||