<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word"> [Public] <div class="WordSection1"> Signed-off-by: Candice Li <a href="mailto:candice.li@amd.com"> candice.li@amd.com</a><o:p></o:p> ---<o:p></o:p> drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 8 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 108 ++++++++++++----------<o:p></o:p> drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h | 32 +++----<o:p></o:p> drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 2 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c | 2 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 4 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/psp_v11_0.c | 8 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/psp_v12_0.c | 4 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/psp_v13_0.c | 6 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 4 +-<o:p></o:p> drivers/gpu/drm/amd/amdgpu/soc15.c | 4 +-<o:p></o:p> 11 files changed, 96 insertions(+), 86 deletions(-)<o:p></o:p> <o:p> </o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c<o:p></o:p> index 66bf8b0c56bb..c239de0ace4f 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c<o:p></o:p> @@ -373,8 +373,8 @@ static int amdgpu_firmware_info(struct drm_amdgpu_info_firmware *fw_info,<o:p></o:p> fw_info->feature = adev->sdma.instance[query_fw->index].feature_version;<o:p></o:p> break;<o:p></o:p> case AMDGPU_INFO_FW_SOS:<o:p></o:p> - fw_info->ver = adev->psp.sos_fw_version;<o:p></o:p> - fw_info->feature = adev->psp.sos_feature_version;<o:p></o:p> + fw_info->ver = adev->psp.sos.fw_version;<o:p></o:p> + fw_info->feature = adev->psp.sos.feature_version;<o:p></o:p> break;<o:p></o:p> case AMDGPU_INFO_FW_ASD:<o:p></o:p> fw_info->ver = adev->psp.asd_fw_version;<o:p></o:p> @@ -389,8 +389,8 @@ static int amdgpu_firmware_info(struct drm_amdgpu_info_firmware *fw_info,<o:p></o:p> fw_info->feature = 0;<o:p></o:p> break;<o:p></o:p> case AMDGPU_INFO_FW_TOC:<o:p></o:p> - fw_info->ver = adev->psp.toc_fw_version;<o:p></o:p> - fw_info->feature = adev->psp.toc_feature_version;<o:p></o:p> + fw_info->ver = adev->psp.toc.fw_version;<o:p></o:p> + fw_info->feature = adev->psp.toc.feature_version;<o:p></o:p> break;<o:p></o:p> default:<o:p></o:p> return -EINVAL;<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<o:p></o:p> index bd079979d5a3..14ade5dfef75 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c<o:p></o:p> @@ -510,9 +510,9 @@ static int psp_load_toc(struct psp_context *psp,<o:p></o:p> return -ENOMEM;<o:p></o:p> /* Copy toc to psp firmware private buffer */<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->toc_start_addr, psp->toc_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->toc.start_addr, psp->toc.size_bytes);<o:p></o:p> <o:p></o:p> - psp_prep_load_toc_cmd_buf(cmd, psp->fw_pri_mc_addr, psp->toc_bin_size);<o:p></o:p> + psp_prep_load_toc_cmd_buf(cmd, psp->fw_pri_mc_addr, psp->toc.size_bytes);<o:p></o:p> <o:p></o:p> ret = psp_cmd_submit_buf(psp, NULL, cmd,<o:p></o:p> psp->fence_buf_mc_addr);<o:p></o:p> @@ -542,8 +542,8 @@ static int psp_tmr_init(struct psp_context *psp)<o:p></o:p> /* For ASICs support RLC autoload, psp will parse the toc<o:p></o:p> * and calculate the total size of TMR needed */<o:p></o:p> if (!amdgpu_sriov_vf(psp->adev) &&<o:p></o:p> - psp->toc_start_addr &&<o:p></o:p> - psp->toc_bin_size &&<o:p></o:p> + psp->toc.start_addr &&<o:p></o:p> + psp->toc.size_bytes &&<o:p></o:p> psp->fw_pri_buf) {<o:p></o:p> ret = psp_load_toc(psp, &tmr_size);<o:p></o:p> if (ret) {<o:p></o:p> @@ -722,18 +722,18 @@ static int psp_rl_load(struct amdgpu_device *adev)<o:p></o:p> struct psp_context *psp = &adev->psp;<o:p></o:p> struct psp_gfx_cmd_resp *cmd = psp->cmd;<o:p></o:p> <o:p></o:p> - if (psp->rl_bin_size == 0)<o:p></o:p> + if (!is_psp_fw_valid(psp->rl))<o:p></o:p> return 0;<o:p></o:p> <o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->rl_start_addr, psp->rl_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->rl.start_addr, psp->rl.size_bytes);<o:p></o:p> <o:p></o:p> memset(cmd, 0, sizeof(struct psp_gfx_cmd_resp));<o:p></o:p> <o:p></o:p> cmd->cmd_id = GFX_CMD_ID_LOAD_IP_FW;<o:p></o:p> cmd->cmd.cmd_load_ip_fw.fw_phy_addr_lo = lower_32_bits(psp->fw_pri_mc_addr);<o:p></o:p> cmd->cmd.cmd_load_ip_fw.fw_phy_addr_hi = upper_32_bits(psp->fw_pri_mc_addr);<o:p></o:p> - cmd->cmd.cmd_load_ip_fw.fw_size = psp->rl_bin_size;<o:p></o:p> + cmd->cmd.cmd_load_ip_fw.fw_size = psp->rl.size_bytes;<o:p></o:p> cmd->cmd.cmd_load_ip_fw.fw_type = GFX_FW_TYPE_REG_LIST;<o:p></o:p> <o:p></o:p> return psp_cmd_submit_buf(psp, NULL, cmd, psp->fence_buf_mc_addr);<o:p></o:p> @@ -2129,7 +2129,7 @@ static int psp_hw_start(struct psp_context *psp)<o:p></o:p> int ret;<o:p></o:p> <o:p></o:p> if (!amdgpu_sriov_vf(adev)) {<o:p></o:p> - if (psp->kdb_bin_size &&<o:p></o:p> + if ((is_psp_fw_valid(psp->kdb)) &&<o:p></o:p> (psp->funcs->bootloader_load_kdb != NULL)) {<o:p></o:p> ret = psp_bootloader_load_kdb(psp);<o:p></o:p> if (ret) {<o:p></o:p> @@ -2138,7 +2138,8 @@ static int psp_hw_start(struct psp_context *psp)<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p></o:p> - if (psp->spl_bin_size) {<o:p></o:p> + if ((is_psp_fw_valid(psp->spl)) &&<o:p></o:p> + (psp->funcs->bootloader_load_spl != NULL)) {<o:p></o:p> ret = psp_bootloader_load_spl(psp);<o:p></o:p> if (ret) {<o:p></o:p> DRM_ERROR("PSP load spl failed!\n");<o:p></o:p> @@ -2146,16 +2147,22 @@ static int psp_hw_start(struct psp_context *psp)<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p></o:p> - ret = psp_bootloader_load_sysdrv(psp);<o:p></o:p> - if (ret) {<o:p></o:p> - DRM_ERROR("PSP load sysdrv failed!\n");<o:p></o:p> - return ret;<o:p></o:p> + if ((is_psp_fw_valid(psp->sys)) &&<o:p></o:p> + (psp->funcs->bootloader_load_sysdrv != NULL)) {<o:p></o:p> + ret = psp_bootloader_load_sysdrv(psp);<o:p></o:p> + if (ret) {<o:p></o:p> + DRM_ERROR("PSP load sysdrv failed!\n");<o:p></o:p> + return ret;<o:p></o:p> + }<o:p></o:p> }<o:p></o:p> <o:p></o:p> - ret = psp_bootloader_load_sos(psp);<o:p></o:p> - if (ret) {<o:p></o:p> - DRM_ERROR("PSP load sos failed!\n");<o:p></o:p> - return ret;<o:p></o:p> + if ((is_psp_fw_valid(psp->sos)) &&<o:p></o:p> + (psp->funcs->bootloader_load_sos != NULL)) {<o:p></o:p> + ret = psp_bootloader_load_sos(psp);<o:p></o:p> + if (ret) {<o:p></o:p> + DRM_ERROR("PSP load sos failed!\n");<o:p></o:p> + return ret;<o:p></o:p> + }<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p></o:p> @@ -2997,10 +3004,10 @@ int psp_init_toc_microcode(struct psp_context *psp,<o:p></o:p> goto out;<o:p></o:p> <o:p></o:p> toc_hdr = (const struct psp_firmware_header_v1_0 *)adev->psp.toc_fw->data;<o:p></o:p> - adev->psp.toc_fw_version = le32_to_cpu(toc_hdr->header.ucode_version);<o:p></o:p> - adev->psp.toc_feature_version = le32_to_cpu(toc_hdr->sos.fw_version);<o:p></o:p> - adev->psp.toc_bin_size = le32_to_cpu(toc_hdr->header.ucode_size_bytes);<o:p></o:p> - adev->psp.toc_start_addr = (uint8_t *)toc_hdr +<o:p></o:p> + adev->psp.toc.fw_version = le32_to_cpu(toc_hdr->header.ucode_version);<o:p></o:p> + adev->psp.toc.feature_version = le32_to_cpu(toc_hdr->sos.fw_version);<o:p></o:p> + adev->psp.toc.size_bytes = le32_to_cpu(toc_hdr->header.ucode_size_bytes);<o:p></o:p> + adev->psp.toc.start_addr = (uint8_t *)toc_hdr +<o:p></o:p> le32_to_cpu(toc_hdr->header.ucode_array_offset_bytes);<o:p></o:p> return 0;<o:p></o:p> out:<o:p></o:p> @@ -3021,32 +3028,32 @@ static int psp_init_sos_base_fw(struct amdgpu_device *adev)<o:p></o:p> le32_to_cpu(sos_hdr->header.ucode_array_offset_bytes);<o:p></o:p> <o:p></o:p> if (adev->gmc.xgmi.connected_to_cpu || (adev->asic_type != CHIP_ALDEBARAN)) {<o:p></o:p> - adev->psp.sos_fw_version = le32_to_cpu(sos_hdr->header.ucode_version);<o:p></o:p> - adev->psp.sos_feature_version = le32_to_cpu(sos_hdr->sos.fw_version);<o:p></o:p> + adev->psp.sos.fw_version = le32_to_cpu(sos_hdr->header.ucode_version);<o:p></o:p> + adev->psp.sos.feature_version = le32_to_cpu(sos_hdr->sos.fw_version);<o:p></o:p> <o:p></o:p> - adev->psp.sys_bin_size = le32_to_cpu(sos_hdr->sos.offset_bytes);<o:p></o:p> - adev->psp.sys_start_addr = ucode_array_start_addr;<o:p></o:p> + adev->psp.sys.size_bytes = le32_to_cpu(sos_hdr->sos.offset_bytes);<o:p></o:p> + adev->psp.sys.start_addr = ucode_array_start_addr;<o:p></o:p> <o:p></o:p> - adev->psp.sos_bin_size = le32_to_cpu(sos_hdr->sos.size_bytes);<o:p></o:p> - adev->psp.sos_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.sos.size_bytes = le32_to_cpu(sos_hdr->sos.size_bytes);<o:p></o:p> + adev->psp.sos.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr->sos.offset_bytes);<o:p></o:p> } else {<o:p></o:p> /* Load alternate PSP SOS FW */<o:p></o:p> sos_hdr_v1_3 = (const struct psp_firmware_header_v1_3 *)adev->psp.sos_fw->data;<o:p></o:p> <o:p></o:p> - adev->psp.sos_fw_version = le32_to_cpu(sos_hdr_v1_3->sos_aux.fw_version);<o:p></o:p> - adev->psp.sos_feature_version = le32_to_cpu(sos_hdr_v1_3->sos_aux.fw_version);<o:p></o:p> + adev->psp.sos.fw_version = le32_to_cpu(sos_hdr_v1_3->sos_aux.fw_version);<o:p></o:p> + adev->psp.sos.feature_version = le32_to_cpu(sos_hdr_v1_3->sos_aux.fw_version);<o:p></o:p> <o:p></o:p> - adev->psp.sys_bin_size = le32_to_cpu(sos_hdr_v1_3->sys_drv_aux.size_bytes);<o:p></o:p> - adev->psp.sys_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.sys.size_bytes = le32_to_cpu(sos_hdr_v1_3->sys_drv_aux.size_bytes);<o:p></o:p> + adev->psp.sys.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->sys_drv_aux.offset_bytes);<o:p></o:p> <o:p></o:p> - adev->psp.sos_bin_size = le32_to_cpu(sos_hdr_v1_3->sos_aux.size_bytes);<o:p></o:p> - adev->psp.sos_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.sos.size_bytes = le32_to_cpu(sos_hdr_v1_3->sos_aux.size_bytes);<o:p></o:p> + adev->psp.sos.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->sos_aux.offset_bytes);<o:p></o:p> }<o:p></o:p> <o:p></o:p> - if ((adev->psp.sys_bin_size == 0) || (adev->psp.sos_bin_size == 0)) {<o:p></o:p> + if ((adev->psp.sys.size_bytes == 0) || (adev->psp.sos.size_bytes == 0)) {<o:p></o:p> dev_warn(adev->dev, "PSP SOS FW not available");<o:p></o:p> return -EINVAL;<o:p></o:p> }<o:p></o:p> @@ -3093,32 +3100,32 @@ int psp_init_sos_microcode(struct psp_context *psp,<o:p></o:p> <o:p></o:p> if (sos_hdr->header.header_version_minor == 1) {<o:p></o:p> sos_hdr_v1_1 = (const struct psp_firmware_header_v1_1 *)adev->psp.sos_fw->data;<o:p></o:p> - adev->psp.toc_bin_size = le32_to_cpu(sos_hdr_v1_1->toc.size_bytes);<o:p></o:p> - adev->psp.toc_start_addr = (uint8_t *)adev->psp.sys_start_addr +<o:p></o:p> + adev->psp.toc.size_bytes = le32_to_cpu(sos_hdr_v1_1->toc.size_bytes);<o:p></o:p> + adev->psp.toc.start_addr = (uint8_t *)adev->psp.sys.start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_1->toc.offset_bytes);<o:p></o:p> - adev->psp.kdb_bin_size = le32_to_cpu(sos_hdr_v1_1->kdb.size_bytes);<o:p></o:p> - adev->psp.kdb_start_addr = (uint8_t *)adev->psp.sys_start_addr +<o:p></o:p> + adev->psp.kdb.size_bytes = le32_to_cpu(sos_hdr_v1_1->kdb.size_bytes);<o:p></o:p> + adev->psp.kdb.start_addr = (uint8_t *)adev->psp.sys.start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_1->kdb.offset_bytes);<o:p></o:p> }<o:p></o:p> if (sos_hdr->header.header_version_minor == 2) {<o:p></o:p> sos_hdr_v1_2 = (const struct psp_firmware_header_v1_2 *)adev->psp.sos_fw->data;<o:p></o:p> - adev->psp.kdb_bin_size = le32_to_cpu(sos_hdr_v1_2->kdb.size_bytes);<o:p></o:p> - adev->psp.kdb_start_addr = (uint8_t *)adev->psp.sys_start_addr +<o:p></o:p> + adev->psp.kdb.size_bytes = le32_to_cpu(sos_hdr_v1_2->kdb.size_bytes);<o:p></o:p> + adev->psp.kdb.start_addr = (uint8_t *)adev->psp.sys.start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_2->kdb.offset_bytes);<o:p></o:p> }<o:p></o:p> if (sos_hdr->header.header_version_minor == 3) {<o:p></o:p> sos_hdr_v1_3 = (const struct psp_firmware_header_v1_3 *)adev->psp.sos_fw->data;<o:p></o:p> - adev->psp.toc_bin_size = le32_to_cpu(sos_hdr_v1_3->v1_1.toc.size_bytes);<o:p></o:p> - adev->psp.toc_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.toc.size_bytes = le32_to_cpu(sos_hdr_v1_3->v1_1.toc.size_bytes);<o:p></o:p> + adev->psp.toc.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->v1_1.toc.offset_bytes);<o:p></o:p> - adev->psp.kdb_bin_size = le32_to_cpu(sos_hdr_v1_3->v1_1.kdb.size_bytes);<o:p></o:p> - adev->psp.kdb_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.kdb.size_bytes = le32_to_cpu(sos_hdr_v1_3->v1_1.kdb.size_bytes);<o:p></o:p> + adev->psp.kdb.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->v1_1.kdb.offset_bytes);<o:p></o:p> - adev->psp.spl_bin_size = le32_to_cpu(sos_hdr_v1_3->spl.size_bytes);<o:p></o:p> - adev->psp.spl_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.spl.size_bytes = le32_to_cpu(sos_hdr_v1_3->spl.size_bytes);<o:p></o:p> + adev->psp.spl.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->spl.offset_bytes);<o:p></o:p> - adev->psp.rl_bin_size = le32_to_cpu(sos_hdr_v1_3->rl.size_bytes);<o:p></o:p> - adev->psp.rl_start_addr = ucode_array_start_addr +<o:p></o:p> + adev->psp.rl.size_bytes = le32_to_cpu(sos_hdr_v1_3->rl.size_bytes);<o:p></o:p> + adev->psp.rl.start_addr = ucode_array_start_addr +<o:p></o:p> le32_to_cpu(sos_hdr_v1_3->rl.offset_bytes);<o:p></o:p> }<o:p></o:p> break;<o:p></o:p> @@ -3345,7 +3352,10 @@ static DEVICE_ATTR(usbc_pd_fw, S_IRUGO | S_IWUSR,<o:p></o:p> psp_usbc_pd_fw_sysfs_read,<o:p></o:p> psp_usbc_pd_fw_sysfs_write);<o:p></o:p> <o:p></o:p> -<o:p></o:p> +int is_psp_fw_valid(struct psp_bin_desc bin)<o:p></o:p> +{<o:p></o:p> + return bin.size_bytes;<o:p></o:p> +}<o:p></o:p> <o:p></o:p> const struct amd_ip_funcs psp_ip_funcs = {<o:p></o:p> .name = "psp",<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h<o:p></o:p> index f5b967b18e3f..fbbb1abf639f 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.h<o:p></o:p> @@ -48,6 +48,7 @@<o:p></o:p> struct psp_context;<o:p></o:p> struct psp_xgmi_node_info;<o:p></o:p> struct psp_xgmi_topology_info;<o:p></o:p> +struct psp_bin_desc;<o:p></o:p> <o:p></o:p> enum psp_bootloader_cmd {<o:p></o:p> PSP_BL__LOAD_SYSDRV = 0x10000,<o:p></o:p> @@ -283,6 +284,13 @@ struct psp_runtime_boot_cfg_entry {<o:p></o:p> uint32_t reserved;<o:p></o:p> };<o:p></o:p> <o:p></o:p> +struct psp_bin_desc {<o:p></o:p> + uint32_t fw_version;<o:p></o:p> + uint32_t feature_version;<o:p></o:p> + uint32_t size_bytes;<o:p></o:p> + uint8_t *start_addr;<o:p></o:p> +};<o:p></o:p> +<o:p></o:p> struct psp_context<o:p></o:p> {<o:p></o:p> struct amdgpu_device *adev;<o:p></o:p> @@ -298,20 +306,12 @@ struct psp_context<o:p></o:p> <o:p></o:p> /* sos firmware */<o:p></o:p> const struct firmware *sos_fw;<o:p></o:p> - uint32_t sos_fw_version;<o:p></o:p> - uint32_t sos_feature_version;<o:p></o:p> - uint32_t sys_bin_size;<o:p></o:p> - uint32_t sos_bin_size;<o:p></o:p> - uint32_t toc_bin_size;<o:p></o:p> - uint32_t kdb_bin_size;<o:p></o:p> - uint32_t spl_bin_size;<o:p></o:p> - uint32_t rl_bin_size;<o:p></o:p> - uint8_t *sys_start_addr;<o:p></o:p> - uint8_t *sos_start_addr;<o:p></o:p> - uint8_t *toc_start_addr;<o:p></o:p> - uint8_t *kdb_start_addr;<o:p></o:p> - uint8_t *spl_start_addr;<o:p></o:p> - uint8_t *rl_start_addr;<o:p></o:p> + struct psp_bin_desc sys;<o:p></o:p> + struct psp_bin_desc sos;<o:p></o:p> + struct psp_bin_desc toc;<o:p></o:p> + struct psp_bin_desc kdb;<o:p></o:p> + struct psp_bin_desc spl;<o:p></o:p> + struct psp_bin_desc rl;<o:p></o:p> <o:p></o:p> /* tmr buffer */<o:p></o:p> struct amdgpu_bo *tmr_bo;<o:p></o:p> @@ -326,8 +326,6 @@ struct psp_context<o:p></o:p> <o:p></o:p> /* toc firmware */<o:p></o:p> const struct firmware *toc_fw;<o:p></o:p> - uint32_t toc_fw_version;<o:p></o:p> - uint32_t toc_feature_version;<o:p></o:p> <o:p></o:p> /* fence buffer */<o:p></o:p> struct amdgpu_bo *fence_buf_bo;<o:p></o:p> @@ -482,4 +480,6 @@ int psp_get_fw_attestation_records_addr(struct psp_context *psp,<o:p></o:p> <o:p></o:p> int psp_load_fw_list(struct psp_context *psp,<o:p></o:p> struct amdgpu_firmware_info **ucode_list, int ucode_count);<o:p></o:p> +<o:p></o:p> +int is_psp_fw_valid(struct psp_bin_desc bin);<o:p></o:p> #endif<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c<o:p></o:p> index fe601f67d3a7..8e9b1f9fa34f 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c<o:p></o:p> @@ -518,7 +518,7 @@ FW_VERSION_ATTR(rlc_srlg_fw_version, 0444, gfx.rlc_srlg_fw_version);<o:p></o:p> FW_VERSION_ATTR(rlc_srls_fw_version, 0444, gfx.rlc_srls_fw_version);<o:p></o:p> FW_VERSION_ATTR(mec_fw_version, 0444, gfx.mec_fw_version);<o:p></o:p> FW_VERSION_ATTR(mec2_fw_version, 0444, gfx.mec2_fw_version);<o:p></o:p> -FW_VERSION_ATTR(sos_fw_version, 0444, psp.sos_fw_version);<o:p></o:p> +FW_VERSION_ATTR(sos_fw_version, 0444, psp.sos.fw_version);<o:p></o:p> FW_VERSION_ATTR(asd_fw_version, 0444, psp.asd_fw_version);<o:p></o:p> FW_VERSION_ATTR(ta_ras_fw_version, 0444, psp.ta_ras_ucode_version);<o:p></o:p> FW_VERSION_ATTR(ta_xgmi_fw_version, 0444, psp.ta_xgmi_ucode_version);<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c<o:p></o:p> index b71dd1deeb2d..12a7cc2f01cd 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_virt.c<o:p></o:p> @@ -531,7 +531,7 @@ static void amdgpu_virt_populate_vf2pf_ucode_info(struct amdgpu_device *adev)<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_RLC_SRLS, adev->gfx.rlc_srls_fw_version);<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_MEC, adev->gfx.mec_fw_version);<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_MEC2, adev->gfx.mec2_fw_version);<o:p></o:p> - POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_SOS, adev->psp.sos_fw_version);<o:p></o:p> + POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_SOS, adev->psp.sos.fw_version);<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_ASD, adev->psp.asd_fw_version);<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_TA_RAS, adev->psp.ta_ras_ucode_version);<o:p></o:p> POPULATE_UCODE_INFO(vf2pf_info, AMD_SRIOV_UCODE_ID_TA_XGMI, adev->psp.ta_xgmi_ucode_version);<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c<o:p></o:p> index 38738ca56786..479a57eaba8e 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c<o:p></o:p> @@ -5401,7 +5401,7 @@ static int gfx_v10_0_parse_rlc_toc(struct amdgpu_device *adev)<o:p></o:p> int ret;<o:p></o:p> RLC_TABLE_OF_CONTENT *rlc_toc;<o:p></o:p> <o:p></o:p> - ret = amdgpu_bo_create_reserved(adev, adev->psp.toc_bin_size, PAGE_SIZE,<o:p></o:p> + ret = amdgpu_bo_create_reserved(adev, adev->psp.toc.size_bytes, PAGE_SIZE,<o:p></o:p> AMDGPU_GEM_DOMAIN_GTT,<o:p></o:p> &adev->gfx.rlc.rlc_toc_bo,<o:p></o:p> &adev->gfx.rlc.rlc_toc_gpu_addr,<o:p></o:p> @@ -5412,7 +5412,7 @@ static int gfx_v10_0_parse_rlc_toc(struct amdgpu_device *adev)<o:p></o:p> }<o:p></o:p> <o:p></o:p> /* Copy toc from psp sos fw to rlc toc buffer */<o:p></o:p> - memcpy(adev->gfx.rlc.rlc_toc_buf, adev->psp.toc_start_addr, adev->psp.toc_bin_size);<o:p></o:p> + memcpy(adev->gfx.rlc.rlc_toc_buf, adev->psp.toc.start_addr, adev->psp.toc.size_bytes);<o:p></o:p> <o:p></o:p> rlc_toc = (RLC_TABLE_OF_CONTENT *)adev->gfx.rlc.rlc_toc_buf;<o:p></o:p> while (rlc_toc && (rlc_toc->id > FIRMWARE_ID_INVALID) &&<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c<o:p></o:p> index 21b8b6d48d8f..3eb14f7e3322 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v11_0.c<o:p></o:p> @@ -288,7 +288,7 @@ static int psp_v11_0_bootloader_load_kdb(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP KDB binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->kdb_start_addr, psp->kdb_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->kdb.start_addr, psp->kdb.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP KDB to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -321,7 +321,7 @@ static int psp_v11_0_bootloader_load_spl(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP SPL binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->spl_start_addr, psp->spl_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->spl.start_addr, psp->spl.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP SPL to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -354,7 +354,7 @@ static int psp_v11_0_bootloader_load_sysdrv(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP System Driver binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sys_start_addr, psp->sys_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sys.start_addr, psp->sys.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the sys driver to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -390,7 +390,7 @@ static int psp_v11_0_bootloader_load_sos(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy Secure OS binary to PSP memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sos_start_addr, psp->sos_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sos.start_addr, psp->sos.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP secure OS to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c b/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c<o:p></o:p> index b0ee77ee80b9..32413c059bee 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v12_0.c<o:p></o:p> @@ -140,7 +140,7 @@ static int psp_v12_0_bootloader_load_sysdrv(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP System Driver binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sys_start_addr, psp->sys_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sys.start_addr, psp->sys.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the sys driver to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -181,7 +181,7 @@ static int psp_v12_0_bootloader_load_sos(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy Secure OS binary to PSP memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sos_start_addr, psp->sos_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sos.start_addr, psp->sos.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP secure OS to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c<o:p></o:p> index e1046bb3bab0..d017da3ceadb 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v13_0.c<o:p></o:p> @@ -136,7 +136,7 @@ static int psp_v13_0_bootloader_load_kdb(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP KDB binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->kdb_start_addr, psp->kdb_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->kdb.start_addr, psp->kdb.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP KDB to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, regMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -169,7 +169,7 @@ static int psp_v13_0_bootloader_load_sysdrv(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP System Driver binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sys_start_addr, psp->sys_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sys.start_addr, psp->sys.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the sys driver to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, regMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -205,7 +205,7 @@ static int psp_v13_0_bootloader_load_sos(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy Secure OS binary to PSP memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sos_start_addr, psp->sos_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sos.start_addr, psp->sos.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP secure OS to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, regMP0_SMN_C2PMSG_36,<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c<o:p></o:p> index be05d9cbd41e..29aac7b793f4 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c<o:p></o:p> @@ -105,7 +105,7 @@ static int psp_v3_1_bootloader_load_sysdrv(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy PSP System Driver binary to memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sys_start_addr, psp->sys_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sys.start_addr, psp->sys.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the sys driver to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> @@ -146,7 +146,7 @@ static int psp_v3_1_bootloader_load_sos(struct psp_context *psp)<o:p></o:p> memset(psp->fw_pri_buf, 0, PSP_1_MEG);<o:p></o:p> <o:p></o:p> /* Copy Secure OS binary to PSP memory */<o:p></o:p> - memcpy(psp->fw_pri_buf, psp->sos_start_addr, psp->sos_bin_size);<o:p></o:p> + memcpy(psp->fw_pri_buf, psp->sos.start_addr, psp->sos.size_bytes);<o:p></o:p> <o:p></o:p> /* Provide the PSP secure OS to bootloader */<o:p></o:p> WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_36,<o:p></o:p> diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c b/drivers/gpu/drm/amd/amdgpu/soc15.c<o:p></o:p> index b7d350be8050..a5e085e570f7 100644<o:p></o:p> --- a/drivers/gpu/drm/amd/amdgpu/soc15.c<o:p></o:p> +++ b/drivers/gpu/drm/amd/amdgpu/soc15.c<o:p></o:p> @@ -575,7 +575,7 @@ soc15_asic_reset_method(struct amdgpu_device *adev)<o:p></o:p> baco_reset = amdgpu_dpm_is_baco_supported(adev);<o:p></o:p> break;<o:p></o:p> case CHIP_VEGA20:<o:p></o:p> - if (adev->psp.sos_fw_version >= 0x80067)<o:p></o:p> + if (adev->psp.sos.fw_version >= 0x80067)<o:p></o:p> baco_reset = amdgpu_dpm_is_baco_supported(adev);<o:p></o:p> <o:p></o:p> /*<o:p></o:p> @@ -635,7 +635,7 @@ static bool soc15_supports_baco(struct amdgpu_device *adev)<o:p></o:p> case CHIP_ARCTURUS:<o:p></o:p> return amdgpu_dpm_is_baco_supported(adev);<o:p></o:p> case CHIP_VEGA20:<o:p></o:p> - if (adev->psp.sos_fw_version >= 0x80067)<o:p></o:p> + if (adev->psp.sos.fw_version >= 0x80067)<o:p></o:p> return amdgpu_dpm_is_baco_supported(adev);<o:p></o:p> return false;<o:p></o:p> default:<o:p></o:p> -- <o:p></o:p> 2.17.1<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> <o:p> </o:p> Thanks,<o:p></o:p> Candice<o:p></o:p> <o:p> </o:p> </div> </body> </html>