[PATCH 02/13] drm/mgag200: Remove unecessary NULL check in gem_free

2014-04-07 Thread Ian Romanick
On 04/05/2014 02:44 AM, Daniel Vetter wrote: > The ->gem_free_object never gets called with a NULL pointer, the check > is redundant. Also checking after the upcast allows compilers to elide > it anyway. Right... because if obj was NULL, subtracting some offset from it won't still be NULL. > Spot

[PATCH 02/13] drm/mgag200: Remove unecessary NULL check in gem_free

2014-04-05 Thread Daniel Vetter
The ->gem_free_object never gets called with a NULL pointer, the check is redundant. Also checking after the upcast allows compilers to elide it anyway. Spotted by coverity. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/mgag200/mgag200_main.c | 2 -- 1 file changed, 2 deletions(-) diff --gi