[AMD Official Use Only]

Looks ok to me . 
You can  add  reviewed-by: Shaoyun.liu <shaoyun....@amd.com>

-----Original Message-----
From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of David Yu
Sent: Friday, April 22, 2022 12:09 PM
To: amd-gfx@lists.freedesktop.org
Cc: Yu, David <david...@amd.com>
Subject: [PATCH] drm/amdgpu: Ta fw needs to be loaded for SRIOV aldebaran

Load ta fw during psp_init_sriov_microcode to enable XGMI. It is required to be 
loaded by both guest and host starting from Arcturus. Cap fw needs to be loaded 
first.
Fix previously patch that was pushed by mistake.

Signed-off-by: David Yu <david...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index 895251f42853..0bd22ebcc3d1 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -275,8 +275,8 @@ static int psp_init_sriov_microcode(struct psp_context *psp)
                ret = psp_init_cap_microcode(psp, "sienna_cichlid");
                break;
        case IP_VERSION(13, 0, 2):
-               ret = psp_init_ta_microcode(psp, "aldebaran");
-               ret &= psp_init_cap_microcode(psp, "aldebaran");
+               ret = psp_init_cap_microcode(psp, "aldebaran");
+               ret &= psp_init_ta_microcode(psp, "aldebaran");
                break;
        default:
                BUG();
-- 
2.25.1

Reply via email to