diff options
| author | Julia Lawall <julia@diku.dk> | 2011-01-27 16:49:06 +0100 |
|---|---|---|
| committer | Tony Lindgren <tony@atomide.com> | 2011-01-28 01:25:35 +0100 |
| commit | 0e6d8cad448bde3d846961bb43db15daae94562e (patch) | |
| tree | 51b1730d736c97a56d0fa23f7f61d2676ac4e620 | |
| parent | arm: omap2: mux: fix compile warning (diff) | |
| download | linux-0e6d8cad448bde3d846961bb43db15daae94562e.tar.xz linux-0e6d8cad448bde3d846961bb43db15daae94562e.zip | |
arch/arm/mach-omap2/dma.c: Convert IS_ERR result to PTR_ERR
This code elsewhere returns a negative constant to an indicate an error,
while IS_ERR returns the result of a >= operation.
The semantic patch that fixes this problem is as follows:
(http://coccinelle.lip6.fr/)
// <smpl>
@@
expression x;
@@
if (...) { ...
- return IS_ERR(x);
+ return PTR_ERR(x);
}
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Jarkko Nikula <jhnikula@gmail.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
| -rw-r--r-- | arch/arm/mach-omap2/dma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/mach-omap2/dma.c b/arch/arm/mach-omap2/dma.c index d2f15f5cfd36..34922b2d2e3f 100644 --- a/arch/arm/mach-omap2/dma.c +++ b/arch/arm/mach-omap2/dma.c @@ -264,7 +264,7 @@ static int __init omap2_system_dma_init_dev(struct omap_hwmod *oh, void *unused) if (IS_ERR(od)) { pr_err("%s: Cant build omap_device for %s:%s.\n", __func__, name, oh->name); - return IS_ERR(od); + return PTR_ERR(od); } mem = platform_get_resource(&od->pdev, IORESOURCE_MEM, 0); |