m68k: amig: Use PTR_RET rather than if(IS_ERR(.. [1]

PATCH Applied Thomas Meyer 2 participants 3 messages Aug 6, 2012 – Aug 18, 2012 m68k architecture

lore

Share & tools

lore

Thomas Meyer

author

2012-08-06 18:33 UTC #0 lore raw

PATCH 1/1 m68k: amig: Use PTR_RET rather than if(IS_ERR(.. [1]

The semantic patch that makes this change is available in scripts/coccinelle/api/ptr_ret.cocci.

More information about semantic patching is available at
http://coccinelle.lip6.fr/

Signed-off-by Thomas Meyer <[email protected]>

---

arch/m68k/amiga/platform.c +1 -4

diff -u -p a/arch/m68k/amiga/platform.c b/arch/m68k/amiga/platform.c
@@ -56,10 +56,7 @@ static int __init amiga_init_bus(void)
 	n = AMIGAHW_PRESENT(ZORRO3) ? 4 : 2;
 	pdev = platform_device_register_simple("amiga-zorro", -1,
 					       zorro_resources, n);
-	if (IS_ERR(pdev))
-		return PTR_ERR(pdev);
-
-	return 0;
+	return PTR_RET(pdev);
 }
 
 subsys_initcall(amiga_init_bus);

Geert Uytterhoeven

maintainer Applied

2012-08-18 13:21 UTC #1 lore raw

Thanks, applied and queued for 3.7.

Quoted 14 lines · Thomas Meyer

The semantic patch that makes this change is available in scripts/coccinelle/api/ptr_ret.cocci.
More information about semantic patching is available at
http://coccinelle.lip6.fr/
Signed-off-by Thomas Meyer <[email protected]>
---
diff -u -p a/arch/m68k/amiga/platform.c b/arch/m68k/amiga/platform.c
arch/m68k/amiga/platform.c +0 -0
@@ -56,10 +56,7 @@ static int __init amiga_init_bus(void)
        n = AMIGAHW_PRESENT(ZORRO3) ? 4 : 2;

arch/m68k/amiga/platform.c static int __init amiga_init_bus(void) · line 62

Quoted 10 lines

arch/m68k/amiga/platform.c +1 -4

        pdev = platform_device_register_simple("amiga-zorro", -1,
                                               zorro_resources, n);
-       if (IS_ERR(pdev))
-               return PTR_ERR(pdev);
-
-       return 0;
+       return PTR_RET(pdev);
 }
 
 subsys_initcall(amiga_init_bus);

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Geert Uytterhoeven

maintainer Applied

2012-08-18 13:21 UTC #2 lore raw

In reply to Thomas Meyer

Thanks, applied and queued for 3.7.

Quoted 14 lines · Thomas Meyer

The semantic patch that makes this change is available in scripts/coccinelle/api/ptr_ret.cocci.
More information about semantic patching is available at
http://coccinelle.lip6.fr/
Signed-off-by Thomas Meyer <[email protected]>
---
diff -u -p a/arch/m68k/amiga/platform.c b/arch/m68k/amiga/platform.c
arch/m68k/amiga/platform.c +0 -0
@@ -56,10 +56,7 @@ static int __init amiga_init_bus(void)
        n = AMIGAHW_PRESENT(ZORRO3) ? 4 : 2;

arch/m68k/amiga/platform.c static int __init amiga_init_bus(void) · line 62

Quoted 10 lines

arch/m68k/amiga/platform.c +1 -4

        pdev = platform_device_register_simple("amiga-zorro", -1,
                                               zorro_resources, n);
-       if (IS_ERR(pdev))
-               return PTR_ERR(pdev);
-
-       return 0;
+       return PTR_RET(pdev);
 }
 
 subsys_initcall(amiga_init_bus);

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds