From: Jake Wang <haonan.wa...@amd.com>

[Why]
During DCC on/off, stutter period is calculated before DCC has fully 
transitioned.
This results in incorrect stutter period calculation.

[How]
Trigger a full update when DCC changes between on/off.

Signed-off-by: Jake Wang <haonan.wa...@amd.com>
Reviewed-by: Aric Cyr <aric....@amd.com>
Acked-by: Stylon Wang <stylon.w...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 34c207f92df9..156ba8d3d4dc 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -1918,8 +1918,13 @@ static enum surface_update_type 
get_plane_info_update_type(const struct dc_surfa
        if (u->plane_info->dcc.enable != u->surface->dcc.enable
                        || u->plane_info->dcc.independent_64b_blks != 
u->surface->dcc.independent_64b_blks
                        || u->plane_info->dcc.meta_pitch != 
u->surface->dcc.meta_pitch) {
+               /* During DCC on/off, stutter period is calculated before
+                * DCC has fully transitioned. This results in incorrect
+                * stutter period calculation. Triggering a full update will
+                * recalculate stutter period.
+                */
                update_flags->bits.dcc_change = 1;
-               elevate_update_type(&update_type, UPDATE_TYPE_MED);
+               elevate_update_type(&update_type, UPDATE_TYPE_FULL);
        }
 
        if (resource_pixel_format_to_bpp(u->plane_info->format) !=
-- 
2.25.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to