[Intel-gfx] [PATCH 49/58] drm/i915: use staged outuput config in lvds->mode_fixup

Jesse Barnes jbarnes at virtuousgeek.org
Wed Sep 5 20:02:47 CEST 2012


On Sun, 19 Aug 2012 21:13:06 +0200
Daniel Vetter <daniel.vetter at ffwll.ch> wrote:

> - Use the check_cloned helper from the previous patch.
> - Use encoder->new_crtc to check crtc properties.
> 
> Signed-Off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> ---
>  drivers/gpu/drm/i915/intel_lvds.c | 13 +++----------
>  1 file changed, 3 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c
> index 52f9e7f..7f730af 100644
> --- a/drivers/gpu/drm/i915/intel_lvds.c
> +++ b/drivers/gpu/drm/i915/intel_lvds.c
> @@ -250,9 +250,8 @@ static bool intel_lvds_mode_fixup(struct drm_encoder *encoder,
>  {
>  	struct drm_device *dev = encoder->dev;
>  	struct drm_i915_private *dev_priv = dev->dev_private;
> -	struct intel_crtc *intel_crtc = to_intel_crtc(encoder->crtc);
>  	struct intel_lvds *intel_lvds = to_intel_lvds(encoder);
> -	struct intel_encoder *tmp_encoder;
> +	struct intel_crtc *intel_crtc = intel_lvds->base.new_crtc;
>  	u32 pfit_control = 0, pfit_pgm_ratios = 0, border = 0;
>  	int pipe;
>  
> @@ -262,14 +261,8 @@ static bool intel_lvds_mode_fixup(struct drm_encoder *encoder,
>  		return false;
>  	}
>  
> -	/* Should never happen!! */
> -	for_each_encoder_on_crtc(dev, encoder->crtc, tmp_encoder) {
> -		if (&tmp_encoder->base != encoder) {
> -			DRM_ERROR("Can't enable LVDS and another "
> -			       "encoder on the same pipe\n");
> -			return false;
> -		}
> -	}
> +	if (!intel_encoder_check_non_cloned(&intel_lvds->base))
> +		return false;
>  
>  	/*
>  	 * We have timings from the BIOS for the panel, put them in

Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org>

-- 
Jesse Barnes, Intel Open Source Technology Center



More information about the Intel-gfx mailing list