From b0eb6d124b1764264778d17b1519bfe62b7b9337 Mon Sep 17 00:00:00 2001 From: Michal Simek Date: Mon, 3 Oct 2022 14:02:57 +0200 Subject: [PATCH] fix(versal-net): use api_id directly without FUNCID_MASK The purpose of this code is to extract api_id from smc_fid but this masking is done already in the code with using generic mask from smccc.h (FUNCID_NUM_MASK). That's why remove FUNCID_MASK is which not needed and actually also equal to already used FUNCID_NUM_MASK. Signed-off-by: Michal Simek Change-Id: I1113825baa5d9d58d9d7c5d9d5855fecf62e8d45 --- plat/xilinx/versal_net/plat_psci.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/plat/xilinx/versal_net/plat_psci.c b/plat/xilinx/versal_net/plat_psci.c index 299eca406..c5833a9ae 100644 --- a/plat/xilinx/versal_net/plat_psci.c +++ b/plat/xilinx/versal_net/plat_psci.c @@ -18,7 +18,6 @@ #include -#define FUNCID_MASK U(0xffff) #define PM_RET_ERROR_NOFEATURE U(19) #define PM_IOCTL 34U @@ -194,7 +193,7 @@ static uint64_t no_pm_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64 api_id = smc_fid & FUNCID_NUM_MASK; VERBOSE("%s: smc_fid: %x, api_id=0x%x\n", __func__, smc_fid, api_id); - switch (smc_fid & FUNCID_MASK) { + switch (api_id) { case PM_IOCTL: { ret = no_pm_ioctl(arg[0], arg[1], arg[2], arg[3]); -- 2.39.5