diff options
| author | Simon Glass <sjg@chromium.org> | 2023-05-08 16:01:39 -0600 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-05-11 14:04:25 +0000 |
| commit | 47acf64182ccc833bb590c6d307d6e7c66a93360 (patch) | |
| tree | 551e66d043888d04d9507865a8265285032defc1 /test | |
| parent | 67debcce5555daf1c24e1eada34f7607bf041308 (diff) | |
| download | chrome-ec-47acf64182ccc833bb590c6d307d6e7c66a93360.tar.gz | |
charger: Rename charge_get_state() to led_pwr_get_state()
We have a new 'charge state' in struct charge_state_data as part of the
charge_state_v2.h file. This is confusing.
Rename this function to match the enum it is referencing.
This makes no functional change.
BUG=b:218332694
TEST=make buildall
Change-Id: I7f08dd4fb11e2939e0ef779018c1e3206ebc8d20
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4516188
Commit-Queue: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Aaron Massey <aaronmassey@google.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/sbs_charging_v2.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/sbs_charging_v2.c b/test/sbs_charging_v2.c index 0b12aea739..482770ff1c 100644 --- a/test/sbs_charging_v2.c +++ b/test/sbs_charging_v2.c @@ -127,7 +127,7 @@ test_static int wait_charging_state(void) enum led_pwr_state state; task_wake(TASK_ID_CHARGER); msleep(WAIT_CHARGER_TASK); - state = charge_get_state(); + state = led_pwr_get_state(); ccprintf("[CHARGING TEST] state = %d\n", state); return state; } @@ -504,7 +504,7 @@ test_static int test_external_funcs(void) TEST_ASSERT(!(flags & CHARGE_FLAG_FORCE_IDLE)); /* and the rest */ - TEST_ASSERT(charge_get_state() == PWR_STATE_CHARGE); + TEST_ASSERT(led_pwr_get_state() == PWR_STATE_CHARGE); TEST_ASSERT(!charge_want_shutdown()); TEST_ASSERT(charge_get_percent() == 50); temp = 0; |