Ping? On Tue, Nov 19, 2019 at 3:56 PM Alex Deucher <alexdeucher@xxxxxxxxx> wrote: > > dm_pp_get_clock_levels_by_type needs to add the default clocks > to the powerplay case as well. This was accidently dropped. > > Fixes: b3ea88fef321de ("drm/amd/powerplay: add get_clock_by_type interface for display") > Bug: https://gitlab.freedesktop.org/drm/amd/issues/906 > Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> > --- > drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c > index 229788bee544..a2e1a73f66b8 100644 > --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c > +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c > @@ -342,7 +342,8 @@ bool dm_pp_get_clock_levels_by_type( > if (adev->powerplay.pp_funcs && adev->powerplay.pp_funcs->get_clock_by_type) { > if (adev->powerplay.pp_funcs->get_clock_by_type(pp_handle, > dc_to_pp_clock_type(clk_type), &pp_clks)) { > - /* Error in pplib. Provide default values. */ > + /* Error in pplib. Provide default values. */ > + get_default_clock_levels(clk_type, dc_clks); > return true; > } > } else if (adev->smu.ppt_funcs && adev->smu.ppt_funcs->get_clock_by_type) { > -- > 2.23.0 > _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx