On Fri, Jul 3, 2020 at 4:33 AM Evan Quan <evan.quan@xxxxxxx> wrote: > > Cache all clocks levels for every dpm table. They are needed > by other APIs. > > Change-Id: Idaa853356720e48ab3279f420ba1ae18bb7de4fd > Signed-off-by: Evan Quan <evan.quan@xxxxxxx> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> > --- > .../drm/amd/powerplay/sienna_cichlid_ppt.c | 234 ++++++++++++++++-- > 1 file changed, 211 insertions(+), 23 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/sienna_cichlid_ppt.c b/drivers/gpu/drm/amd/powerplay/sienna_cichlid_ppt.c > index f2bbe56798d7..d750d06378e9 100644 > --- a/drivers/gpu/drm/amd/powerplay/sienna_cichlid_ppt.c > +++ b/drivers/gpu/drm/amd/powerplay/sienna_cichlid_ppt.c > @@ -597,41 +597,229 @@ static int sienna_cichlid_allocate_dpm_context(struct smu_context *smu) > > static int sienna_cichlid_set_default_dpm_table(struct smu_context *smu) > { > - struct smu_dpm_context *smu_dpm = &smu->smu_dpm; > - struct smu_table_context *table_context = &smu->smu_table; > - struct smu_11_0_dpm_context *dpm_context = smu_dpm->dpm_context; > - PPTable_t *driver_ppt = NULL; > + struct smu_11_0_dpm_context *dpm_context = smu->smu_dpm.dpm_context; > + PPTable_t *driver_ppt = smu->smu_table.driver_pptable; > + struct smu_11_0_dpm_table *dpm_table; > + int ret = 0; > int i; > > - driver_ppt = table_context->driver_pptable; > + /* socclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.soc_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_SOCCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_SOCCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_SOCCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.socclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > + > + /* gfxclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.gfx_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_GFXCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_GFXCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_GFXCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.gfxclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > + > + /* uclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.uclk_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_UCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_UCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_UCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.uclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.soc_table.min = driver_ppt->FreqTableSocclk[0]; > - dpm_context->dpm_tables.soc_table.max = driver_ppt->FreqTableSocclk[NUM_SOCCLK_DPM_LEVELS - 1]; > + /* fclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.fclk_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_FCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_FCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_FCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.fclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.gfx_table.min = driver_ppt->FreqTableGfx[0]; > - dpm_context->dpm_tables.gfx_table.max = driver_ppt->FreqTableGfx[NUM_GFXCLK_DPM_LEVELS - 1]; > + /* vclk0 dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.vclk_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_MM_DPM_PG_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_VCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_VCLK_0].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.vclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.uclk_table.min = driver_ppt->FreqTableUclk[0]; > - dpm_context->dpm_tables.uclk_table.max = driver_ppt->FreqTableUclk[NUM_UCLK_DPM_LEVELS - 1]; > + /* vclk1 dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.vclk1_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_MM_DPM_PG_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_VCLK1, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_VCLK_1].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.vclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.vclk_table.min = driver_ppt->FreqTableVclk[0]; > - dpm_context->dpm_tables.vclk_table.max = driver_ppt->FreqTableVclk[NUM_VCLK_DPM_LEVELS - 1]; > + /* dclk0 dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.dclk_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_MM_DPM_PG_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_DCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_DCLK_0].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.dclk_table.min = driver_ppt->FreqTableDclk[0]; > - dpm_context->dpm_tables.dclk_table.max = driver_ppt->FreqTableDclk[NUM_DCLK_DPM_LEVELS - 1]; > + /* dclk1 dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.dclk1_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_MM_DPM_PG_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_DCLK1, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_DCLK_1].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.dcef_table.min = driver_ppt->FreqTableDcefclk[0]; > - dpm_context->dpm_tables.dcef_table.max = driver_ppt->FreqTableDcefclk[NUM_DCEFCLK_DPM_LEVELS - 1]; > + /* dcefclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.dcef_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_DCEFCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_DCEFCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_DCEFCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dcefclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.pixel_table.min = driver_ppt->FreqTablePixclk[0]; > - dpm_context->dpm_tables.pixel_table.max = driver_ppt->FreqTablePixclk[NUM_PIXCLK_DPM_LEVELS - 1]; > + /* pixelclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.pixel_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_DCEFCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_PIXCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_PIXCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dcefclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.display_table.min = driver_ppt->FreqTableDispclk[0]; > - dpm_context->dpm_tables.display_table.max = driver_ppt->FreqTableDispclk[NUM_DISPCLK_DPM_LEVELS - 1]; > + /* displayclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.display_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_DCEFCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_DISPCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_DISPCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dcefclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > - dpm_context->dpm_tables.phy_table.min = driver_ppt->FreqTablePhyclk[0]; > - dpm_context->dpm_tables.phy_table.max = driver_ppt->FreqTablePhyclk[NUM_PHYCLK_DPM_LEVELS - 1]; > + /* phyclk dpm table setup */ > + dpm_table = &dpm_context->dpm_tables.phy_table; > + if (smu_feature_is_enabled(smu, SMU_FEATURE_DPM_DCEFCLK_BIT)) { > + ret = smu_v11_0_set_single_dpm_table(smu, > + SMU_PHYCLK, > + dpm_table); > + if (ret) > + return ret; > + dpm_table->is_fine_grained = > + !driver_ppt->DpmDescriptor[PPCLK_PHYCLK].SnapToDiscrete; > + } else { > + dpm_table->count = 1; > + dpm_table->dpm_levels[0].value = smu->smu_table.boot_values.dcefclk / 100; > + dpm_table->dpm_levels[0].enabled = true; > + dpm_table->min = dpm_table->dpm_levels[0].value; > + dpm_table->max = dpm_table->dpm_levels[0].value; > + } > > + /* lclk dpm table setup */ > for (i = 0; i < MAX_PCIE_CONF; i++) { > dpm_context->dpm_tables.pcie_table.pcie_gen[i] = driver_ppt->PcieGenSpeed[i]; > dpm_context->dpm_tables.pcie_table.pcie_lane[i] = driver_ppt->PcieLaneCount[i]; > -- > 2.27.0 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx