[AMD Official Use Only]

Reviewed-by: Bokun Zhang <bokun.zh...@amd.com>

Thanks!

-----Original Message-----
From: Yifan Zha <yifan....@amd.com> 
Sent: Tuesday, March 1, 2022 2:19 AM
To: amd-gfx@lists.freedesktop.org; Zhang, Bokun <bokun.zh...@amd.com>
Cc: Chen, Jingwen <jingwen.c...@amd.com>; Liu, Monk <monk....@amd.com>; Chen, 
Guchun <guchun.c...@amd.com>; Zha, YiFan(Even) <yifan....@amd.com>
Subject: [PATCH] drm/amdgpu: Move CAP firmware loading to the beginning of PSP 
firmware list

[Why]
As PSP needs to verify the signature, CAP firmware must be loaded first when 
PSP loads firmwares.
Otherwise, when DFC feature is enabled, CP firmwares would be loaded failed.

[ 1149.160480] [drm] MM table gpu addr = 0x800022f000, cpu addr = 
00000000a62afcea.
[ 1149.209874] [drm] failed to load ucode CP_CE(0x8) [ 1149.209878] [drm] psp 
gfx command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007) [ 
1149.215914] [drm] failed to load ucode CP_PFP(0x9) [ 1149.215917] [drm] psp 
gfx command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007) [ 
1149.221941] [drm] failed to load ucode CP_ME(0xA) [ 1149.221944] [drm] psp gfx 
command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007) [ 
1149.228082] [drm] failed to load ucode CP_MEC1(0xB) [ 1149.228085] [drm] psp 
gfx command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007) [ 
1149.234209] [drm] failed to load ucode CP_MEC2(0xD) [ 1149.234212] [drm] psp 
gfx command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007) [ 
1149.242379] [drm] failed to load ucode VCN(0x1C) [ 1149.242382] [drm] psp gfx 
command LOAD_IP_FW(0x6) failed and response status is (0xFFFF0007)

[How]
Move CAP UCODE ID to the beginning of AMDGPU_UCODE_ID enum list.

Signed-off-by: Yifan Zha <yifan....@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
index 428f4df184d0..40dffbac85a0 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.h
@@ -343,7 +343,8 @@ union amdgpu_firmware_header {
  * fw loading support
  */
 enum AMDGPU_UCODE_ID {
-       AMDGPU_UCODE_ID_SDMA0 = 0,
+       AMDGPU_UCODE_ID_CAP = 0,
+       AMDGPU_UCODE_ID_SDMA0,
        AMDGPU_UCODE_ID_SDMA1,
        AMDGPU_UCODE_ID_SDMA2,
        AMDGPU_UCODE_ID_SDMA3,
@@ -378,7 +379,6 @@ enum AMDGPU_UCODE_ID {
        AMDGPU_UCODE_ID_VCN0_RAM,
        AMDGPU_UCODE_ID_VCN1_RAM,
        AMDGPU_UCODE_ID_DMCUB,
-       AMDGPU_UCODE_ID_CAP,
        AMDGPU_UCODE_ID_MAXIMUM,
 };
 
--
2.25.1

Reply via email to