drm/i915/dsi: Group MIPI_SEQ_BACKLIGHT_ON/OFF with panel_[en|dis]able_backlight
authorHans de Goede <hdegoede@redhat.com>
Wed, 1 Mar 2017 13:15:02 +0000 (15:15 +0200)
committerJani Nikula <jani.nikula@intel.com>
Wed, 1 Mar 2017 13:57:59 +0000 (15:57 +0200)
Execute the MIPI_SEQ_BACKLIGHT_ON/OFF VBT sequences at the same time as
we call intel_panel_enable_backlight() / intel_panel_disable_backlight().

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Bob Paauwe <bob.j.paauwe@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1488374106-4949-7-git-send-email-jani.nikula@intel.com
drivers/gpu/drm/i915/intel_dsi.c

index b7b590d78b31e14f2b77ede17a1ac3b15204c289..b132841587cc70c31b8791ba80ab57d108aef782 100644 (file)
@@ -829,12 +829,12 @@ static void intel_dsi_pre_enable(struct intel_encoder *encoder,
                msleep(100);
 
                intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_DISPLAY_ON);
-               intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_BACKLIGHT_ON);
 
                intel_dsi_port_enable(encoder);
        }
 
        intel_panel_enable_backlight(intel_dsi->attached_connector);
+       intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_BACKLIGHT_ON);
 }
 
 static void intel_dsi_enable_nop(struct intel_encoder *encoder,
@@ -860,6 +860,7 @@ static void intel_dsi_pre_disable(struct intel_encoder *encoder,
 
        DRM_DEBUG_KMS("\n");
 
+       intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_BACKLIGHT_OFF);
        intel_panel_disable_backlight(intel_dsi->attached_connector);
 
        /*
@@ -915,7 +916,6 @@ static void intel_dsi_post_disable(struct intel_encoder *encoder,
         * if disable packets are sent before sending shutdown packet then in
         * some next enable sequence send turn on packet error is observed
         */
-       intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_BACKLIGHT_OFF);
        intel_dsi_exec_vbt_sequence(intel_dsi, MIPI_SEQ_DISPLAY_OFF);
 
        /* Transition to LP-00 */