[PATCH 10/25] drm/gem: Use container_of in drm_gem_object_free

2015-10-15 Thread David Herrmann
Hi

On Thu, Oct 15, 2015 at 9:36 AM, Daniel Vetter  
wrote:
> Just a random thing I spotted while reading code - better safe than
> sorry.
>
> Signed-off-by: Daniel Vetter 

Reviewed-by: David Herrmann 

Thanks
David

> ---
>  drivers/gpu/drm/drm_gem.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
> index 7dc4a8a066a3..ab8ea42264f4 100644
> --- a/drivers/gpu/drm/drm_gem.c
> +++ b/drivers/gpu/drm/drm_gem.c
> @@ -763,7 +763,8 @@ EXPORT_SYMBOL(drm_gem_object_release);
>  void
>  drm_gem_object_free(struct kref *kref)
>  {
> -   struct drm_gem_object *obj = (struct drm_gem_object *) kref;
> +   struct drm_gem_object *obj =
> +   container_of(kref, struct drm_gem_object, refcount);
> struct drm_device *dev = obj->dev;
>
> WARN_ON(!mutex_is_locked(>struct_mutex));
> --
> 2.5.1
>
> ___
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 10/25] drm/gem: Use container_of in drm_gem_object_free

2015-10-15 Thread Daniel Vetter
Just a random thing I spotted while reading code - better safe than
sorry.

Signed-off-by: Daniel Vetter 
---
 drivers/gpu/drm/drm_gem.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
index 7dc4a8a066a3..ab8ea42264f4 100644
--- a/drivers/gpu/drm/drm_gem.c
+++ b/drivers/gpu/drm/drm_gem.c
@@ -763,7 +763,8 @@ EXPORT_SYMBOL(drm_gem_object_release);
 void
 drm_gem_object_free(struct kref *kref)
 {
-   struct drm_gem_object *obj = (struct drm_gem_object *) kref;
+   struct drm_gem_object *obj =
+   container_of(kref, struct drm_gem_object, refcount);
struct drm_device *dev = obj->dev;

WARN_ON(!mutex_is_locked(>struct_mutex));
-- 
2.5.1