diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-09-06 22:15:42 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-09-20 14:23:03 +0200 |
commit | 0767935e8682157dc98bee03f821faa08b944fe8 (patch) | |
tree | 7eab8aec8fced72170a0c70098e1d266bcac7f3c /firmware/ess | |
parent | 2bd2ad643def79b843e5ff3a88d42d505db158ad (diff) | |
download | blackbird-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