diff options
| author | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-13 13:54:03 +0000 |
|---|---|---|
| committer | Jonathan Wright <jonathan.wright@arm.com> | 2018-03-26 12:43:05 +0100 |
| commit | c9662db960526757a755f34da4c79742a1c2acf3 (patch) | |
| tree | f8a08034e3c94d9837b2829715aab1a9ce8e77a9 /bl1 | |
| parent | 8ae0df93a01bfd869fa80068b5df17359d798714 (diff) | |
bl1: fix switch statements to comply with MISRA rules
Ensure (where possible) that switch statements in bl1 comply with MISRA
rules 16.1 - 16.7
Return statements inside switch clauses mean that we do not comply with
rule 16.3.
Change-Id: I8342389ba525dfc68b88e67dbb3690a529abfeb1
Signed-off-by: Jonathan Wright <jonathan.wright@arm.com>
Diffstat (limited to 'bl1')
| -rw-r--r-- | bl1/bl1_fwu.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/bl1/bl1_fwu.c b/bl1/bl1_fwu.c index ed027abc..49e4e8e2 100644 --- a/bl1/bl1_fwu.c +++ b/bl1/bl1_fwu.c @@ -87,6 +87,7 @@ register_t bl1_fwu_smc_handler(unsigned int smc_fid, case FWU_SMC_UPDATE_DONE: bl1_fwu_done((void *)x1, NULL); /* We should never return from bl1_fwu_done() */ + break; default: assert(0); @@ -747,6 +748,7 @@ static int bl1_fwu_image_reset(unsigned int image_id, unsigned int flags) case IMAGE_STATE_EXECUTED: default: assert(0); + break; } return 0; |