[Intel-gfx] [PATCH 03/11] drm/i915: Factor out ilk_update_display_irq()

[Paulo Zanoni]

2015-08-12 12:44 GMT-03:00  <ville.syrjala at linux.intel.com>:
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
>
> Extract the core of ironlake_{enable,disable}_display_irq() into a new
> function. We'll have further use for it later.
>
> Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> ---
>  drivers/gpu/drm/i915/i915_irq.c | 39 ++++++++++++++++++++++++---------------
>  1 file changed, 24 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
> index de0edbd..8a1e35e 100644
> --- a/drivers/gpu/drm/i915/i915_irq.c
> +++ b/drivers/gpu/drm/i915/i915_irq.c
> @@ -154,35 +154,44 @@ static const u32 hpd_bxt[HPD_NUM_PINS] = {
>
>  static void gen6_rps_irq_handler(struct drm_i915_private *dev_priv, u32 pm_iir);
>
> -/* For display hotplug interrupt */
> -void
> -ironlake_enable_display_irq(struct drm_i915_private *dev_priv, u32 mask)
> +/**
> + * ilk_update_display_irq - update DEIMR
> + * @dev_priv: driver private
> + * @interrupt_mask: mask of interrupt bits to update
> + * @enabled_irq_mask: mask of interrupt bits to enable
> + */
> +static void ilk_update_display_irq(struct drm_i915_private *dev_priv,
> +                                  uint32_t interrupt_mask,
> +                                  uint32_t enabled_irq_mask)
>  {
> +       uint32_t new_val;
> +
>         assert_spin_locked(&dev_priv->irq_lock);
>

WARN_ON(enabled_irq_mask & ~interrupt_mask);

With that: Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com>

>         if (WARN_ON(!intel_irqs_enabled(dev_priv)))
>                 return;
>
> -       if ((dev_priv->irq_mask & mask) != 0) {
> -               dev_priv->irq_mask &= ~mask;
> +       new_val = dev_priv->irq_mask;
> +       new_val &= ~interrupt_mask;
> +       new_val |= (~enabled_irq_mask & interrupt_mask);
> +
> +       if (new_val != dev_priv->irq_mask) {
> +               dev_priv->irq_mask = new_val;
>                 I915_WRITE(DEIMR, dev_priv->irq_mask);
>                 POSTING_READ(DEIMR);
>         }
>  }
>
>  void
> -ironlake_disable_display_irq(struct drm_i915_private *dev_priv, u32 mask)
> +ironlake_enable_display_irq(struct drm_i915_private *dev_priv, u32 mask)
>  {
> -       assert_spin_locked(&dev_priv->irq_lock);
> -
> -       if (WARN_ON(!intel_irqs_enabled(dev_priv)))
> -               return;
> +       ilk_update_display_irq(dev_priv, mask, mask);
> +}
>
> -       if ((dev_priv->irq_mask & mask) != mask) {
> -               dev_priv->irq_mask |= mask;
> -               I915_WRITE(DEIMR, dev_priv->irq_mask);
> -               POSTING_READ(DEIMR);
> -       }
> +void
> +ironlake_disable_display_irq(struct drm_i915_private *dev_priv, u32 mask)
> +{
> +       ilk_update_display_irq(dev_priv, mask, 0);
>  }
>
>  /**
> --
> 2.4.6
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Paulo Zanoni