[PATCH] drm: Release resources with a safer function

2016-10-10 Thread Daniel Vetter
On Fri, Oct 07, 2016 at 09:27:41AM +0200, Christophe JAILLET wrote: > We should use 'ida_simple_remove()' instead of 'ida_remove()' when freeing > resources allocated with 'ida_simple_get()'. > > Signed-off-by: Christophe JAILLET Applied to drm-misc, thanks. -Daniel > --- > drivers/gpu/drm/drm

[PATCH] drm: Release resources with a safer function

2016-10-07 Thread Christophe JAILLET
We should use 'ida_simple_remove()' instead of 'ida_remove()' when freeing resources allocated with 'ida_simple_get()'. Signed-off-by: Christophe JAILLET --- drivers/gpu/drm/drm_connector.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_connector.

[PATCH] drm: Release resources with a safer function

2016-10-05 Thread Daniel Vetter
On Sun, Oct 02, 2016 at 08:01:22AM +0200, Christophe JAILLET wrote: > We should use 'ida_simple_remove()' instead of 'ida_remove()' when freeing > resources allocated with 'ida_simple_get()'. > > This as been spotted with the following coccinelle script which tries to > detect missing 'ida_simple_

[PATCH] drm: Release resources with a safer function

2016-10-03 Thread Ville Syrjälä
On Sun, Oct 02, 2016 at 08:01:22AM +0200, Christophe JAILLET wrote: > We should use 'ida_simple_remove()' instead of 'ida_remove()' when freeing > resources allocated with 'ida_simple_get()'. Should fix drm_connector_cleanup() then as well... > > This as been spotted with the following coccinell

[PATCH] drm: Release resources with a safer function

2016-10-02 Thread Christophe JAILLET
We should use 'ida_simple_remove()' instead of 'ida_remove()' when freeing resources allocated with 'ida_simple_get()'. This as been spotted with the following coccinelle script which tries to detect missing 'ida_simple_remove()' call in error handling paths. /// @@ expression x; iden