Re: [PATCH v2 4/9] drm/i915: Fix HDMI 12bpc TRANSCONF bpc value

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Intel-gfx [mailto:intel-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of
> ville.syrjala@xxxxxxxxxxxxxxx
> Sent: Tuesday, May 05, 2015 7:06 AM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Subject:  [PATCH v2 4/9] drm/i915: Fix HDMI 12bpc TRANSCONF bpc
> value
> 
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> 
> IBX BSpec says we must specify 8bpc in TRANSCONF for both 8bpc and 12bpc
> HDMI output. Do so.
> 
> v2: Pass intel_crtc to intel_pipe_has_type()
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/intel_display.c | 10 +++++++---
>  1 file changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_display.c
> b/drivers/gpu/drm/i915/intel_display.c
> index 96ce8f6..a392188 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -2001,11 +2001,15 @@ static void ironlake_enable_pch_transcoder(struct
> drm_i915_private *dev_priv,
> 
>  	if (HAS_PCH_IBX(dev_priv->dev)) {
>  		/*
> -		 * make the BPC in transcoder be consistent with
> -		 * that in pipeconf reg.
> +		 * Make the BPC in transcoder be consistent with
> +		 * that in pipeconf reg. For HDMI we must use 8bpc
> +		 * here for both 8bpc and 12bpc.
>  		 */
>  		val &= ~PIPECONF_BPC_MASK;
> -		val |= pipeconf_val & PIPECONF_BPC_MASK;
> +		if (intel_pipe_has_type(intel_crtc, INTEL_OUTPUT_HDMI))
> +			val |= PIPECONF_8BPC;
> +		else
> +			val |= pipeconf_val & PIPECONF_BPC_MASK;
>  	}
> 
>  	val &= ~TRANS_INTERLACE_MASK;
> --

Reviewed by: Chandra Konduru <Chandra.konduru@xxxxxxxxx>

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





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux