From 3cebeec2ae452d33ec0cea322f4ab18137e41631 Mon Sep 17 00:00:00 2001 From: Yann Gautier Date: Thu, 24 Nov 2022 18:17:02 +0100 Subject: [PATCH] fix(st-pmic): define pmic_regs table size This corrects MISRA C2012-9.5: Where designated initializers are used to initialize an array object the size of the array shall be specified explicitly. Signed-off-by: Yann Gautier Change-Id: I4f8da6b0ce73df65b2b45ba10d4ca16ed8b70113 --- drivers/st/pmic/stm32mp_pmic.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/st/pmic/stm32mp_pmic.c b/drivers/st/pmic/stm32mp_pmic.c index 2bf7ce74c..1e1628770 100644 --- a/drivers/st/pmic/stm32mp_pmic.c +++ b/drivers/st/pmic/stm32mp_pmic.c @@ -20,6 +20,7 @@ #include #define PMIC_NODE_NOT_FOUND 1 +#define NB_REG 14U static struct i2c_handle_s i2c_handle; static uint32_t pmic_i2c_addr; @@ -460,7 +461,7 @@ static const struct regul_ops pmic_ops = { .enable_ramp_delay = 1000, \ } -static const struct regul_description pmic_regs[] = { +static const struct regul_description pmic_regs[NB_REG] = { [STPMIC1_BUCK1] = DEFINE_REGU("buck1"), [STPMIC1_BUCK2] = DEFINE_REGU("buck2"), [STPMIC1_BUCK3] = DEFINE_REGU("buck3"), @@ -477,8 +478,6 @@ static const struct regul_description pmic_regs[] = { [STPMIC1_SW_OUT] = DEFINE_REGU("pwr_sw2"), }; -#define NB_REG ARRAY_SIZE(pmic_regs) - static int register_pmic(void) { void *fdt; @@ -506,7 +505,7 @@ static int register_pmic(void) unsigned int i; int ret; - for (i = 0; i < NB_REG; i++) { + for (i = 0U; i < NB_REG; i++) { desc = &pmic_regs[i]; if (strcmp(desc->node_name, reg_name) == 0) { break; -- 2.39.5