[Intel-gfx] [PATCH 02/15] drm/i915: Drop unecessary complexity from psr_inactivate

Rodrigo Vivi rodrigo.vivi at gmail.com
Tue Jun 17 01:58:23 CEST 2014


Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>


On Mon, Jun 16, 2014 at 10:51 AM, Daniel Vetter <daniel.vetter at ffwll.ch>
wrote:

> It's not needed and further more will get in the way of a sane
> locking scheme - psr_exit _can't_ take modeset locks due to lock
> inversion, and at least once dp mst hits the connector list
> is no longer static.
>
> But since we track all state in dev_priv->psr there is no need
> at all.
>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> ---
>  drivers/gpu/drm/i915/intel_dp.c | 24 +++---------------------
>  1 file changed, 3 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_dp.c
> b/drivers/gpu/drm/i915/intel_dp.c
> index 190df701edd5..90f6f0a42d15 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -1910,29 +1910,11 @@ static void intel_edp_psr_work(struct work_struct
> *work)
>  static void intel_edp_psr_inactivate(struct drm_device *dev)
>  {
>         struct drm_i915_private *dev_priv = dev->dev_private;
> -       struct intel_connector *connector;
> -       struct intel_encoder *encoder;
> -       struct intel_crtc *intel_crtc;
> -       struct intel_dp *intel_dp = NULL;
> -
> -       list_for_each_entry(connector, &dev->mode_config.connector_list,
> -                           base.head) {
>
> -               if (connector->base.dpms != DRM_MODE_DPMS_ON)
> -                       continue;
> -
> -               encoder = to_intel_encoder(connector->base.encoder);
> -               if (encoder->type == INTEL_OUTPUT_EDP) {
> +       dev_priv->psr.active = false;
>
> -                       intel_dp = enc_to_intel_dp(&encoder->base);
> -                       intel_crtc = to_intel_crtc(encoder->base.crtc);
> -
> -                       dev_priv->psr.active = false;
> -
> -                       I915_WRITE(EDP_PSR_CTL(dev),
> I915_READ(EDP_PSR_CTL(dev))
> -                                  & ~EDP_PSR_ENABLE);
> -               }
> -       }
> +       I915_WRITE(EDP_PSR_CTL(dev), I915_READ(EDP_PSR_CTL(dev))
> +                  & ~EDP_PSR_ENABLE);
>  }
>
>  void intel_edp_psr_exit(struct drm_device *dev, bool schedule_back)
> --
> 2.0.0
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>



-- 
Rodrigo Vivi
Blog: http://blog.vivi.eng.br
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20140616/e4a9fc1d/attachment.html>


More information about the Intel-gfx mailing list