diff options
Diffstat (limited to 'common/recipes-kernel/linux/linux-yocto-4.14.71/0855-drm-amdgpu-fixed-raven-psp-cmd-prepare-and-submit.patch')
-rw-r--r-- | common/recipes-kernel/linux/linux-yocto-4.14.71/0855-drm-amdgpu-fixed-raven-psp-cmd-prepare-and-submit.patch | 77 |
1 files changed, 77 insertions, 0 deletions
diff --git a/common/recipes-kernel/linux/linux-yocto-4.14.71/0855-drm-amdgpu-fixed-raven-psp-cmd-prepare-and-submit.patch b/common/recipes-kernel/linux/linux-yocto-4.14.71/0855-drm-amdgpu-fixed-raven-psp-cmd-prepare-and-submit.patch new file mode 100644 index 00000000..da05b95b --- /dev/null +++ b/common/recipes-kernel/linux/linux-yocto-4.14.71/0855-drm-amdgpu-fixed-raven-psp-cmd-prepare-and-submit.patch @@ -0,0 +1,77 @@ +From 030b2cdbc47e052815afd484480d883fe232f5a5 Mon Sep 17 00:00:00 2001 +From: Evan Quan <evan.quan@amd.com> +Date: Mon, 4 Sep 2017 17:42:28 +0800 +Subject: [PATCH 0855/4131] drm/amdgpu: fixed raven psp cmd prepare and submit +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +- fw_size in psp_v10_0_prep_cmd_buf is wrongly set as 0 +- fixed the wrong calculation of psp_write_ptr_reg in psp_v10_0_cmd_submit + +Change-Id: I9b7ebc99b7c75c03fb46d16c4c49348dd551325e +Signed-off-by: Evan Quan <evan.quan@amd.com> +Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com> +Acked-by: Christian König <christian.koenig@amd.com> +Reviewed-by: Huang Rui <ray.huang@amd.com> +--- + drivers/gpu/drm/amd/amdgpu/psp_v10_0.c | 16 +++++++++------- + 1 file changed, 9 insertions(+), 7 deletions(-) + +diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c b/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c +index 86db90f..8b30c2a 100644 +--- a/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c ++++ b/drivers/gpu/drm/amd/amdgpu/psp_v10_0.c +@@ -139,15 +139,13 @@ int psp_v10_0_prep_cmd_buf(struct amdgpu_firmware_info *ucode, struct psp_gfx_cm + { + int ret; + uint64_t fw_mem_mc_addr = ucode->mc_addr; +- struct common_firmware_header *header; + + memset(cmd, 0, sizeof(struct psp_gfx_cmd_resp)); +- header = (struct common_firmware_header *)ucode->fw; + + cmd->cmd_id = GFX_CMD_ID_LOAD_IP_FW; + cmd->cmd.cmd_load_ip_fw.fw_phy_addr_lo = lower_32_bits(fw_mem_mc_addr); + cmd->cmd.cmd_load_ip_fw.fw_phy_addr_hi = upper_32_bits(fw_mem_mc_addr); +- cmd->cmd.cmd_load_ip_fw.fw_size = le32_to_cpu(header->ucode_size_bytes); ++ cmd->cmd.cmd_load_ip_fw.fw_size = ucode->ucode_size; + + ret = psp_v10_0_get_fw_type(ucode, &cmd->cmd.cmd_load_ip_fw.fw_type); + if (ret) +@@ -248,15 +246,20 @@ int psp_v10_0_cmd_submit(struct psp_context *psp, + struct psp_gfx_rb_frame * write_frame = psp->km_ring.ring_mem; + struct psp_ring *ring = &psp->km_ring; + struct amdgpu_device *adev = psp->adev; ++ uint32_t ring_size_dw = ring->ring_size / 4; ++ uint32_t rb_frame_size_dw = sizeof(struct psp_gfx_rb_frame) / 4; + + /* KM (GPCOM) prepare write pointer */ + psp_write_ptr_reg = RREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_67); + + /* Update KM RB frame pointer to new frame */ +- if ((psp_write_ptr_reg % ring->ring_size) == 0) ++ if ((psp_write_ptr_reg % ring_size_dw) == 0) + write_frame = ring->ring_mem; + else +- write_frame = ring->ring_mem + (psp_write_ptr_reg / (sizeof(struct psp_gfx_rb_frame) / 4)); ++ write_frame = ring->ring_mem + (psp_write_ptr_reg / rb_frame_size_dw); ++ ++ /* Initialize KM RB frame */ ++ memset(write_frame, 0, sizeof(struct psp_gfx_rb_frame)); + + /* Update KM RB frame */ + write_frame->cmd_buf_addr_hi = upper_32_bits(cmd_buf_mc_addr); +@@ -266,8 +269,7 @@ int psp_v10_0_cmd_submit(struct psp_context *psp, + write_frame->fence_value = index; + + /* Update the write Pointer in DWORDs */ +- psp_write_ptr_reg += sizeof(struct psp_gfx_rb_frame) / 4; +- psp_write_ptr_reg = (psp_write_ptr_reg >= ring->ring_size) ? 0 : psp_write_ptr_reg; ++ psp_write_ptr_reg = (psp_write_ptr_reg + rb_frame_size_dw) % ring_size_dw; + WREG32_SOC15(MP0, 0, mmMP0_SMN_C2PMSG_67, psp_write_ptr_reg); + + return 0; +-- +2.7.4 + |