From: Russell King Date: Mon, 30 Jul 2018 10:52:34 +0000 (+0100) Subject: drm/armada: clean up armada_drm_crtc_page_flip() X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=1729f56010a960f182a36e09ce8d6504fa637d76;p=openwrt%2Fstaging%2Fblogic.git drm/armada: clean up armada_drm_crtc_page_flip() drm_mode_page_flip_ioctl() already takes care of checking the framebuffer format, and also assigns primary->fb after a successful call to this handler. These are both redundant, and can be removed. Signed-off-by: Russell King --- diff --git a/drivers/gpu/drm/armada/armada_crtc.c b/drivers/gpu/drm/armada/armada_crtc.c index 42a40daff132..78bb3d51417b 100644 --- a/drivers/gpu/drm/armada/armada_crtc.c +++ b/drivers/gpu/drm/armada/armada_crtc.c @@ -1039,10 +1039,6 @@ static int armada_drm_crtc_page_flip(struct drm_crtc *crtc, unsigned i; int ret; - /* We don't support changing the pixel format */ - if (fb->format != crtc->primary->fb->format) - return -EINVAL; - work = armada_drm_crtc_alloc_plane_work(dcrtc->crtc.primary); if (!work) return -ENOMEM; @@ -1068,14 +1064,6 @@ static int armada_drm_crtc_page_flip(struct drm_crtc *crtc, return ret; } - /* - * Don't take a reference on the new framebuffer; - * drm_mode_page_flip_ioctl() has already grabbed a reference and - * will _not_ drop that reference on successful return from this - * function. Simply mark this new framebuffer as the current one. - */ - dcrtc->crtc.primary->fb = fb; - /* * Finally, if the display is blanked, we won't receive an * interrupt, so complete it now.