mirror of
https://gitee.com/bianbu-linux/linux-6.6
synced 2025-07-08 00:23:23 -04:00
drm/amd/pm: fix driver reload SMC firmware fail issue for smu13
issue calltrace: [ 402.773695] [drm] failed to load ucode SMC(0x2C) [ 402.773754] [drm] psp gfx command LOAD_IP_FW(0x6) failed and response status is (0x0) [ 402.773762] [drm:psp_load_smu_fw [amdgpu]] *ERROR* PSP load smu failed! [ 402.966758] [drm:psp_v13_0_ring_destroy [amdgpu]] *ERROR* Fail to stop psp ring [ 402.966949] [drm:psp_hw_init [amdgpu]] *ERROR* PSP firmware loading failed [ 402.967116] [drm:amdgpu_device_fw_loading [amdgpu]] *ERROR* hw_init of IP block <psp> failed -22 [ 402.967252] amdgpu 0000:03:00.0: amdgpu: amdgpu_device_ip_init failed [ 402.967255] amdgpu 0000:03:00.0: amdgpu: Fatal error during GPU init if not reset mp1 state during kernel driver unload, it will cause psp load pmfw fail at the second time. add PPSMC_MSG_PrepareMp1ForUnload support for smu_v13_0_0/smu_v13_0_7 Signed-off-by: Yang Wang <KevinYang.Wang@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
7fc83cd079
commit
72aeb6ee0c
3 changed files with 11 additions and 0 deletions
|
@ -66,6 +66,7 @@ static int smu_set_fan_control_mode(void *handle, u32 value);
|
||||||
static int smu_set_power_limit(void *handle, uint32_t limit);
|
static int smu_set_power_limit(void *handle, uint32_t limit);
|
||||||
static int smu_set_fan_speed_rpm(void *handle, uint32_t speed);
|
static int smu_set_fan_speed_rpm(void *handle, uint32_t speed);
|
||||||
static int smu_set_gfx_cgpg(struct smu_context *smu, bool enabled);
|
static int smu_set_gfx_cgpg(struct smu_context *smu, bool enabled);
|
||||||
|
static int smu_set_mp1_state(void *handle, enum pp_mp1_state mp1_state);
|
||||||
|
|
||||||
static int smu_sys_get_pp_feature_mask(void *handle,
|
static int smu_sys_get_pp_feature_mask(void *handle,
|
||||||
char *buf)
|
char *buf)
|
||||||
|
@ -1414,6 +1415,12 @@ static int smu_disable_dpms(struct smu_context *smu)
|
||||||
switch (adev->ip_versions[MP1_HWIP][0]) {
|
switch (adev->ip_versions[MP1_HWIP][0]) {
|
||||||
case IP_VERSION(13, 0, 0):
|
case IP_VERSION(13, 0, 0):
|
||||||
case IP_VERSION(13, 0, 7):
|
case IP_VERSION(13, 0, 7):
|
||||||
|
ret = smu_set_mp1_state(smu, PP_MP1_STATE_UNLOAD);
|
||||||
|
if (ret) {
|
||||||
|
dev_err(adev->dev, "Fail set mp1 state to UNLOAD!\n");
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
|
|
|
@ -118,6 +118,7 @@ static struct cmn2asic_msg_mapping smu_v13_0_0_message_map[SMU_MSG_MAX_COUNT] =
|
||||||
MSG_MAP(GetPptLimit, PPSMC_MSG_GetPptLimit, 0),
|
MSG_MAP(GetPptLimit, PPSMC_MSG_GetPptLimit, 0),
|
||||||
MSG_MAP(NotifyPowerSource, PPSMC_MSG_NotifyPowerSource, 0),
|
MSG_MAP(NotifyPowerSource, PPSMC_MSG_NotifyPowerSource, 0),
|
||||||
MSG_MAP(Mode1Reset, PPSMC_MSG_Mode1Reset, 0),
|
MSG_MAP(Mode1Reset, PPSMC_MSG_Mode1Reset, 0),
|
||||||
|
MSG_MAP(PrepareMp1ForUnload, PPSMC_MSG_PrepareMp1ForUnload, 0),
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct cmn2asic_mapping smu_v13_0_0_clk_map[SMU_CLK_COUNT] = {
|
static struct cmn2asic_mapping smu_v13_0_0_clk_map[SMU_CLK_COUNT] = {
|
||||||
|
@ -1662,6 +1663,7 @@ static const struct pptable_funcs smu_v13_0_0_ppt_funcs = {
|
||||||
.baco_exit = smu_v13_0_baco_exit,
|
.baco_exit = smu_v13_0_baco_exit,
|
||||||
.mode1_reset_is_support = smu_v13_0_0_is_mode1_reset_supported,
|
.mode1_reset_is_support = smu_v13_0_0_is_mode1_reset_supported,
|
||||||
.mode1_reset = smu_v13_0_mode1_reset,
|
.mode1_reset = smu_v13_0_mode1_reset,
|
||||||
|
.set_mp1_state = smu_cmn_set_mp1_state,
|
||||||
};
|
};
|
||||||
|
|
||||||
void smu_v13_0_0_set_ppt_funcs(struct smu_context *smu)
|
void smu_v13_0_0_set_ppt_funcs(struct smu_context *smu)
|
||||||
|
|
|
@ -116,6 +116,7 @@ static struct cmn2asic_msg_mapping smu_v13_0_7_message_map[SMU_MSG_MAX_COUNT] =
|
||||||
MSG_MAP(DramLogSetDramSize, PPSMC_MSG_DramLogSetDramSize, 0),
|
MSG_MAP(DramLogSetDramSize, PPSMC_MSG_DramLogSetDramSize, 0),
|
||||||
MSG_MAP(AllowGfxOff, PPSMC_MSG_AllowGfxOff, 0),
|
MSG_MAP(AllowGfxOff, PPSMC_MSG_AllowGfxOff, 0),
|
||||||
MSG_MAP(DisallowGfxOff, PPSMC_MSG_DisallowGfxOff, 0),
|
MSG_MAP(DisallowGfxOff, PPSMC_MSG_DisallowGfxOff, 0),
|
||||||
|
MSG_MAP(PrepareMp1ForUnload, PPSMC_MSG_PrepareMp1ForUnload, 0),
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct cmn2asic_mapping smu_v13_0_7_clk_map[SMU_CLK_COUNT] = {
|
static struct cmn2asic_mapping smu_v13_0_7_clk_map[SMU_CLK_COUNT] = {
|
||||||
|
@ -1647,6 +1648,7 @@ static const struct pptable_funcs smu_v13_0_7_ppt_funcs = {
|
||||||
.baco_set_state = smu_v13_0_7_baco_set_state,
|
.baco_set_state = smu_v13_0_7_baco_set_state,
|
||||||
.baco_enter = smu_v13_0_7_baco_enter,
|
.baco_enter = smu_v13_0_7_baco_enter,
|
||||||
.baco_exit = smu_v13_0_baco_exit,
|
.baco_exit = smu_v13_0_baco_exit,
|
||||||
|
.set_mp1_state = smu_cmn_set_mp1_state,
|
||||||
};
|
};
|
||||||
|
|
||||||
void smu_v13_0_7_set_ppt_funcs(struct smu_context *smu)
|
void smu_v13_0_7_set_ppt_funcs(struct smu_context *smu)
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue