Re: [Intel-gfx] [PATCH 1/7] drm/plane-helper: Test for plane disable earlier

2014-12-11 Thread Daniel Vetter
On Thu, Dec 11, 2014 at 07:20:57AM -0800, Matt Roper wrote: > drm_plane_helper_check_update() currently uses crtc before testing whether > we're disabling the plane (fb == NULL). Move the fb test before the first > crtc > usage so that crtc == NULL doesn't have to be handled by the caller. > > S

[Intel-gfx] [PATCH 1/7] drm/plane-helper: Test for plane disable earlier

2014-12-11 Thread Matt Roper
drm_plane_helper_check_update() currently uses crtc before testing whether we're disabling the plane (fb == NULL). Move the fb test before the first crtc usage so that crtc == NULL doesn't have to be handled by the caller. Signed-off-by: Matt Roper --- drivers/gpu/drm/drm_plane_helper.c | 16 ++

Re: [Intel-gfx] [PATCH 1/7] drm/plane-helper: Test for plane disable earlier

2014-12-10 Thread Daniel Vetter
On Tue, Dec 09, 2014 at 11:53:37AM -0800, Matt Roper wrote: > drm_plane_helper_check_update() currently uses crtc before testing whether > we're disabling the plane (fb == NULL). Move the fb test before the first > crtc > usage so that crtc == NULL doesn't have to be handled by the caller. > > S

[Intel-gfx] [PATCH 1/7] drm/plane-helper: Test for plane disable earlier

2014-12-09 Thread Matt Roper
drm_plane_helper_check_update() currently uses crtc before testing whether we're disabling the plane (fb == NULL). Move the fb test before the first crtc usage so that crtc == NULL doesn't have to be handled by the caller. Signed-off-by: Matt Roper --- drivers/gpu/drm/drm_plane_helper.c | 16 ++