Browse Source

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 <michal.simek@amd.com>
Change-Id: I1113825baa5d9d58d9d7c5d9d5855fecf62e8d45
pull/1988/merge
Michal Simek 2 years ago
parent
commit
b0eb6d124b
  1. 3
      plat/xilinx/versal_net/plat_psci.c

3
plat/xilinx/versal_net/plat_psci.c

@ -18,7 +18,6 @@
#include <plat_private.h> #include <plat_private.h>
#define FUNCID_MASK U(0xffff)
#define PM_RET_ERROR_NOFEATURE U(19) #define PM_RET_ERROR_NOFEATURE U(19)
#define PM_IOCTL 34U #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; api_id = smc_fid & FUNCID_NUM_MASK;
VERBOSE("%s: smc_fid: %x, api_id=0x%x\n", __func__, smc_fid, api_id); 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: case PM_IOCTL:
{ {
ret = no_pm_ioctl(arg[0], arg[1], arg[2], arg[3]); ret = no_pm_ioctl(arg[0], arg[1], arg[2], arg[3]);

Loading…
Cancel
Save