From f1a6c7142f5cede9ae05e4116e5cf317c91a0c1a Mon Sep 17 00:00:00 2001
From: Patrick Georgi <pgeorgi@google.com>
Date: Mon, 2 Sep 2019 13:40:09 +0200
Subject: 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>
---
 common/charge_ramp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'common/charge_ramp.c')

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 */
 	}
 
-- 
cgit v1.2.1