[PATCH] drm/amd/powerplay: correct Arcturus OD support

[Alex Deucher]

On Thu, Nov 7, 2019 at 2:38 AM Quan, Evan <Evan.Quan at amd.com> wrote:
>
> OD is not supported on Arcturus. Thus the
> pp_od_clk_voltage sysfs interface is also not supported.
>
> Change-Id: Ib70632a55a0980cf04c3432d43dbcf869cd1b4bf
> Signed-off-by: Evan Quan <evan.quan at amd.com>

You might want to check with Chris and Kent about this.  I think there
is a use case for OD on ROCm in some cases.  Assuming they are ok with
it,
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

Alex

> ---
>  drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c
> index c21fe7ac5df8..76a4154b3be2 100644
> --- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c
> +++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c
> @@ -714,6 +714,9 @@ static int smu_set_funcs(struct amdgpu_device *adev)
>  {
>         struct smu_context *smu = &adev->smu;
>
> +       if (adev->pm.pp_feature & PP_OVERDRIVE_MASK)
> +               smu->od_enabled = true;
> +
>         switch (adev->asic_type) {
>         case CHIP_VEGA20:
>                 vega20_set_ppt_funcs(smu);
> @@ -725,6 +728,8 @@ static int smu_set_funcs(struct amdgpu_device *adev)
>                 break;
>         case CHIP_ARCTURUS:
>                 arcturus_set_ppt_funcs(smu);
> +               /* OD is not supported on Arcturus */
> +               smu->od_enabled =false;
>                 break;
>         case CHIP_RENOIR:
>                 renoir_set_ppt_funcs(smu);
> @@ -733,9 +738,6 @@ static int smu_set_funcs(struct amdgpu_device *adev)
>                 return -EINVAL;
>         }
>
> -       if (adev->pm.pp_feature & PP_OVERDRIVE_MASK)
> -               smu->od_enabled = true;
> -
>         return 0;
>  }
>
> --
> 2.23.0
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx