Commit 5d9e7fe8 authored by Nikola Cornij's avatar Nikola Cornij Committed by Alex Deucher
Browse files

drm/amd/display: Clamp VStartup value at DML calculations time 



[why]
Some timings with a large VBlank cause the value to overflow the
register related, while also producing other wrong values in DML output.

[how]
Clamp VStartup at the DCN3.1 maximum value

Signed-off-by: default avatarNikola Cornij <nikola.cornij@amd.com>
Reviewed-by: default avatarDmytro Laktyushkin <Dmytro.Laktyushkin@amd.com>
Acked-by: default avatarBindu Ramamurthy <bindu.r@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent d9b20b45

drivers/gpu/drm/amd/display/dc/dml/dcn31/display_mode_vba_31.c

+4 −0
Original line number Diff line number Diff line
@@ -2668,6 +2668,8 @@ static void DISPCLKDPPCLKDCFCLKDeepSleepPrefetchParametersWatermarksAndPerforman 
												(double) v->WritebackDelay[v->VoltageLevel][k]

														/ (v->HTotal[k] / v->PixelClock[k]),

												1));

		if (v->MaxVStartupLines[k] > 1023)

			v->MaxVStartupLines[k] = 1023;



#ifdef __DML_VBA_DEBUG__

		dml_print("DML::%s: k=%d MaxVStartupLines = %d\n", __func__, k, v->MaxVStartupLines[k]);
@@ -5064,6 +5066,8 @@ void dml31_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l 
																/ (v->HTotal[k]

																		/ v->PixelClock[k]),

														1.0));

				if (v->MaximumVStartup[i][j][k] > 1023)

					v->MaximumVStartup[i][j][k] = 1023;

				v->MaxMaxVStartup[i][j] = dml_max(v->MaxMaxVStartup[i][j], v->MaximumVStartup[i][j][k]);

			}

		}