Messages in this thread Patch in this message | | | From | Sachin Kamat <> | Subject | [PATCH 1/1] dma: pl330: Convert to devm_ioremap_resource() | Date | Thu, 31 Jan 2013 12:37:04 +0530 |
| |
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> Cc: Vinod Koul <vinod.koul@intel.com> Cc: Thierry Reding <thierry.reding@avionic-design.de> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- This change is based against linux-next tree (20130128). This change however introduces the followign sparse warning: drivers/dma/pl330.c:2883:22: warning: incorrect type in argument 1 (different address spaces) drivers/dma/pl330.c:2883:22: expected void const *ptr drivers/dma/pl330.c:2883:22: got void [noderef] <asn:2>*base drivers/dma/pl330.c:2884:34: warning: incorrect type in argument 1 (different address spaces) drivers/dma/pl330.c:2884:34: expected void const *ptr drivers/dma/pl330.c:2884:34: got void [noderef] <asn:2>*base --- 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 316a43e..142fe4d 100644 --- a/drivers/dma/pl330.c +++ b/drivers/dma/pl330.c @@ -25,6 +25,7 @@ #include <linux/amba/pl330.h> #include <linux/scatterlist.h> #include <linux/of.h> +#include <linux/err.h> #include "dmaengine.h" #define PL330_MAX_CHAN 8 @@ -2878,9 +2879,9 @@ pl330_probe(struct amba_device *adev, const struct amba_id *id) pi->mcbufsz = pdat ? pdat->mcbuf_sz : 0; res = &adev->res; - pi->base = devm_request_and_ioremap(&adev->dev, res); - if (!pi->base) - return -ENXIO; + pi->base = devm_ioremap_resource(&adev->dev, res); + if (IS_ERR(pi->base)) + return PTR_ERR(pi->base); amba_set_drvdata(adev, pdmac); -- 1.7.4.1
| |