[PATCH] drm/amdgpu/powerplay: using the FCLK DPM table to set the MCLK for DPM states consist of three entities :FCLK, UCLK, MEMCLK all these three clk change together , MEMCLK from FCLK.

[Huang Rui]

On Tue, Mar 31, 2020 at 09:41:44AM -0400, Alex Deucher wrote:
> On Tue, Mar 31, 2020 at 6:10 AM Yuxian Dai <Yuxian.Dai at amd.com> wrote:
> >
> > From: "yuxiadai at amd.com" <yuxiadai at amd.com>
> 
> Your patch title is too long; it is basically the whole patch
> description rather than just a title.  Please split it up between the
> title and descriptions.  E.g.,
> 
> drm/amdgpu/powerplay: fix MCLK DPM handling for renoir
> 
> Using the FCLK DPM table to set the MCLK for DPM states consist of
> three entities:
> FCLK
> UCLK
> MEMCLK
> All these three clk change together, MEMCLK from FCLK, so use the fclk
> frequency.
> 
> With that fixed, patch is:
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
> 

Yes, and usually, we don't leave the commit message as empty. David, you need
describe the detailed info in the commit not the title.

Thanks,
Ray

> >
> > Change-Id: Ia45f3069fc7ae56db495cb5a3865e2c50c550774
> > Signed-off-by: Yuxian Dai <Yuxian.Dai at amd.com>
> > ---
> >  drivers/gpu/drm/amd/powerplay/renoir_ppt.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.h b/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
> > index 2a390ddd37dd..89cd6da118a3 100644
> > --- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
> > +++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
> > @@ -37,7 +37,7 @@ extern void renoir_set_ppt_funcs(struct smu_context *smu);
> >                         freq = table->SocClocks[dpm_level].Freq;        \
> >                         break;                                          \
> >                 case SMU_MCLK:                                          \
> > -                       freq = table->MemClocks[dpm_level].Freq;        \
> > +                       freq = table->FClocks[dpm_level].Freq;  \
> >                         break;                                          \
> >                 case SMU_DCEFCLK:                                       \
> >                         freq = table->DcfClocks[dpm_level].Freq;        \
> > --
> > 2.17.1
> >
> > _______________________________________________
> > amd-gfx mailing list
> > amd-gfx at lists.freedesktop.org
> > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Cray.huang%40amd.com%7C541f2e15da804d81c5b508d7d57949d2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637212590166168794&sdata=7x4pBWEPuVHbb8ro%2Fpy5%2BgiJhrxWbLrxGqHSro9KUV8%3D&reserved=0
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Cray.huang%40amd.com%7C541f2e15da804d81c5b508d7d57949d2%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637212590166168794&sdata=7x4pBWEPuVHbb8ro%2Fpy5%2BgiJhrxWbLrxGqHSro9KUV8%3D&reserved=0