summaryrefslogtreecommitdiffstats
path: root/firmware/ess
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2012-09-06 22:15:42 +0200
committerDaniel Vetter <daniel.vetter@ffwll.ch>2012-09-20 14:23:03 +0200
commit0767935e8682157dc98bee03f821faa08b944fe8 (patch)
tree7eab8aec8fced72170a0c70098e1d266bcac7f3c /firmware/ess
parent2bd2ad643def79b843e5ff3a88d42d505db158ad (diff)
downloadblackbird-op-linux-0767935e8682157dc98bee03f821faa08b944fe8.tar.gz
blackbird-op-linux-0767935e8682157dc98bee03f821faa08b944fe8.zip
drm/i915: robustify edp_pll_on/off
With the previous patch to clean up where exactly these two functions are getting called, this patch can tackle the enable/disable code itself: - WARN if the port enable bit is in the wrong state or if the edp pll bit is in the wrong state, just for paranoia's sake. - Don't disable the edp pll harder in the modeset functions just for fun. - Don't set the edp pll enable flag in intel_dp->DP in modeset, do that while changing the actual hw state. We do the same with the actual port enable bit, so this is a bit more consistent. - Track the current DP register value when setting things up and add some comments how intel_dp->DP is used in the disable code. v2: Be more careful with resetting intel_dp->DP - otherwise dpms off->on will fail spectacularly, becuase we enable the eDP port when we should only enable the eDP pll. Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'firmware/ess')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud