Djam/arm-trusted-firmware
1
0
Fork 0
mirror of https://github.com/ARM-software/arm-trusted-firmware.git synced 2025-04-23 13:36:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(mmc): explicitly check operators precedence

Browse source 
This corrects the MISRA violation C2012-12.1:
The precedence of operators within expressions should be made explicit
While at it put the test on one line.

Signed-off-by: Yann Gautier <yann.gautier@st.com>
Change-Id: Id19231faaf033c1e7dbe703690e0dddc7aa8f346
This commit is contained in:
Yann Gautier 2022-11-18 15:02:35 +01:00
parent d1c6c49554
commit 14cda5168d
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
drivers/mmc/mmc.c
View file

@ -780,8 +780,7 @@ int mmc_part_switch_current_boot(void)
unsigned char current_boot_part = mmc_current_boot_part(); unsigned char current_boot_part = mmc_current_boot_part();
int ret; int ret;
if (current_boot_part != 1U && if ((current_boot_part != 1U) && (current_boot_part != 2U)) {
current_boot_part != 2U) {
ERROR("Got unexpected value for active boot partition, %u\n", current_boot_part); ERROR("Got unexpected value for active boot partition, %u\n", current_boot_part);
return -EIO; return -EIO;
} }