Commit c8727233 authored by Jani Nikula's avatar Jani Nikula Committed by Daniel Vetter

drm/i915: simplify condition for digital port

As the hpd loops have been merged together, we don't have to maintain
state for all hpd triggers.
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 9ace0433
...@@ -1432,7 +1432,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev, ...@@ -1432,7 +1432,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
bool storm_detected = false; bool storm_detected = false;
bool queue_dig = false, queue_hp = false; bool queue_dig = false, queue_hp = false;
u32 dig_shift; u32 dig_shift;
u32 dig_port_mask = 0; bool is_dig_port;
if (!hotplug_trigger) if (!hotplug_trigger)
return; return;
...@@ -1446,7 +1446,9 @@ static void intel_hpd_irq_handler(struct drm_device *dev, ...@@ -1446,7 +1446,9 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
continue; continue;
port = get_port_from_pin(i); port = get_port_from_pin(i);
if (port && dev_priv->hotplug.irq_port[port]) { is_dig_port = port && dev_priv->hotplug.irq_port[port];
if (is_dig_port) {
bool long_hpd; bool long_hpd;
if (!HAS_GMCH_DISPLAY(dev_priv)) { if (!HAS_GMCH_DISPLAY(dev_priv)) {
...@@ -1466,8 +1468,6 @@ static void intel_hpd_irq_handler(struct drm_device *dev, ...@@ -1466,8 +1468,6 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
queue_dig = true; queue_dig = true;
if (long_hpd) { if (long_hpd) {
dev_priv->hotplug.long_port_mask |= (1 << port); dev_priv->hotplug.long_port_mask |= (1 << port);
/* FIXME: this can be simplified. */
dig_port_mask |= hpd[i];
} else { } else {
/* for short HPD just trigger the digital queue */ /* for short HPD just trigger the digital queue */
dev_priv->hotplug.short_port_mask |= (1 << port); dev_priv->hotplug.short_port_mask |= (1 << port);
...@@ -1492,7 +1492,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev, ...@@ -1492,7 +1492,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
if (dev_priv->hotplug.stats[i].state != HPD_ENABLED) if (dev_priv->hotplug.stats[i].state != HPD_ENABLED)
continue; continue;
if (!(dig_port_mask & hpd[i])) { if (!is_dig_port) {
dev_priv->hotplug.event_bits |= (1 << i); dev_priv->hotplug.event_bits |= (1 << i);
queue_hp = true; queue_hp = true;
} }
......
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