drm/amd/pp: Fix overflow when setup decf/pix/disp dpm table.

Clear the count in the single table setup function to
avoid missing any tables.

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Rex Zhu <Rex.Zhu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
index 500f177254ac2540688365ff75f8ff86d6accd1a..0519338e0e5e1d31a9c543863f470be51d0b1872 100644 (file)
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c
@@ -1161,6 +1161,8 @@ static void vega10_setup_default_single_dpm_table(struct pp_hwmgr *hwmgr,
 {
        int i;
 
+       dpm_table->count = 0;
+
        for (i = 0; i < dep_table->count; i++) {
                if (i == 0 || dpm_table->dpm_levels[dpm_table->count - 1].value <=
                                dep_table->entries[i].clk) {
@@ -1269,10 +1271,6 @@ static int vega10_setup_default_dpm_tables(struct pp_hwmgr *hwmgr)
                        return -EINVAL);
 
        /* Initialize Sclk DPM table based on allow Sclk values */
-       data->dpm_table.soc_table.count = 0;
-       data->dpm_table.gfx_table.count = 0;
-       data->dpm_table.dcef_table.count = 0;
-
        dpm_table = &(data->dpm_table.soc_table);
        vega10_setup_default_single_dpm_table(hwmgr,
                        dpm_table,