drm/amd/display: fix minor coding errors where dml21 phase 5 uses wrong variables
authorWenjing Liu <wenjing.liu@amd.com>
Mon, 27 May 2024 14:30:45 +0000 (10:30 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 13 Aug 2024 14:31:40 +0000 (10:31 -0400)
[why & how]
There is a coding error which causes incorrect variables to be assigned
in DML21 phase 5.

Reviewed-by: Austin Zheng <austin.zheng@amd.com>
Signed-off-by: Wenjing Liu <wenjing.liu@amd.com>
Signed-off-by: Tom Chung <chiahsuan.chung@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_top/dml_top.c

index e9b40a4..a9c3ed6 100644 (file)
@@ -274,7 +274,7 @@ bool dml2_build_mode_programming(struct dml2_build_mode_programming_in_out *in_o
        /*
        * Phase 5: Optimize for Stutter
        */
-       memset(&l->vmin_phase, 0, sizeof(struct optimization_phase_params));
+       memset(&l->stutter_phase, 0, sizeof(struct optimization_phase_params));
        l->stutter_phase.dml = dml;
        l->stutter_phase.display_config = &l->base_display_config_with_meta;
        l->stutter_phase.init_function = dml2_top_optimization_init_function_stutter;
@@ -287,7 +287,7 @@ bool dml2_build_mode_programming(struct dml2_build_mode_programming_in_out *in_o
 
        if (stutter_success) {
                memcpy(&l->base_display_config_with_meta, &l->optimized_display_config_with_meta, sizeof(struct display_configuation_with_meta));
-               l->base_display_config_with_meta.stage4.success = true;
+               l->base_display_config_with_meta.stage5.success = true;
        }
 
        /*