[Intel-gfx] [PATCH 1/7] drm/i915: add for_each_hpd_pin to iterate over hotplug pins

[Jani Nikula]

No functional changes.

Signed-off-by: Jani Nikula <jani.nik...@intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h | 3 +++
 drivers/gpu/drm/i915/i915_irq.c | 8 ++++----
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index 173c9051bf86..60aa9626f91f 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -217,6 +217,9 @@ enum hpd_pin {
        HPD_NUM_PINS
 };
 
+#define for_each_hpd_pin(__pin) \
+       for ((__pin) = (HPD_NONE + 1); (__pin) < HPD_NUM_PINS; (__pin)++)
+
 struct i915_hotplug {
        struct work_struct hotplug_work;
 
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 91cb0b6ec47b..ad8897828b0c 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -1441,7 +1441,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
                         hotplug_trigger, dig_hotplug_reg);
 
        spin_lock(&dev_priv->irq_lock);
-       for (i = 1; i < HPD_NUM_PINS; i++) {
+       for_each_hpd_pin(i) {
                bool long_hpd;
 
                if (!(hpd[i] & hotplug_trigger))
@@ -1477,7 +1477,7 @@ static void intel_hpd_irq_handler(struct drm_device *dev,
                queue_dig = true;
        }
 
-       for (i = 1; i < HPD_NUM_PINS; i++) {
+       for_each_hpd_pin(i) {
                if (hpd[i] & hotplug_trigger &&
                    dev_priv->hotplug.stats[i].state == HPD_DISABLED) {
                        /*
@@ -4292,7 +4292,7 @@ static void intel_hpd_irq_reenable_work(struct 
work_struct *work)
        intel_runtime_pm_get(dev_priv);
 
        spin_lock_irq(&dev_priv->irq_lock);
-       for (i = (HPD_NONE + 1); i < HPD_NUM_PINS; i++) {
+       for_each_hpd_pin(i) {
                struct drm_connector *connector;
 
                if (dev_priv->hotplug.stats[i].state != HPD_DISABLED)
@@ -4450,7 +4450,7 @@ void intel_hpd_init(struct drm_i915_private *dev_priv)
        struct drm_connector *connector;
        int i;
 
-       for (i = 1; i < HPD_NUM_PINS; i++) {
+       for_each_hpd_pin(i) {
                dev_priv->hotplug.stats[i].count = 0;
                dev_priv->hotplug.stats[i].state = HPD_ENABLED;
        }
-- 
2.1.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx