2012-08-07 07:02:36

by Thomas Meyer

[permalink] [raw]
Subject: [PATCH] drm/exynos: Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(.. [1]

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

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

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

diff -u -p a/drivers/gpu/drm/exynos/exynos_drm_gem.c b/drivers/gpu/drm/exynos/exynos_drm_gem.c
--- a/drivers/gpu/drm/exynos/exynos_drm_gem.c
+++ b/drivers/gpu/drm/exynos/exynos_drm_gem.c
@@ -122,7 +122,7 @@ fail:
__free_page(pages[i]);

drm_free_large(pages);
- return ERR_PTR(PTR_ERR(p));
+ return ERR_CAST(p);
}

static void exynos_gem_put_pages(struct drm_gem_object *obj,


Subject: RE: [PATCH] drm/exynos: Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(.. [1]



> -----Original Message-----
> From: Thomas Meyer [mailto:[email protected]]
> Sent: Tuesday, August 07, 2012 3:57 PM
> To: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; dri-
> [email protected]; [email protected]
> Subject: [PATCH] drm/exynos: Use ERR_CAST inlined function instead of
> ERR_PTR(PTR_ERR(.. [1]
>
> The semantic patch that makes this change is available
> in scripts/coccinelle/api/err_cast.cocci.
>
> More information about semantic patching is available at
> http://coccinelle.lip6.fr/
>
> Signed-off-by: Thomas Meyer <[email protected]>
> ---
>
> diff -u -p a/drivers/gpu/drm/exynos/exynos_drm_gem.c
> b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> --- a/drivers/gpu/drm/exynos/exynos_drm_gem.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> @@ -122,7 +122,7 @@ fail:
> __free_page(pages[i]);
>
> drm_free_large(pages);
> - return ERR_PTR(PTR_ERR(p));
> + return ERR_CAST(p);
> }
>
> static void exynos_gem_put_pages(struct drm_gem_object *obj,

Applied.

Thanks for your patch.