From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heinrich Schuchardt Date: Fri, 17 Jul 2020 00:46:05 +0200 Subject: [PATCH] efi_loader: Correct value for ARM_SMC_MM_RET_NO_MEMORY In-Reply-To: <20200716202754.164026-1-ilias.apalodimas@linaro.org> References: <20200716202754.164026-1-ilias.apalodimas@linaro.org> Message-ID: <2cd52149-e9bd-59b2-f9fe-bfe3aacbc7db@gmx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 7/16/20 10:27 PM, Ilias Apalodimas wrote: > Instead of adding the definition for the specific MM SVC used in > StandAloneMM we added the one used in the standard SMC calls. > So change the value from -4 to -5 to match the correct one defined in > EDK2 Please, rename all your MM return code constants to avoid further confusion: /* MM SVC return error codes */ #define ARM_SVC_SPM_RET_SUCCESS 0 #define ARM_SVC_SPM_RET_NOT_SUPPORTED -1 #define ARM_SVC_SPM_RET_INVALID_PARAMS -2 #define ARM_SVC_SPM_RET_DENIED -3 #define ARM_SVC_SPM_RET_NO_MEMORY -5 Best regards Heinrich > > Fixes 23a397d2e2fb: ("efi_loader: Add headers for EDK2 StandAloneMM communication") > Signed-off-by: Ilias Apalodimas > --- > include/mm_communication.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/mm_communication.h b/include/mm_communication.h > index 193c4d157874..5aa60160ac63 100644 > --- a/include/mm_communication.h > +++ b/include/mm_communication.h > @@ -52,14 +52,14 @@ struct efi_mm_communicate_header { > #define MM_COMMUNICATE_HEADER_SIZE \ > (sizeof(struct efi_mm_communicate_header)) > > -/* Defined in EDK2 ArmPkg/Include/IndustryStandard/ArmStdSmc.h */ > +/* Defined in EDK2 ArmPkg/Include/IndustryStandard/ArmMmSvc.h */ > > /* MM return error codes */ > #define ARM_SMC_MM_RET_SUCCESS 0 > #define ARM_SMC_MM_RET_NOT_SUPPORTED -1 > #define ARM_SMC_MM_RET_INVALID_PARAMS -2 > #define ARM_SMC_MM_RET_DENIED -3 > -#define ARM_SMC_MM_RET_NO_MEMORY -4 > +#define ARM_SMC_MM_RET_NO_MEMORY -5 > > /* Defined in EDK2 MdeModulePkg/Include/Guid/SmmVariableCommon.h */ > >