[PATCH] drm/amdgpu: Update PSP TA unload function

Clements, John John.Clements at amd.com
Tue Sep 14 13:13:49 UTC 2021


[AMD Official Use Only]

Reviewed-by: John Clements <john.clements at amd.com>

________________________________
From: Li, Candice <Candice.Li at amd.com>
Sent: Monday, September 13, 2021 3:55 PM
To: amd-gfx at lists.freedesktop.org <amd-gfx at lists.freedesktop.org>
Cc: Clements, John <John.Clements at amd.com>; Li, Candice <Candice.Li at amd.com>
Subject: [PATCH] drm/amdgpu: Update PSP TA unload function

Update PSP TA unload function to use PSP TA context as input argument.

Signed-off-by: Candice Li <candice.li at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
index bc861f2fe0ecf6..7d09b28889afef 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
@@ -46,7 +46,7 @@ static int psp_sysfs_init(struct amdgpu_device *adev);
 static void psp_sysfs_fini(struct amdgpu_device *adev);

 static int psp_load_smu_fw(struct psp_context *psp);
-static int psp_ta_unload(struct psp_context *psp, uint32_t session_id);
+static int psp_ta_unload(struct psp_context *psp, struct ta_context *context);
 static int psp_ta_load(struct psp_context *psp, struct ta_context *context);
 static int psp_rap_terminate(struct psp_context *psp);
 static int psp_securedisplay_terminate(struct psp_context *psp);
@@ -816,12 +816,12 @@ static void psp_prep_ta_unload_cmd_buf(struct psp_gfx_cmd_resp *cmd,
         cmd->cmd.cmd_unload_ta.session_id = session_id;
 }

-static int psp_ta_unload(struct psp_context *psp, uint32_t session_id)
+static int psp_ta_unload(struct psp_context *psp, struct ta_context *context)
 {
         int ret;
         struct psp_gfx_cmd_resp *cmd = acquire_psp_cmd_buf(psp);

-       psp_prep_ta_unload_cmd_buf(cmd, session_id);
+       psp_prep_ta_unload_cmd_buf(cmd, context->session_id);

         ret = psp_cmd_submit_buf(psp, NULL, cmd, psp->fence_buf_mc_addr);

@@ -832,7 +832,7 @@ static int psp_ta_unload(struct psp_context *psp, uint32_t session_id)

 static int psp_asd_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->asd_context.session_id);
+       return psp_ta_unload(psp, &psp->asd_context);
 }

 static int psp_asd_terminate(struct psp_context *psp)
@@ -984,7 +984,7 @@ static int psp_xgmi_load(struct psp_context *psp)

 static int psp_xgmi_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->xgmi_context.context.session_id);
+       return psp_ta_unload(psp, &psp->xgmi_context.context);
 }

 int psp_xgmi_invoke(struct psp_context *psp, uint32_t ta_cmd_id)
@@ -1275,7 +1275,7 @@ static int psp_ras_load(struct psp_context *psp)

 static int psp_ras_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->ras_context.context.session_id);
+       return psp_ta_unload(psp, &psp->ras_context.context);
 }

 int psp_ras_invoke(struct psp_context *psp, uint32_t ta_cmd_id)
@@ -1540,7 +1540,7 @@ static int psp_hdcp_initialize(struct psp_context *psp)

 static int psp_hdcp_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->hdcp_context.context.session_id);
+       return psp_ta_unload(psp, &psp->hdcp_context.context);
 }

 int psp_hdcp_invoke(struct psp_context *psp, uint32_t ta_cmd_id)
@@ -1632,7 +1632,7 @@ static int psp_dtm_initialize(struct psp_context *psp)

 static int psp_dtm_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->dtm_context.context.session_id);
+       return psp_ta_unload(psp, &psp->dtm_context.context);
 }

 int psp_dtm_invoke(struct psp_context *psp, uint32_t ta_cmd_id)
@@ -1690,7 +1690,7 @@ static int psp_rap_load(struct psp_context *psp)

 static int psp_rap_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->rap_context.context.session_id);
+       return psp_ta_unload(psp, &psp->rap_context.context);
 }

 static int psp_rap_initialize(struct psp_context *psp)
@@ -1805,7 +1805,7 @@ static int psp_securedisplay_load(struct psp_context *psp)

 static int psp_securedisplay_unload(struct psp_context *psp)
 {
-       return psp_ta_unload(psp, psp->securedisplay_context.context.session_id);
+       return psp_ta_unload(psp, &psp->securedisplay_context.context);
 }

 static int psp_securedisplay_initialize(struct psp_context *psp)
--
2.17.1

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20210914/4d9cf90f/attachment.htm>


More information about the amd-gfx mailing list