diff options
| author | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-13 15:24:29 +0000 |
|---|---|---|
| committer | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-26 12:43:05 +0100 |
| commit | 5aa7498abdddcfd4ac8a04d7726798d452f13bb2 (patch) | |
| tree | 9a6d278ea74afc2412e91b1a027ba448684ede7c /drivers/arm/cci/cci.c | |
| parent | c9662db960526757a755f34da4c79742a1c2acf3 (diff) | |
| download | platform_external_arm-trusted-firmware-5aa7498abdddcfd4ac8a04d7726798d452f13bb2.tar.gz platform_external_arm-trusted-firmware-5aa7498abdddcfd4ac8a04d7726798d452f13bb2.tar.bz2 platform_external_arm-trusted-firmware-5aa7498abdddcfd4ac8a04d7726798d452f13bb2.zip | |
drivers: fix switch statements to comply with MISRA rules
Ensure (where possible) that switch statements in drivers comply with
MISRA rules 16.1 - 16.7.
Change-Id: I7a91e04b02af80fbc4673a52293386c0f81a0f7a
Signed-off-by: Jonathan Wright <jonathan.wright@arm.com>
Diffstat (limited to 'drivers/arm/cci/cci.c')
| -rw-r--r-- | drivers/arm/cci/cci.c | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/drivers/arm/cci/cci.c b/drivers/arm/cci/cci.c index e15683840..81808b9f7 100644 --- a/drivers/arm/cci/cci.c +++ b/drivers/arm/cci/cci.c @@ -83,22 +83,25 @@ static unsigned int read_cci_part_number(uintptr_t base) */ static int get_slave_ports(unsigned int part_num) { - /* Macro to match CCI products */ -#define RET_ON_MATCH(product) \ - case CCI ## product ## _PART_NUM: \ - return CCI ## product ## _SLAVE_PORTS + int num_slave_ports = -1; switch (part_num) { - RET_ON_MATCH(400); - RET_ON_MATCH(500); - RET_ON_MATCH(550); - + case CCI400_PART_NUM: + num_slave_ports = CCI400_SLAVE_PORTS; + break; + case CCI500_PART_NUM: + num_slave_ports = CCI500_SLAVE_PORTS; + break; + case CCI550_PART_NUM: + num_slave_ports = CCI550_SLAVE_PORTS; + break; default: - return -1; + /* Do nothing in default case */ + break; } -#undef RET_ON_MATCH + return num_slave_ports; } #endif /* ENABLE_ASSERTIONS */ |