diff options
| author | Daisuke Nojiri <dnojiri@chromium.org> | 2018-09-14 10:47:53 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-09-20 21:29:59 -0700 |
| commit | 69246cf51011ca951e5c405f5a4d2ee4448eb66c (patch) | |
| tree | d99ec6c42874d825f705bcd3ba3c7a709c103232 /board/fizz | |
| parent | d5b9b8c12570a57def102d0039f0aef379582fb0 (diff) | |
| download | chrome-ec-69246cf51011ca951e5c405f5a4d2ee4448eb66c.tar.gz | |
chgmgr: Allow charge_manager_update_charge to accept NULL
Currently, charge_manager_update_charge does not handle NULL pointer
for struct charge_port_info any differently. It's not sanity-checked
either (thus memory access violation can occur).
This patch will make charge_manager_update_charge accept NULL pointer
and set available current and voltage to zero.
This also helps callers' intentions be clear because callers can
explicitly specify NULL (instead of passing a pointer to chg = {0},
which is initialized somewhere else).
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
BUG=none
BRANCH=none
TEST=buildall
Change-Id: I518662ab6a3a07f93da5d34cf62a6f856884f67d
Reviewed-on: https://chromium-review.googlesource.com/1226125
Commit-Ready: Daisuke Nojiri <dnojiri@chromium.org>
Tested-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Diffstat (limited to 'board/fizz')
| -rw-r--r-- | board/fizz/board.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/board/fizz/board.c b/board/fizz/board.c index ac51d43c22..b3617155fa 100644 --- a/board/fizz/board.c +++ b/board/fizz/board.c @@ -748,13 +748,12 @@ static void setup_bj(void) static void board_charge_manager_init(void) { enum charge_port port; - struct charge_port_info cpi = { 0 }; int i, j; /* Initialize all charge suppliers to 0 */ for (i = 0; i < CHARGE_PORT_COUNT; i++) { for (j = 0; j < CHARGE_SUPPLIER_COUNT; j++) - charge_manager_update_charge(j, i, &cpi); + charge_manager_update_charge(j, i, NULL); } port = gpio_get_level(GPIO_ADP_IN_L) ? |