From: Jesse Barnes <[email protected]> Per eDP spec, we must disable the backlight in order to power down the panel. However, in our code, we have always disabled the backlight before we try to turn off the panel. The assertions from the previous patch make sure this is the case.
Signed-off-by: Jesse Barnes <[email protected]> Signed-off-by: Wayne Boyer <[email protected]> Change-Id: I68ae4ee80cee40aa4745697ce7acf3883ffb721d Reviewed-on: https://chromium-review.googlesource.com/194250 Reviewed-by: Stéphane Marchesin <[email protected]> Reviewed-by: Aaron Durbin <[email protected]> Tested-by: Wayne Boyer <[email protected]> Commit-Queue: Wayne Boyer <[email protected]> Conflicts: drivers/gpu/drm/i915/intel_dp.c --- drivers/gpu/drm/i915/intel_dp.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 90b24f6..9242499 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -1371,8 +1371,7 @@ void intel_edp_panel_off(struct intel_dp *intel_dp) pp = ironlake_get_pp_control(intel_dp); /* We need to switch off panel power _and_ force vdd, for otherwise some * panels get very unhappy and cease to work. */ - pp &= ~(POWER_TARGET_ON | PANEL_POWER_RESET | EDP_FORCE_VDD | - EDP_BLC_ENABLE); + pp &= ~(POWER_TARGET_ON | EDP_FORCE_VDD | PANEL_POWER_RESET); pp_ctrl_reg = _pp_ctrl_reg(intel_dp); -- 1.7.9.5 _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
