Commit 1e6210f4 authored by Paulo Zanoni's avatar Paulo Zanoni Committed by Daniel Vetter

drm/i915: don't rely on previous values set on DDI_BUF_CTL

Just set the only bit we need, everything else is either ignored on
HDMI or should be set to zero.
Signed-off-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 6441ab5f
...@@ -1017,16 +1017,12 @@ void intel_enable_ddi(struct intel_encoder *encoder) ...@@ -1017,16 +1017,12 @@ void intel_enable_ddi(struct intel_encoder *encoder)
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base);
int port = intel_hdmi->ddi_port; int port = intel_hdmi->ddi_port;
u32 temp;
temp = I915_READ(DDI_BUF_CTL(port));
temp |= DDI_BUF_CTL_ENABLE;
/* Enable DDI_BUF_CTL. In HDMI/DVI mode, the port width, /* Enable DDI_BUF_CTL. In HDMI/DVI mode, the port width,
* and swing/emphasis values are ignored so nothing special needs * and swing/emphasis values are ignored so nothing special needs
* to be done besides enabling the port. * to be done besides enabling the port.
*/ */
I915_WRITE(DDI_BUF_CTL(port), temp); I915_WRITE(DDI_BUF_CTL(port), DDI_BUF_CTL_ENABLE);
} }
void intel_disable_ddi(struct intel_encoder *encoder) void intel_disable_ddi(struct intel_encoder *encoder)
......
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