Commit e9c9e5ae authored by Puthikorn Voravootivat's avatar Puthikorn Voravootivat Committed by Jani Nikula

drm/i915: Set backlight mode before enable backlight

We should set backlight mode register before set register to
enable the backlight.
Signed-off-by: default avatarPuthikorn Voravootivat <puthik@chromium.org>
Reviewed-by: default avatarDhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170511230225.142870-6-puthik@chromium.org
parent 73ab484c
...@@ -99,8 +99,6 @@ static void intel_dp_aux_enable_backlight(struct intel_connector *connector) ...@@ -99,8 +99,6 @@ static void intel_dp_aux_enable_backlight(struct intel_connector *connector)
uint8_t dpcd_buf = 0; uint8_t dpcd_buf = 0;
uint8_t edp_backlight_mode = 0; uint8_t edp_backlight_mode = 0;
set_aux_backlight_enable(intel_dp, true);
if (drm_dp_dpcd_readb(&intel_dp->aux, if (drm_dp_dpcd_readb(&intel_dp->aux,
DP_EDP_BACKLIGHT_MODE_SET_REGISTER, &dpcd_buf) != 1) { DP_EDP_BACKLIGHT_MODE_SET_REGISTER, &dpcd_buf) != 1) {
DRM_DEBUG_KMS("Failed to read DPCD register 0x%x\n", DRM_DEBUG_KMS("Failed to read DPCD register 0x%x\n",
...@@ -127,6 +125,8 @@ static void intel_dp_aux_enable_backlight(struct intel_connector *connector) ...@@ -127,6 +125,8 @@ static void intel_dp_aux_enable_backlight(struct intel_connector *connector)
default: default:
break; break;
} }
set_aux_backlight_enable(intel_dp, true);
} }
static void intel_dp_aux_disable_backlight(struct intel_connector *connector) static void intel_dp_aux_disable_backlight(struct intel_connector *connector)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment