Attachment 140450 Details for Bug 106959 – Drop to the maximum supported colour depth on HDMI.

[patch] Drop to the maximum supported colour depth on HDMI.

0001-drm-amdgpu-add-a-check-for-display-depth-validity.patch (text/plain), 2.64 KB, created by Mikita Lipski on 2018-07-03 14:00:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mikita Lipski

Created: 2018-07-03 14:00:21 UTC

Size: 2.64 KB

patch

obsolete

>From 83e0a8a33c9456dcce35d51db0875dd2ac4f6308 Mon Sep 17 00:00:00 2001
>From: Mikita Lipski <mikita.lipski@amd.com>
>Date: Tue, 3 Jul 2018 09:42:51 -0400
>Subject: [PATCH] drm/amdgpu: add a check for display depth validity
>
>[why]
>HDMI 2.0 fails to validate 4K@60 timing with 10 bpc
>[how]
>Adding a helper function that would verify if the display depth
>assigned would pass a bandwidth validation.
>Drop the display depth by one level if TMDS max clock is higer
>
>Signed-off-by: Mikita Lipski <mikita.lipski@amd.com>
>---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 42 +++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
>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..b5268dd 100644
>--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>@@ -2218,6 +2218,46 @@ get_output_color_space(const struct dc_crtc_timing *dc_crtc_timing)
> 	return color_space;
> }
> 
>+static void reduce_mode_colour_depth(struct dc_crtc_timing *timing_out)
>+{
>+	if (timing_out->display_color_depth <= COLOR_DEPTH_888)
>+		return;
>+
>+	timing_out->display_color_depth--;
>+}
>+
>+static void check_if_display_depth_is_supported(struct dc_crtc_timing *timing_out,
>+						const struct drm_display_info *info)
>+{
>+	int normalized_clk;
>+	if (timing_out->display_color_depth <= COLOR_DEPTH_888)
>+		return;
>+	do {
>+		normalized_clk = timing_out->pix_clk_khz;
>+		/* YCbCr 4:2:0 requires additional adjustment of 1/2 */
>+		if (timing_out->pixel_encoding == PIXEL_ENCODING_YCBCR420)
>+			normalized_clk /= 2;
>+		/* Adjusting pix clock following on HDMI spec based on colour depth */
>+		switch (timing_out->display_color_depth) {
>+		case COLOR_DEPTH_101010:
>+			normalized_clk = (normalized_clk * 30) / 24;
>+			break;
>+		case COLOR_DEPTH_121212:
>+			normalized_clk = (normalized_clk * 36) / 24;
>+			break;
>+		case COLOR_DEPTH_161616:
>+			normalized_clk = (normalized_clk * 48) / 24;
>+			break;
>+		default:
>+			return;
>+		}
>+		if (normalized_clk <= info->max_tmds_clock)
>+			return;
>+		reduce_mode_colour_depth(timing_out);
>+
>+	} while (timing_out->display_color_depth > COLOR_DEPTH_888);
>+
>+}
> /*****************************************************************************/
> 
> static void
>@@ -2271,6 +2311,8 @@ fill_stream_properties_from_drm_display_mode(struct dc_stream_state *stream,
> 
> 	stream->out_transfer_func->type = TF_TYPE_PREDEFINED;
> 	stream->out_transfer_func->tf = TRANSFER_FUNCTION_SRGB;
>+	if (stream->sink->sink_signal == SIGNAL_TYPE_HDMI_TYPE_A)
>+		check_if_display_depth_is_supported(timing_out, info);
> }
> 
> static void fill_audio_info(struct audio_info *audio_info,
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 106959: 140217 | 140218 | 140219 | 140220 | 140247 | 140248 | 140256 | 140266 | 140267 | 140449 | 140450 | 140464

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.