diff options
author | Eric Yilun Lin <yllin@chromium.org> | 2019-12-18 13:59:21 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-12-23 15:07:34 +0000 |
commit | adf28b166427442c3534e6e70b2da9f39d11ced6 (patch) | |
tree | 1b4a52c17d6220c58dc8a31f3ae91dd1d3f3a7b7 | |
parent | bc25889ce6d7f0768bd9041d3c1f581d393e3315 (diff) | |
download | chrome-ec-adf28b166427442c3534e6e70b2da9f39d11ced6.tar.gz |
battery: combine strings to reduce code size
TEST=see kodama's RW flash space shrank by 44 bytes.
BUG=chromium:1034518
BRANCH=kukui
Change-Id: Ia21b31d08090dca1b91965f47a90120f0dfbde11
Signed-off-by: Eric Yilun Lin <yllin@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1972999
Reviewed-by: Ting Shen <phoenixshen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1979583
Tested-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Commit-Queue: Tim Wawrzynczak <twawrzynczak@chromium.org>
-rw-r--r-- | common/battery.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/common/battery.c b/common/battery.c index 67d08b4150..d55d4d393e 100644 --- a/common/battery.c +++ b/common/battery.c @@ -20,6 +20,7 @@ #define CPRINTF(format, args...) cprintf(CC_CHARGER, format, ## args) #define CPRINTS(format, args...) cprints(CC_CHARGER, format, ## args) +#define CUTOFFPRINTS(info) CPRINTS("%s %s", "Battery cut off", info) /* See config.h for details */ const static int batt_full_factor = CONFIG_BATT_FULL_FACTOR; @@ -305,9 +306,9 @@ static void pending_cutoff_deferred(void) rv = board_cut_off_battery(); if (rv == EC_RES_SUCCESS) - CPRINTS("Battery cut off succeeded."); + CUTOFFPRINTS("succeeded."); else - CPRINTS("Battery cut off failed!"); + CUTOFFPRINTS("failed!"); } DECLARE_DEFERRED(pending_cutoff_deferred); @@ -329,17 +330,17 @@ static enum ec_status battery_command_cutoff(struct host_cmd_handler_args *args) p = args->params; if (p->flags & EC_BATTERY_CUTOFF_FLAG_AT_SHUTDOWN) { battery_cutoff_state = BATTERY_CUTOFF_STATE_PENDING; - CPRINTS("Battery cut off at-shutdown is scheduled"); + CUTOFFPRINTS("at-shutdown is scheduled"); return EC_RES_SUCCESS; } } rv = board_cut_off_battery(); if (rv == EC_RES_SUCCESS) { - CPRINTS("Battery cut off is successful."); + CUTOFFPRINTS("is successful."); battery_cutoff_state = BATTERY_CUTOFF_STATE_CUT_OFF; } else { - CPRINTS("Battery cut off has failed."); + CUTOFFPRINTS("has failed."); } return rv; |