drm/exynos: Remove crtc->disable
authorSean Paul <seanpaul@chromium.org>
Thu, 21 Feb 2013 18:35:49 +0000 (13:35 -0500)
committerChromeBot <chrome-bot@google.com>
Fri, 22 Feb 2013 13:01:26 +0000 (05:01 -0800)
Remove the crtc->disable callback since it's not needed and most likely
incorrect anyways. DRM will fall back to dpms if disable is not defined,
and that will simplify the various paths we have. Furthermore, all
disable did was disable the default overlay on the crtc, which is not
the intended result.

BUG=None
TEST=Tested on snow with boot/hotplug/suspend/idle

Change-Id: I2b134662bfa56d35a0de9fbf761d7a8a3708009e
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/43725

drivers/gpu/drm/exynos/exynos_drm_crtc.c

index db9167a..09d98ff 100644 (file)
@@ -299,16 +299,6 @@ static void exynos_drm_crtc_dpms(struct drm_crtc *crtc, int mode)
 #endif
 }
 
-static void exynos_drm_crtc_disable(struct drm_crtc *crtc)
-{
-       struct exynos_drm_crtc *exynos_crtc = to_exynos_crtc(crtc);
-       struct exynos_drm_overlay *overlay = &exynos_crtc->overlay;
-       int win = overlay->zpos;
-
-       exynos_drm_fn_encoder(crtc, &win,
-               exynos_drm_encoder_crtc_disable);
-}
-
 static void exynos_drm_crtc_prepare(struct drm_crtc *crtc)
 {
        DRM_DEBUG_KMS("%s\n", __FILE__);
@@ -420,7 +410,6 @@ static void exynos_drm_crtc_load_lut(struct drm_crtc *crtc)
 
 static struct drm_crtc_helper_funcs exynos_crtc_helper_funcs = {
        .dpms           = exynos_drm_crtc_dpms,
-       .disable        = exynos_drm_crtc_disable,
        .prepare        = exynos_drm_crtc_prepare,
        .commit         = exynos_drm_crtc_commit,
        .mode_fixup     = exynos_drm_crtc_mode_fixup,