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>
Acked-by: Alex Deucher <alexander.deuc...@amd.com>
Reviewed-by: Guchun Chen <guchun.c...@amd.com>
--
v1->v2:
  - limit the SKU further by subsystem IDs (Alex)
---
 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 fe9a7cc8d9eb..42a35d9520f9 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;
        }
-- 
2.29.0

Reply via email to