Attachment 140626 Details for Bug 107066 – Update pix clk for HDMI connector with deep color

[patch] Update pix clk for HDMI connector with deep color

0001-drm-amdgpu-display-update-clk-for-various-HDMI-color.patch (text/plain), 1.78 KB, created by Mikita Lipski on 2018-07-13 15:24:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mikita Lipski

Created: 2018-07-13 15:24:32 UTC

Size: 1.78 KB

patch

obsolete

>From ac1829e82c3389a5c57f7b7a26c8ebf39415520d Mon Sep 17 00:00:00 2001
>From: Mikita Lipski <mikita.lipski@amd.com>
>Date: Fri, 13 Jul 2018 09:07:35 -0400
>Subject: [PATCH] drm/amdgpu/display: update clk for various HDMI color depths
>
>[why]
>When programming tonga's connector's backend we didn't take
>in account that HDMI's colour depth might be more than 8bpc
>therefore we need to add a switch statement that would adjust
>the pixel clock accordingly.
>
>[how]
>Add a switch statement updating clock by its appropriate
>coefficient.
>
>Change-Id: I0d7f1fe6915f8f9c19eb3a39e56a4ae8cdd1b30a
>Signed-off-by: Mikita Lipski <mikita.lipski@amd.com>
>---
> .../gpu/drm/amd/display/dc/bios/command_table.c    | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
>diff --git a/drivers/gpu/drm/amd/display/dc/bios/command_table.c b/drivers/gpu/drm/amd/display/dc/bios/command_table.c
>index 651e1fd..4a78914 100644
>--- a/drivers/gpu/drm/amd/display/dc/bios/command_table.c
>+++ b/drivers/gpu/drm/amd/display/dc/bios/command_table.c
>@@ -808,6 +808,28 @@ static enum bp_result transmitter_control_v1_5(
> 	 * (=1: 8bpp, =1.25: 10bpp, =1.5:12bpp, =2: 16bpp)
> 	 * LVDS mode: usPixelClock = pixel clock
> 	 */
>+	switch (cntl->signal) {
>+	case SIGNAL_TYPE_HDMI_TYPE_A:
>+		switch (cntl->color_depth) {
>+		case COLOR_DEPTH_101010:
>+			params.usSymClock =
>+				cpu_to_le16((le16_to_cpu(params.usSymClock) * 30) / 24);
>+			break;
>+		case COLOR_DEPTH_121212:
>+			params.usSymClock =
>+				cpu_to_le16((le16_to_cpu(params.usSymClock) * 36) / 24);
>+			break;
>+		case COLOR_DEPTH_161616:
>+			params.usSymClock =
>+				cpu_to_le16((le16_to_cpu(params.usSymClock) * 48) / 24);
>+			break;
>+		default:
>+			break;
>+		}
>+		break;
>+		default:
>+			break;
>+	}
> 
> 	if (EXEC_BIOS_CMD_TABLE(UNIPHYTransmitterControl, params))
> 		result = BP_RESULT_OK;
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 107066: 140384 | 140625 | 140626 | 140627 | 140629

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.