[PATCH v2] drm/amd/display: adding ycbcr420 pixel encoding for hdmi

[Harry Wentland]

On 2018-07-04 05:26 PM, mikita.lipski at amd.com wrote:
> From: Mikita Lipski <mikita.lipski at amd.com>
> 
> [why]
> HDMI EDID's VSDB contains spectial timings for specifically
> YCbCr 4:2:0 colour space. In those cases we need to verify
> if the mode provided is one of the special ones has to use
> YCbCr 4:2:0 pixel encoding for display info.
> [how]
> Verify if the mode is using specific ycbcr420 colour space with
> the help of DRM helper function and assign the mode to use
> ycbcr420 pixel encoding.
> 
> Signed-off-by: Mikita Lipski <mikita.lipski at amd.com>

Reviewed-by: Harry Wentland <harry.wentland at amd.com>

Harry

> ---
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 01d14d8..9529043 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -2226,6 +2226,7 @@ fill_stream_properties_from_drm_display_mode(struct dc_stream_state *stream,
>  					     const struct drm_connector *connector)
>  {
>  	struct dc_crtc_timing *timing_out = &stream->timing;
> +	const struct drm_display_info *info = &connector->display_info;
>  
>  	memset(timing_out, 0, sizeof(struct dc_crtc_timing));
>  
> @@ -2234,8 +2235,10 @@ fill_stream_properties_from_drm_display_mode(struct dc_stream_state *stream,
>  	timing_out->v_border_top = 0;
>  	timing_out->v_border_bottom = 0;
>  	/* TODO: un-hardcode */
> -
> -	if ((connector->display_info.color_formats & DRM_COLOR_FORMAT_YCRCB444)
> +	if (drm_mode_is_420_only(info, mode_in)
> +			&& stream->sink->sink_signal == SIGNAL_TYPE_HDMI_TYPE_A)
> +		timing_out->pixel_encoding = PIXEL_ENCODING_YCBCR420;
> +	else if ((connector->display_info.color_formats & DRM_COLOR_FORMAT_YCRCB444)
>  			&& stream->sink->sink_signal == SIGNAL_TYPE_HDMI_TYPE_A)
>  		timing_out->pixel_encoding = PIXEL_ENCODING_YCBCR444;
>  	else
>