Acked-by: Alex Deucher <alexander.deuc...@amd.com>

On Fri, Aug 13, 2021 at 4:09 AM Evan Quan <evan.q...@amd.com> wrote:
>
> We have a S3 issue on that SKU with BACO enabled. Will bring back this
> when that root caused.
>
> Change-Id: I56d4830e6275e20a415808896eecbadfe944070b
> Signed-off-by: Evan Quan <evan.q...@amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/vi.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
> index fe9a7cc8d9eb..7210f80815b8 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vi.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vi.c
> @@ -904,7 +904,12 @@ static bool vi_asic_supports_baco(struct amdgpu_device 
> *adev)
>         case CHIP_POLARIS11:
>         case CHIP_POLARIS12:
>         case CHIP_TOPAZ:
> -               return amdgpu_dpm_is_baco_supported(adev);
> +               /* Disable BACO support for the specific polaris12 SKU 
> temporarily */
> +               if ((adev->pdev->device == 0x699F) &&
> +                   (adev->pdev->revision == 0xC7))
> +                       return false;
> +               else
> +                       return amdgpu_dpm_is_baco_supported(adev);
>         default:
>                 return false;
>         }
> --
> 2.29.0
>

Reply via email to