diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2018-10-25 15:10:59 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-12-17 20:47:14 -0800 |
commit | f234b0aeb2d51f13de683ed6400f3e07aa4b717c (patch) | |
tree | 956d43ea3cb0decf81fc8200466118b7f5c037af | |
parent | 8641a3876fb60bec2faf548c7baa53a562943c56 (diff) | |
download | chrome-ec-f234b0aeb2d51f13de683ed6400f3e07aa4b717c.tar.gz |
USB-PD: Don't make charge_manager save log if it's not present
Currently, pd_log assumes charge manager exists, thus, build fails
if CONFIG_CHARGE_MANAGER is undefined because charge_manager_save_log
is missing. This patch fixes it.
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
BUG=none
BRANCH=none
TEST=buildall
Change-Id: I36156e7b376717a53e8a2cab7483b5150b4f2d61
Reviewed-on: https://chromium-review.googlesource.com/1300615
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
-rw-r--r-- | common/pd_log.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/common/pd_log.c b/common/pd_log.c index f0e9312117..39a4edd207 100644 --- a/common/pd_log.c +++ b/common/pd_log.c @@ -102,7 +102,9 @@ static int hc_pd_write_log_entry(struct host_cmd_handler_args *args) switch (type) { /* Charge event: Log data for all ports */ case PD_EVENT_MCU_CHARGE: +#ifdef CONFIG_CHARGE_MANAGER charge_manager_save_log(port); +#endif break; /* Other events: no extra data, just log event type + port */ |