Commit 5b03127d authored by Lijo Lazar's avatar Lijo Lazar Committed by Alex Deucher

drm/amdgpu: Skip TMR allocation if not required

On ASICs with PSPv13.0.6, TMR is reserved at boot time. There is no need
to allocate TMR region by driver. However, it's still required to send
SETUP_TMR command to PSP.
Signed-off-by: default avatarLijo Lazar <lijo.lazar@amd.com>
Reviewed-by: default avatarHawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 845c9b31
...@@ -703,8 +703,13 @@ static void psp_prep_tmr_cmd_buf(struct psp_context *psp, ...@@ -703,8 +703,13 @@ static void psp_prep_tmr_cmd_buf(struct psp_context *psp,
uint64_t tmr_mc, struct amdgpu_bo *tmr_bo) uint64_t tmr_mc, struct amdgpu_bo *tmr_bo)
{ {
struct amdgpu_device *adev = psp->adev; struct amdgpu_device *adev = psp->adev;
uint32_t size = amdgpu_bo_size(tmr_bo); uint32_t size = 0;
uint64_t tmr_pa = amdgpu_gmc_vram_pa(adev, tmr_bo); uint64_t tmr_pa = 0;
if (tmr_bo) {
size = amdgpu_bo_size(tmr_bo);
tmr_pa = amdgpu_gmc_vram_pa(adev, tmr_bo);
}
if (amdgpu_sriov_vf(psp->adev)) if (amdgpu_sriov_vf(psp->adev))
cmd->cmd_id = GFX_CMD_ID_SETUP_VMR; cmd->cmd_id = GFX_CMD_ID_SETUP_VMR;
...@@ -749,6 +754,16 @@ static int psp_load_toc(struct psp_context *psp, ...@@ -749,6 +754,16 @@ static int psp_load_toc(struct psp_context *psp,
return ret; return ret;
} }
static bool psp_boottime_tmr(struct psp_context *psp)
{
switch (psp->adev->ip_versions[MP0_HWIP][0]) {
case IP_VERSION(13, 0, 6):
return true;
default:
return false;
}
}
/* Set up Trusted Memory Region */ /* Set up Trusted Memory Region */
static int psp_tmr_init(struct psp_context *psp) static int psp_tmr_init(struct psp_context *psp)
{ {
...@@ -820,8 +835,9 @@ static int psp_tmr_load(struct psp_context *psp) ...@@ -820,8 +835,9 @@ static int psp_tmr_load(struct psp_context *psp)
cmd = acquire_psp_cmd_buf(psp); cmd = acquire_psp_cmd_buf(psp);
psp_prep_tmr_cmd_buf(psp, cmd, psp->tmr_mc_addr, psp->tmr_bo); psp_prep_tmr_cmd_buf(psp, cmd, psp->tmr_mc_addr, psp->tmr_bo);
DRM_INFO("reserve 0x%lx from 0x%llx for PSP TMR\n", if (psp->tmr_bo)
amdgpu_bo_size(psp->tmr_bo), psp->tmr_mc_addr); DRM_INFO("reserve 0x%lx from 0x%llx for PSP TMR\n",
amdgpu_bo_size(psp->tmr_bo), psp->tmr_mc_addr);
ret = psp_cmd_submit_buf(psp, NULL, cmd, ret = psp_cmd_submit_buf(psp, NULL, cmd,
psp->fence_buf_mc_addr); psp->fence_buf_mc_addr);
...@@ -2080,10 +2096,12 @@ static int psp_hw_start(struct psp_context *psp) ...@@ -2080,10 +2096,12 @@ static int psp_hw_start(struct psp_context *psp)
if (amdgpu_sriov_vf(adev) && amdgpu_in_reset(adev)) if (amdgpu_sriov_vf(adev) && amdgpu_in_reset(adev))
goto skip_pin_bo; goto skip_pin_bo;
ret = psp_tmr_init(psp); if (!psp_boottime_tmr(psp)) {
if (ret) { ret = psp_tmr_init(psp);
DRM_ERROR("PSP tmr init failed!\n"); if (ret) {
return ret; DRM_ERROR("PSP tmr init failed!\n");
return ret;
}
} }
skip_pin_bo: skip_pin_bo:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment