[PATCH] drm/amd/pm: correct MGpuFanBoost setting

[Alex Deucher]

On Tue, May 18, 2021 at 10:13 PM Alex Deucher <alexdeucher at gmail.com> wrote:
>
> Can also add:
> Fixes: https://gitlab.freedesktop.org/drm/amd/-/issues/1580

er,
Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1580

>
> Reviewed-by: Alex Deucher <alexander.deucher at amd.com>
>
> Alex
>
> On Tue, May 18, 2021 at 10:12 PM Alex Deucher <alexdeucher at gmail.com> wrote:
> >
> > Do we need a similar check in the vega powerplay code?
> >
> > Alex
> >
> >
> > On Mon, May 17, 2021 at 10:59 PM Feng, Kenneth <Kenneth.Feng at amd.com> wrote:
> > >
> > > [AMD Official Use Only]
> > >
> > > Reviewed-by: Kenneth Feng <kenneth.feng at amd.com>
> > >
> > > -----Original Message-----
> > > From: Quan, Evan <Evan.Quan at amd.com>
> > > Sent: Tuesday, May 18, 2021 10:05 AM
> > > To: amd-gfx at lists.freedesktop.org
> > > Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; Feng, Kenneth <Kenneth.Feng at amd.com>; Quan, Evan <Evan.Quan at amd.com>
> > > Subject: [PATCH] drm/amd/pm: correct MGpuFanBoost setting
> > >
> > > No MGpuFanBoost setting for those ASICs which do not support it.
> > > Otherwise, it may breaks their fan control feature.
> > >
> > > Change-Id: Ifa9c87ac537a07937a0f0f6a670f21368eb29218
> > > Signed-off-by: Evan Quan <evan.quan at amd.com>
> > > ---
> > >  drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c        |  9 +++++++++
> > >  .../gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c    | 10 ++++++++++
> > >  2 files changed, 19 insertions(+)
> > >
> > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
> > > index d2063b1e7936..f16c76038f13 100644
> > > --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
> > > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c
> > > @@ -2936,6 +2936,8 @@ static ssize_t navi1x_get_gpu_metrics(struct smu_context *smu,
> > >
> > >  static int navi10_enable_mgpu_fan_boost(struct smu_context *smu)  {
> > > +       struct smu_table_context *table_context = &smu->smu_table;
> > > +       PPTable_t *smc_pptable = table_context->driver_pptable;
> > >         struct amdgpu_device *adev = smu->adev;
> > >         uint32_t param = 0;
> > >
> > > @@ -2943,6 +2945,13 @@ static int navi10_enable_mgpu_fan_boost(struct smu_context *smu)
> > >         if (adev->asic_type == CHIP_NAVI12)
> > >                 return 0;
> > >
> > > +       /*
> > > +        * Skip the MGpuFanBoost setting for those ASICs
> > > +        * which do not support it
> > > +        */
> > > +       if (!smc_pptable->MGpuFanBoostLimitRpm)
> > > +               return 0;
> > > +
> > >         /* Workaround for WS SKU */
> > >         if (adev->pdev->device == 0x7312 &&
> > >             adev->pdev->revision == 0)
> > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> > > index 3c3a7f9233e0..159cd698323e 100644
> > > --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> > > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c
> > > @@ -3201,6 +3201,16 @@ static ssize_t sienna_cichlid_get_gpu_metrics(struct smu_context *smu,
> > >
> > >  static int sienna_cichlid_enable_mgpu_fan_boost(struct smu_context *smu)  {
> > > +       struct smu_table_context *table_context = &smu->smu_table;
> > > +       PPTable_t *smc_pptable = table_context->driver_pptable;
> > > +
> > > +       /*
> > > +        * Skip the MGpuFanBoost setting for those ASICs
> > > +        * which do not support it
> > > +        */
> > > +       if (!smc_pptable->MGpuFanBoostLimitRpm)
> > > +               return 0;
> > > +
> > >         return smu_cmn_send_smc_msg_with_param(smu,
> > >                                                SMU_MSG_SetMGpuFanBoostLimitRpm,
> > >                                                0,
> > > --
> > > 2.29.0
> > > _______________________________________________
> > > amd-gfx mailing list
> > > amd-gfx at lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/amd-gfx