Attachment 140613 Details for Bug 107153 – Patch 1/2

[patch] Patch 1/2

0001-drm-amd-display-adding-ycbcr420-pixel-encoding-for-h.patch (text/plain), 2.07 KB, created by Leo Li on 2018-07-12 21:31:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Leo Li

Created: 2018-07-12 21:31:56 UTC

Size: 2.07 KB

patch

obsolete

>From d585209b23040906994e5de269ca3e360daef326 Mon Sep 17 00:00:00 2001
>From: Mikita Lipski <mikita.lipski@amd.com>
>Date: Wed, 4 Jul 2018 09:27:02 -0400
>Subject: [PATCH 1/2] drm/amd/display: adding ycbcr420 pixel encoding for hdmi
>
>[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@amd.com>
>Tested-by: Mike Lothian <mike@fireburn.co.uk>
>Reviewed-by: Harry Wentland <harry.wentland@amd.com>
>---
> 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
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 107153: 140613 | 140614 | 140677 | 140678 | 140681 | 140685 | 140711

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.