Commit 385c3e4c authored by Paul Hsieh's avatar Paul Hsieh Committed by Alex Deucher

drm/amd/display: Correct DML calculation to follow HW SPEC

[Why]
In 2560x1600@240p eDP panel, driver use lowest voltage level
to play 1080p video cause underflow. According to HW SPEC,
the senario should use high voltage level.

[How]
ChromaPre value is zero when bandwidth validation.
Correct ChromaPre calculation.
Reviewed-by: default avatarNicholas Kazlauskas <Nicholas.Kazlauskas@amd.com>
Reviewed-by: default avatarJun Lei <Jun.Lei@amd.com>
Acked-by: default avatarQingqing Zhuo <qingqing.zhuo@amd.com>
Signed-off-by: default avatarPaul Hsieh <Paul.Hsieh@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6f6869dc
...@@ -4864,7 +4864,7 @@ void dml30_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l ...@@ -4864,7 +4864,7 @@ void dml30_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l
v->DETBufferSizeCThisState[k], v->DETBufferSizeCThisState[k],
&v->UrgentBurstFactorCursorPre[k], &v->UrgentBurstFactorCursorPre[k],
&v->UrgentBurstFactorLumaPre[k], &v->UrgentBurstFactorLumaPre[k],
&v->UrgentBurstFactorChroma[k], &v->UrgentBurstFactorChromaPre[k],
&v->NoUrgentLatencyHidingPre[k]); &v->NoUrgentLatencyHidingPre[k]);
} }
......
...@@ -5191,7 +5191,7 @@ void dml31_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l ...@@ -5191,7 +5191,7 @@ void dml31_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l
v->DETBufferSizeCThisState[k], v->DETBufferSizeCThisState[k],
&v->UrgentBurstFactorCursorPre[k], &v->UrgentBurstFactorCursorPre[k],
&v->UrgentBurstFactorLumaPre[k], &v->UrgentBurstFactorLumaPre[k],
&v->UrgentBurstFactorChroma[k], &v->UrgentBurstFactorChromaPre[k],
&v->NotUrgentLatencyHidingPre[k]); &v->NotUrgentLatencyHidingPre[k]);
} }
......
...@@ -5288,7 +5288,7 @@ void dml314_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_ ...@@ -5288,7 +5288,7 @@ void dml314_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_
v->DETBufferSizeCThisState[k], v->DETBufferSizeCThisState[k],
&v->UrgentBurstFactorCursorPre[k], &v->UrgentBurstFactorCursorPre[k],
&v->UrgentBurstFactorLumaPre[k], &v->UrgentBurstFactorLumaPre[k],
&v->UrgentBurstFactorChroma[k], &v->UrgentBurstFactorChromaPre[k],
&v->NotUrgentLatencyHidingPre[k]); &v->NotUrgentLatencyHidingPre[k]);
} }
......
...@@ -3354,7 +3354,7 @@ void dml32_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l ...@@ -3354,7 +3354,7 @@ void dml32_ModeSupportAndSystemConfigurationFull(struct display_mode_lib *mode_l
/* Output */ /* Output */
&mode_lib->vba.UrgentBurstFactorCursorPre[k], &mode_lib->vba.UrgentBurstFactorCursorPre[k],
&mode_lib->vba.UrgentBurstFactorLumaPre[k], &mode_lib->vba.UrgentBurstFactorLumaPre[k],
&mode_lib->vba.UrgentBurstFactorChroma[k], &mode_lib->vba.UrgentBurstFactorChromaPre[k],
&mode_lib->vba.NotUrgentLatencyHidingPre[k]); &mode_lib->vba.NotUrgentLatencyHidingPre[k]);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment