drm/i915: Don't enable backlight at setup time.
authorDhinakaran Pandiyan <[email protected]>
Mon, 19 Jun 2017 18:08:28 +0000 (11:08 -0700)
committerJani Nikula <[email protected]>
Mon, 19 Jun 2017 19:15:15 +0000 (22:15 +0300)
commita8ae0a773d38b4b1d4566b0edcb6bb63f4a9d22f
tree44d8f2401b8719f0e525d0b5d3590ae3066115be
parentb7f5dd36e0c5cb9ca1070a5e0f22f666bcff07ef
drm/i915: Don't enable backlight at setup time.

Maarten and Ville noticed that we are enabling backlight via DP aux very
early in the modeset_init path via the intel_dp_aux_setup_backlight()
function, since commit e7156c833903 ("drm/i915: Add Backlight Control using
DPCD for eDP connectors (v9)"). Looks like all we need to do during
_setup_backlight() is read the current brightness state instead of
modifying it.

v2: Rewrote commit message.

Cc: Ville Syrjala <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Cc: Jani Nikula <[email protected]>
Cc: Yetunde Adebisi <[email protected]>
Signed-off-by: Dhinakaran Pandiyan <[email protected]>
Reviewed-by: Maarten Lankhorst <[email protected]>
Acked-by: Jani Nikula <[email protected]>
Tested-by: Puthikorn Voravootivat <[email protected]>
Fixes: e7156c833903 ("drm/i915: Add Backlight Control using DPCD for eDP connectors (v9)")
Link: http://patchwork.freedesktop.org/patch/msgid/[email protected]
Signed-off-by: Ville Syrjälä <[email protected]>
(cherry picked from commit f6262bda462e81e959b80a96dac799bd9df27f73)
Signed-off-by: Jani Nikula <[email protected]>
Link: http://patchwork.freedesktop.org/patch/msgid/[email protected]
drivers/gpu/drm/i915/intel_dp_aux_backlight.c