From f234b0aeb2d51f13de683ed6400f3e07aa4b717c Mon Sep 17 00:00:00 2001 From: Daisuke Nojiri Date: Thu, 25 Oct 2018 15:10:59 -0700 Subject: 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 BUG=none BRANCH=none TEST=buildall Change-Id: I36156e7b376717a53e8a2cab7483b5150b4f2d61 Reviewed-on: https://chromium-review.googlesource.com/1300615 Commit-Ready: Daisuke Nojiri Tested-by: Daisuke Nojiri Reviewed-by: Daisuke Nojiri --- common/pd_log.c | 2 ++ 1 file changed, 2 insertions(+) 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 */ -- cgit v1.2.1