On Wed, Oct 18, 2017 at 5:45 AM, Rex Zhu <[email protected]> wrote: > Change-Id: I9e72043ea619a86b2bcf14ce9d6386fee107bd19 > Signed-off-by: Rex Zhu <[email protected]>
Please include a better patch description. E.g., Clear the count in the single table setup function to avoid missing any tables. or something like that. With that fixed: Reviewed-by: Alex Deucher <[email protected]> > --- > drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c > index 500f177..0519338 100644 > --- 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, > -- > 1.9.1 > > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
