diff options
author | Daisuke Nojiri <dnojiri@chromium.org> | 2021-03-15 19:58:15 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-03-18 19:27:56 +0000 |
commit | 4126a065489fe40668186c5f5b4dee4797388b25 (patch) | |
tree | c23634ebfb60c035a64b705936372d210b0e6d51 /common/peripheral_charger.c | |
parent | 2913c2f6c5afa41f086a6f0242165cbdf9dad730 (diff) | |
download | chrome-ec-4126a065489fe40668186c5f5b4dee4797388b25.tar.gz |
PCHG: Count number of dropped events per port
Currently, dropped events are counted for all ports combined. This
patch makes them counted for each port separately.
BUG=b:173235954
BRANCH=trogdor
TEST=Verify 'pchg 0' prints dropped event count.
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Change-Id: I09380927613defe34bef0fece022fc9d47ebb8a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2770502
Diffstat (limited to 'common/peripheral_charger.c')
-rw-r--r-- | common/peripheral_charger.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/common/peripheral_charger.c b/common/peripheral_charger.c index c7371c5dc5..3c11c68be1 100644 --- a/common/peripheral_charger.c +++ b/common/peripheral_charger.c @@ -20,13 +20,11 @@ #define CPRINTS(fmt, args...) cprints(CC_PCHG, "PCHG: " fmt, ##args) -static int dropped_event; - static void pchg_queue_event(struct pchg *ctx, enum pchg_event event) { mutex_lock(&ctx->mtx); if (queue_add_unit(&ctx->events, &event) == 0) { - dropped_event++; + ctx->dropped_event_count++; CPRINTS("ERR: Queue is full"); } mutex_unlock(&ctx->mtx); @@ -487,6 +485,8 @@ static int cc_pchg(int argc, char **argv) ccprintf("P%d STATE_%s EVENT_%s SOC=%d%%\n", port, _text_state(ctx->state), _text_event(ctx->event), ctx->battery_percent); + ccprintf("error=0x%x dropped=%u\n", + ctx->error, ctx->dropped_event_count); return EC_SUCCESS; } |