drm/amdgpu: Change a few function names
Function name "psp_np_fw_load" is not proper as people don't know _np_fw_ means "non psp firmware". Change the function name to psp_load_non_psp_fw for better understanding. Same thing for function psp_execute_np_fw_load. Signed-off-by: Oak Zeng <Oak.Zeng@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Reviewed-by: Christian Konig <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
95f71f12aa
commit
cd5955f401
@ -2373,7 +2373,7 @@ static int psp_prep_load_ip_fw_cmd_buf(struct amdgpu_firmware_info *ucode,
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int psp_execute_np_fw_load(struct psp_context *psp,
|
||||
static int psp_execute_non_psp_fw_load(struct psp_context *psp,
|
||||
struct amdgpu_firmware_info *ucode)
|
||||
{
|
||||
int ret = 0;
|
||||
@ -2409,7 +2409,7 @@ static int psp_load_smu_fw(struct psp_context *psp)
|
||||
}
|
||||
}
|
||||
|
||||
ret = psp_execute_np_fw_load(psp, ucode);
|
||||
ret = psp_execute_non_psp_fw_load(psp, ucode);
|
||||
|
||||
if (ret)
|
||||
DRM_ERROR("PSP load smu failed!\n");
|
||||
@ -2464,14 +2464,14 @@ int psp_load_fw_list(struct psp_context *psp,
|
||||
for (i = 0; i < ucode_count; ++i) {
|
||||
ucode = ucode_list[i];
|
||||
psp_print_fw_hdr(psp, ucode);
|
||||
ret = psp_execute_np_fw_load(psp, ucode);
|
||||
ret = psp_execute_non_psp_fw_load(psp, ucode);
|
||||
if (ret)
|
||||
return ret;
|
||||
}
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int psp_np_fw_load(struct psp_context *psp)
|
||||
static int psp_load_non_psp_fw(struct psp_context *psp)
|
||||
{
|
||||
int i, ret;
|
||||
struct amdgpu_firmware_info *ucode;
|
||||
@ -2510,7 +2510,7 @@ static int psp_np_fw_load(struct psp_context *psp)
|
||||
|
||||
psp_print_fw_hdr(psp, ucode);
|
||||
|
||||
ret = psp_execute_np_fw_load(psp, ucode);
|
||||
ret = psp_execute_non_psp_fw_load(psp, ucode);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
@ -2587,7 +2587,7 @@ skip_memalloc:
|
||||
if (ret)
|
||||
goto failed;
|
||||
|
||||
ret = psp_np_fw_load(psp);
|
||||
ret = psp_load_non_psp_fw(psp);
|
||||
if (ret)
|
||||
goto failed;
|
||||
|
||||
@ -2787,7 +2787,7 @@ static int psp_resume(void *handle)
|
||||
if (ret)
|
||||
goto failed;
|
||||
|
||||
ret = psp_np_fw_load(psp);
|
||||
ret = psp_load_non_psp_fw(psp);
|
||||
if (ret)
|
||||
goto failed;
|
||||
|
||||
@ -2885,7 +2885,7 @@ int psp_update_vcn_sram(struct amdgpu_device *adev, int inst_idx,
|
||||
ucode.mc_addr = cmd_gpu_addr;
|
||||
ucode.ucode_size = cmd_size;
|
||||
|
||||
return psp_execute_np_fw_load(&adev->psp, &ucode);
|
||||
return psp_execute_non_psp_fw_load(&adev->psp, &ucode);
|
||||
}
|
||||
|
||||
int psp_ring_cmd_submit(struct psp_context *psp,
|
||||
|
Loading…
Reference in New Issue
Block a user