ACK/cmnt: [PATCH 1/1][SRU][U/H/OEM-5.10/OEM-5.13] UBUNTU: SAUCE: drm/amdgpu: disable BACO support for 699F:C7 polaris12 SKU temporarily

[Tim Gardner]

Acked-by: Tim Gardner <tim.gardner at canonical.com>

For how long will this SKU exist ? We'll end up carrying this patch 
forward indefinitely.

On 8/16/21 9:33 PM, Koba Ko wrote:
> From: Evan Quan <evan.quan at amd.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1940204
> 
> 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.quan at amd.com>
> Acked-by: Alex Deucher <alexander.deucher at amd.com>
> Reviewed-by: Guchun Chen <guchun.chen at amd.com>
> Ref: https://patchwork.freedesktop.org/patch/450065/?series=93662&rev=3
> Signed-off-by: Koba Ko <koba.ko at canonical.com>
> ---
>   drivers/gpu/drm/amd/amdgpu/vi.c | 9 ++++++++-
>   1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c
> index 735ebbd1148fd..3e911fcd9b1d1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/vi.c
> +++ b/drivers/gpu/drm/amd/amdgpu/vi.c
> @@ -904,7 +904,14 @@ 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) &&
> +		     (adev->pdev->subsystem_vendor == 0x1028) &&
> +		     (adev->pdev->subsystem_device == 0x0039))
> +			return false;
> +		else
> +			return amdgpu_dpm_is_baco_supported(adev);
>   	default:
>   		return false;
>   	}
> 

-- 
-----------
Tim Gardner
Canonical, Inc