diff options
author | Dimitris Papastamos <dimitris.papastamos@arm.com> | 2018-03-29 13:20:05 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-03-29 13:20:05 +0100 |
commit | 6ab136c258ed4616d8cf1ebf1c5a4a74b75d35bc (patch) | |
tree | 59d6d3499fff950a6a5e18593f2166cd9d0c6565 /plat/mediatek | |
parent | 875a85aae6b0c9ae411ed899f827c4bbc0703438 (diff) | |
parent | 185a23ffa3b6fc007021ac9bbab2213c63911c0a (diff) |
Merge pull request #1313 from jonathanwright-ARM/jw/MISRA-switch-statements
Fix switch statements to comply with MISRA rules
Diffstat (limited to 'plat/mediatek')
-rw-r--r-- | plat/mediatek/common/custom/oem_svc.c | 11 | ||||
-rw-r--r-- | plat/mediatek/common/mtk_sip_svc.c | 3 |
2 files changed, 5 insertions, 9 deletions
diff --git a/plat/mediatek/common/custom/oem_svc.c b/plat/mediatek/common/custom/oem_svc.c index 08baed87..49e7571d 100644 --- a/plat/mediatek/common/custom/oem_svc.c +++ b/plat/mediatek/common/custom/oem_svc.c @@ -41,15 +41,8 @@ uint64_t oem_smc_handler(uint32_t smc_fid, void *handle, uint64_t flags) { - uint64_t rc; - - switch (smc_fid) { - default: - rc = SMC_UNK; - WARN("Unimplemented OEM Call: 0x%x\n", smc_fid); - } - - SMC_RET1(handle, rc); + WARN("Unimplemented OEM Call: 0x%x\n", smc_fid); + SMC_RET1(handle, SMC_UNK); } /* diff --git a/plat/mediatek/common/mtk_sip_svc.c b/plat/mediatek/common/mtk_sip_svc.c index beb2a697..869a9590 100644 --- a/plat/mediatek/common/mtk_sip_svc.c +++ b/plat/mediatek/common/mtk_sip_svc.c @@ -71,6 +71,9 @@ uint64_t mediatek_sip_handler(uint32_t smc_fid, boot_to_kernel(x1, x2, x3, x4); SMC_RET0(handle); #endif + default: + /* Do nothing in default case */ + break; } } |