Commit feb000fd authored by Jane Jian's avatar Jane Jian Committed by Alex Deucher

drm/amd/powerplay: skip judging if baco support for Arcturus sriov

since for sriov, baco happens on host side, no need and meaning
to judge is baco.
also, since kiq reads strap0 in here, if kiq is not ready
or gpu reset(kiq resume) happens after this read, would fail
to read and wrongly set baco as true(1).
Acked-by: default avatarEvan Quan <evan.quan@amd.com>
Signed-off-by: default avatarJane Jian <Jane.Jian@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent b5861544
...@@ -2251,7 +2251,7 @@ static bool arcturus_is_baco_supported(struct smu_context *smu) ...@@ -2251,7 +2251,7 @@ static bool arcturus_is_baco_supported(struct smu_context *smu)
struct amdgpu_device *adev = smu->adev; struct amdgpu_device *adev = smu->adev;
uint32_t val; uint32_t val;
if (!smu_v11_0_baco_is_support(smu)) if (!smu_v11_0_baco_is_support(smu) || amdgpu_sriov_vf(adev))
return false; return false;
val = RREG32_SOC15(NBIO, 0, mmRCC_BIF_STRAP0); val = RREG32_SOC15(NBIO, 0, mmRCC_BIF_STRAP0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment