drm/mediatek: Use drm_plane_helper_check_state()
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 26 Jul 2016 16:07:03 +0000 (19:07 +0300)
committerSean Paul <seanpaul@chromium.org>
Mon, 8 Aug 2016 18:19:58 +0000 (14:19 -0400)
commit0e4faf679e67b5ab55752d18fc82f3c7351b2f03
treef61edda4127ecb4ce3c82155c447c3e996446cca
parentf9b96be0ecb349c65875b55c14345d0604405e88
drm/mediatek: Use drm_plane_helper_check_state()

Replace the use of drm_plane_helper_check_update() with
drm_plane_helper_check_state() since we have a plane state.

This also eliminates the double clipping the driver was doing
in both check and commit phases). And it should fix src coordinate
addr adjustement. Previously the driver was expecting negative dst
coordinates after clipping, which is not going happen, so any clipping
induced addr adjustment simply didn't happen. Neither did the driver
respect any user configured src coordinates, so panning and such would
have been totally broken. It should be all good now.

Cc: CK Hu <ck.hu@mediatek.com>
Cc: linux-mediatek@lists.infradead.org
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Bibby Hsieh <bibby.hsieh@mediatek.com>
Tested-by: Bibby Hsieh <bibby.hsieh@mediatek.com>
Acked-by: CK Hu <ck.hu@mediatek.com>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: http://patchwork.freedesktop.org/patch/msgid/1469549224-1860-9-git-send-email-ville.syrjala@linux.intel.com
drivers/gpu/drm/mediatek/mtk_drm_plane.c