commit f7a20d8db012b2894d1518f58e84790fcf986b1b
author Kevin Peng <kevin.peng@arm.com> Mon Dec 13 14:38:37 2021 +0800
committer Ken Liu <ken.liu@arm.com> Thu Dec 16 13:43:31 2021 +0100
tree 8699ea7b808e9d6999f512ce910d966366a32a0b
parent 4cc609deed33526a8cee0d7fce132c52f98b5d66

SPM: Make the parameters of interrupt APIs common

The following APIs used to be in the SVC handler codes.
The parameters are not describing.
- tfm_flih_prepare_depriv_flih
- tfm_flih_return_to_isr

Now since they are been separated in dedicated file, the prototype
should be describing.

This patch changes the parameters for this purpose.

Change-Id: Iec908551b39faf9899f7434670698f399bacda3e
Signed-off-by: Kevin Peng <kevin.peng@arm.com>

secure_fw/spm/ffm/interrupt.c

diff --git a/secure_fw/spm/ffm/interrupt.c b/secure_fw/spm/ffm/interrupt.c
index 8a54e0f..87c0751 100644
--- a/secure_fw/spm/ffm/interrupt.c
+++ b/secure_fw/spm/ffm/interrupt.c
@@ -8,7 +8,6 @@
 #include "interrupt.h"
 
 #include "bitops.h"
-#include "spm_ipc.h"
 #include "tfm_arch.h"
 #include "tfm_hal_interrupt.h"
 #include "tfm_hal_isolation.h"
@@ -50,10 +49,10 @@
 
 extern void tfm_flih_func_return(psa_flih_result_t result);
 
-uint32_t tfm_flih_prepare_depriv_flih(uint32_t *svc_args)
+uint32_t tfm_flih_prepare_depriv_flih(struct partition_t *p_owner_sp,
+                                      uintptr_t flih_func)
 {
     struct partition_t *p_curr_sp;
-    struct partition_t *p_owner_sp = (struct partition_t *)svc_args[0];
     uintptr_t sp_limit, stack;
     struct context_ctrl_t flih_ctx_ctrl;
 
@@ -85,7 +84,7 @@
     }
 
     tfm_arch_init_context(&flih_ctx_ctrl,
-                          (uintptr_t)svc_args[1], NULL,
+                          flih_func, NULL,
                           (uintptr_t)tfm_flih_func_return,
                           sp_limit, stack);
 
@@ -95,11 +94,10 @@
 }
 
 /* Go back to ISR from FLIH functions */
-uint32_t tfm_flih_return_to_isr(psa_flih_result_t result, uint32_t *msp)
+uint32_t tfm_flih_return_to_isr(psa_flih_result_t result,
+                                struct context_flih_ret_t *p_ctx_flih_ret)
 {
     struct partition_t *p_prev_sp, *p_owner_sp;
-    struct context_flih_ret_t *p_ctx_flih_ret =
-                                               (struct context_flih_ret_t *)msp;
 
     p_prev_sp = GET_CTX_OWNER(p_ctx_flih_ret->state_ctx.r2);
     p_owner_sp = GET_CTX_OWNER(CURRENT_THREAD->p_context_ctrl);