提交 641a969e 编写于 作者: J Jani Nikula 提交者: Daniel Vetter

drm/i915: simplify conditions for skipping the 2nd hpd loop iterations

Multiple positive and negative checks for hpd[i] & hotplug_trigger gets
hard to read. Simplify. This should make follow-up patches merging the
two loops easier. No functional changes.
Signed-off-by: NJani Nikula <jani.nikula@intel.com>
Reviewed-by: NPaulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: NDaniel Vetter <daniel.vetter@ffwll.ch>
上级 c91711f9
......@@ -1478,8 +1478,10 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
}
for_each_hpd_pin(i) {
if (hpd[i] & hotplug_trigger &&
dev_priv->hotplug.stats[i].state == HPD_DISABLED) {
if (!(hpd[i] & hotplug_trigger))
continue;
if (dev_priv->hotplug.stats[i].state == HPD_DISABLED) {
/*
* On GMCH platforms the interrupt mask bits only
* prevent irq generation, not the setting of the
......@@ -1493,8 +1495,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
continue;
}
if (!(hpd[i] & hotplug_trigger) ||
dev_priv->hotplug.stats[i].state != HPD_ENABLED)
if (dev_priv->hotplug.stats[i].state != HPD_ENABLED)
continue;
if (!(dig_port_mask & hpd[i])) {
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册