From 9efdbc2cd894e99680c191a3c8e4a868cec12282 Mon Sep 17 00:00:00 2001 From: Antonio Nino Diaz Date: Thu, 14 Feb 2019 11:40:19 +0000 Subject: SPM: Remove unnecessary register save Since commit 01fc1c24b9a0 ("BL31: Use helper function to save registers in SMC handler") all the general-purpose registers are saved when entering EL3. It isn't needed to save them here. Change-Id: Ic540a5441b89b70888da587ab8fc3b2508cef8cc Signed-off-by: Antonio Nino Diaz --- services/std_svc/spm/sprt.c | 8 -------- 1 file changed, 8 deletions(-) (limited to 'services') diff --git a/services/std_svc/spm/sprt.c b/services/std_svc/spm/sprt.c index 8aa2a88a..20ad2afb 100644 --- a/services/std_svc/spm/sprt.c +++ b/services/std_svc/spm/sprt.c @@ -178,14 +178,6 @@ static uintptr_t sprt_smc_handler(uint32_t smc_fid, u_register_t x1, SMC_RET1(handle, SPRT_VERSION_COMPILED); case SPRT_PUT_RESPONSE_AARCH64: - /* - * Registers x1-x3 aren't saved by default to the context, - * but they are needed after spm_sp_synchronous_exit() because - * they hold return values. - */ - SMC_SET_GP(handle, CTX_GPREG_X1, x1); - SMC_SET_GP(handle, CTX_GPREG_X2, x2); - SMC_SET_GP(handle, CTX_GPREG_X3, x3); spm_sp_synchronous_exit(SPRT_PUT_RESPONSE_AARCH64); case SPRT_YIELD_AARCH64: -- cgit v1.2.3