From 68ac8294e9bd15dd79628fbca0aebae93865adb9 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Fri, 14 Oct 2022 08:43:28 -0700 Subject: driver/als_si114x.c: Fix LED enable https://crrev.com/c/2603076 introduced a change that inadvertently used SI114X_PARAM_CHLIST_EN_PS3, regardless of the number of LEDs. This was found on inspection when fixing https://crrev.com/c/3949622. BRANCH=none BUG=b:172020503 TEST=make BOARD=eve Signed-off-by: Tom Hughes Change-Id: Iccc77d0cfc6c70635df8553406dbf28e5f37f129 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3957419 Reviewed-by: Daisuke Nojiri --- driver/als_si114x.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/driver/als_si114x.c b/driver/als_si114x.c index 3355fd0bd1..fb54a7e4ac 100644 --- a/driver/als_si114x.c +++ b/driver/als_si114x.c @@ -318,9 +318,9 @@ static int si114x_set_chlist(const struct motion_sensor_t *s) case 3: reg |= SI114X_PARAM_CHLIST_EN_PS3; case 2: - reg |= SI114X_PARAM_CHLIST_EN_PS3; + reg |= SI114X_PARAM_CHLIST_EN_PS2; case 1: - reg |= SI114X_PARAM_CHLIST_EN_PS3; + reg |= SI114X_PARAM_CHLIST_EN_PS1; break; } -- cgit v1.2.1