diff options
author | Patrick Georgi <pgeorgi@google.com> | 2019-09-02 13:40:09 +0200 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-09-03 21:47:50 +0000 |
commit | f1a6c7142f5cede9ae05e4116e5cf317c91a0c1a (patch) | |
tree | 5bfe959fd1ca4a642f803992f04f67c22d7a3acd | |
parent | af51b9ea19c0ba4c6d57cdc4c5f3380647be3034 (diff) | |
download | chrome-ec-f1a6c7142f5cede9ae05e4116e5cf317c91a0c1a.tar.gz |
common/charge_ramp: code cleanup
No use keeping the return statement outside the #if bracket since
the preceding line is also a return statement.
Found by Coverity #190406
BUG=none
BRANCH=none
TEST=none
Change-Id: Ifcd36b9d65e4b2682337b4fc963d3dd64a066c7c
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1781464
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Patrick Georgi <pgeorgi@chromium.org>
Commit-Queue: Patrick Georgi <pgeorgi@chromium.org>
-rw-r--r-- | common/charge_ramp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/charge_ramp.c b/common/charge_ramp.c index 415e36a816..68a9c8ef96 100644 --- a/common/charge_ramp.c +++ b/common/charge_ramp.c @@ -46,8 +46,8 @@ test_mockable int chg_ramp_max(int supplier, int sup_curr) #ifdef CONFIG_CHARGE_RAMP_HW case CHARGE_SUPPLIER_PD: case CHARGE_SUPPLIER_TYPEC: -#endif return sup_curr; +#endif /* default: fall through */ } |