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>
Fri, 14 Jun 2024 20:18:55 +0000 (16:18 -0400)
There is a coding error which causes incorrect variables to be assigned
in DML21 phase 5.

Reviewed-by: Dillon Varone <dillon.varone@amd.com>
Acked-by: Hamza Mahfooz <hamza.mahfooz@amd.com>
Signed-off-by: Wenjing Liu <wenjing.liu@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 1142fdade33436e09164613d9ae63537c699a612..6f334fdc6eb86b08b61472612d7921d2fb6a7e09 100644 (file)
@@ -259,7 +259,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;
@@ -272,7 +272,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;
        }
 
        /*