On Thu, Nov 26, 2020 at 01:44:36PM +0530, Uma Shankar wrote:
> This patch handles the quantization range for Lspcon.

I would state it as "fixes quantization range for YCbCr output" or
something along those lins.

Reviewed-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

> 
> Signed-off-by: Uma Shankar <uma.shan...@intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_lspcon.c | 17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_lspcon.c 
> b/drivers/gpu/drm/i915/display/intel_lspcon.c
> index f98891f058da..7cb65e0f241e 100644
> --- a/drivers/gpu/drm/i915/display/intel_lspcon.c
> +++ b/drivers/gpu/drm/i915/display/intel_lspcon.c
> @@ -523,12 +523,17 @@ void lspcon_set_infoframes(struct intel_encoder 
> *encoder,
>       else
>               frame.avi.colorspace = HDMI_COLORSPACE_RGB;
>  
> -     drm_hdmi_avi_infoframe_quant_range(&frame.avi,
> -                                        conn_state->connector,
> -                                        adjusted_mode,
> -                                        crtc_state->limited_color_range ?
> -                                        HDMI_QUANTIZATION_RANGE_LIMITED :
> -                                        HDMI_QUANTIZATION_RANGE_FULL);
> +     if (crtc_state->output_format == INTEL_OUTPUT_FORMAT_RGB) {
> +             drm_hdmi_avi_infoframe_quant_range(&frame.avi,
> +                                                conn_state->connector,
> +                                                adjusted_mode,
> +                                                
> crtc_state->limited_color_range ?
> +                                                
> HDMI_QUANTIZATION_RANGE_LIMITED :
> +                                                
> HDMI_QUANTIZATION_RANGE_FULL);
> +     } else {
> +             frame.avi.quantization_range = HDMI_QUANTIZATION_RANGE_DEFAULT;
> +             frame.avi.ycc_quantization_range = 
> HDMI_YCC_QUANTIZATION_RANGE_LIMITED;
> +     }
>  
>       ret = hdmi_infoframe_pack(&frame, buf, sizeof(buf));
>       if (ret < 0) {
> -- 
> 2.26.2

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to