On Wed, Jun 05, 2013 at 01:34:17PM +0200, Daniel Vetter wrote:
> Simply grew too big. This also makes the fixup and restore logic in
> setup_hw_state stand out a bit more clearly.
> 
> Signed-off-by: Daniel Vetter <daniel.vet...@ffwll.ch>

Reviewed-by: Damien Lespiau <damien.lesp...@intel.com>

-- 
Damien

> ---
>  drivers/gpu/drm/i915/intel_display.c | 20 +++++++++++++++-----
>  1 file changed, 15 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_display.c 
> b/drivers/gpu/drm/i915/intel_display.c
> index 998ba5c..95ed27b 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -9630,14 +9630,10 @@ void i915_redisable_vga(struct drm_device *dev)
>       }
>  }
>  
> -/* Scan out the current hw modeset state, sanitizes it and maps it into the 
> drm
> - * and i915 state tracking structures. */
> -void intel_modeset_setup_hw_state(struct drm_device *dev,
> -                               bool force_restore)
> +static void intel_modeset_readout_hw_state(struct drm_device *dev)
>  {
>       struct drm_i915_private *dev_priv = dev->dev_private;
>       enum pipe pipe;
> -     struct drm_plane *plane;
>       struct intel_crtc *crtc;
>       struct intel_encoder *encoder;
>       struct intel_connector *connector;
> @@ -9713,6 +9709,20 @@ void intel_modeset_setup_hw_state(struct drm_device 
> *dev,
>                             drm_get_connector_name(&connector->base),
>                             connector->base.encoder ? "enabled" : "disabled");
>       }
> +}
> +
> +/* Scan out the current hw modeset state, sanitizes it and maps it into the 
> drm
> + * and i915 state tracking structures. */
> +void intel_modeset_setup_hw_state(struct drm_device *dev,
> +                               bool force_restore)
> +{
> +     struct drm_i915_private *dev_priv = dev->dev_private;
> +     enum pipe pipe;
> +     struct drm_plane *plane;
> +     struct intel_crtc *crtc;
> +     struct intel_encoder *encoder;
> +
> +     intel_modeset_readout_hw_state(dev);
>  
>       /* HW state is read out, now we need to sanitize this mess. */
>       list_for_each_entry(encoder, &dev->mode_config.encoder_list,
> -- 
> 1.7.11.7
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to