diff options
| author | Naman Patel <naman.patel@amd.com> | 2022-12-01 02:58:46 -0800 |
|---|---|---|
| committer | Naman Patel <naman.patel@amd.com> | 2022-12-01 03:11:04 -0800 |
| commit | e65584a017fadf002d5bdd1e95527c48610a6963 (patch) | |
| tree | 00f401f47c58f1ac3305d7806345b627edee8e72 /plat/xilinx/zynqmp/pm_service | |
| parent | 2b9c8b877c33782df1731b7b52658ce3aab7126d (diff) | |
| download | arm-trusted-firmware-e65584a017fadf002d5bdd1e95527c48610a6963.tar.gz | |
fix(zynqmp): initialize uint32 with value 0U in pm code
MISRA Violation: MISRA C-2012 Rule 7.2
- Initialize the unsigned int with value 0u in pm_service component.
Current misra warning detection tool is not reporting this as
warning. It reports only when the initialized value exceeds the
range of data type based on compiler used.
But, this change is added as a part of precaution as some other
misra checker tool may report it as violation of rule 7.2.
Signed-off-by: Naman Patel <naman.patel@amd.com>
Change-Id: I50a5cee2a077fe157e79757d959ce33064225af3
Diffstat (limited to 'plat/xilinx/zynqmp/pm_service')
| -rw-r--r-- | plat/xilinx/zynqmp/pm_service/pm_svc_main.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/plat/xilinx/zynqmp/pm_service/pm_svc_main.c b/plat/xilinx/zynqmp/pm_service/pm_svc_main.c index b45ce6c76..4adbef366 100644 --- a/plat/xilinx/zynqmp/pm_service/pm_svc_main.c +++ b/plat/xilinx/zynqmp/pm_service/pm_svc_main.c @@ -356,7 +356,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_FPGA_GET_STATUS: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_fpga_get_status(&value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value) << 32); @@ -374,7 +374,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, (uint64_t)result[2] | ((uint64_t)result[3] << 32)); case PM_IOCTL: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_ioctl(pm_arg[0], pm_arg[1], pm_arg[2], pm_arg[3], &value); @@ -401,7 +401,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_CLOCK_GETSTATE: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_clock_getstate(pm_arg[0], &value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value) << 32); @@ -413,7 +413,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_CLOCK_GETDIVIDER: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_clock_getdivider(pm_arg[0], &value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value) << 32); @@ -442,7 +442,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_CLOCK_GETPARENT: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_clock_getparent(pm_arg[0], &value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value) << 32U); @@ -476,7 +476,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_FPGA_READ: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_fpga_read(pm_arg[0], pm_arg[1], pm_arg[2], pm_arg[3], &value); @@ -485,7 +485,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_SECURE_AES: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_aes_engine(pm_arg[0], pm_arg[1], &value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value) << 32U); @@ -497,7 +497,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_PLL_GET_PARAMETER: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_pll_get_parameter(pm_arg[0], pm_arg[1], &value); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)value << 32U)); @@ -509,7 +509,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_PLL_GET_MODE: { - uint32_t mode = 0; + uint32_t mode = 0U; ret = pm_pll_get_mode(pm_arg[0], &mode); SMC_RET1(handle, (uint64_t)ret | ((uint64_t)mode << 32U)); @@ -517,7 +517,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_REGISTER_ACCESS: { - uint32_t value = 0; + uint32_t value = 0U; ret = pm_register_access(pm_arg[0], pm_arg[1], pm_arg[2], pm_arg[3], &value); @@ -526,7 +526,7 @@ uint64_t pm_smc_handler(uint32_t smc_fid, uint64_t x1, uint64_t x2, uint64_t x3, case PM_EFUSE_ACCESS: { - uint32_t value = 0; + uint32_t value = 0U; #if defined(ZYNQMP_SECURE_EFUSES) if (is_caller_non_secure(flags)) { |