]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
drm/amd/display: update clk for various HDMI color depths
authorMikita Lipski <mikita.lipski@amd.com>
Fri, 13 Jul 2018 13:07:35 +0000 (09:07 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 6 Aug 2018 20:57:29 +0000 (15:57 -0500)
[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.

Signed-off-by: Mikita Lipski <mikita.lipski@amd.com>
Reviewed-by: Charlene Liu <Charlene.Liu@amd.com>
Acked-by: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
drivers/gpu/drm/amd/display/dc/bios/command_table.c

index 651e1fd4622f9b4e4d832e219d683120b0bc2349..a558bfaa0c468728c4a1441cb56b390b2960cd28 100644 (file)
@@ -808,6 +808,24 @@ static enum bp_result transmitter_control_v1_5(
         * (=1: 8bpp, =1.25: 10bpp, =1.5:12bpp, =2: 16bpp)
         * LVDS mode: usPixelClock = pixel clock
         */
+       if  (cntl->signal == 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;
+               }
+       }
 
        if (EXEC_BIOS_CMD_TABLE(UNIPHYTransmitterControl, params))
                result = BP_RESULT_OK;