diff options
| author | Tom Hughes <tomhughes@chromium.org> | 2022-10-14 08:43:28 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-14 23:57:02 +0000 |
| commit | 68ac8294e9bd15dd79628fbca0aebae93865adb9 (patch) | |
| tree | 3cb9e618391041279e29e895eacdcbe6ac5d51da | |
| parent | 8c83707d8fef9f644b79e498c9ac998f82639d15 (diff) | |
| download | chrome-ec-68ac8294e9bd15dd79628fbca0aebae93865adb9.tar.gz | |
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 <tomhughes@chromium.org>
Change-Id: Iccc77d0cfc6c70635df8553406dbf28e5f37f129
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3957419
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
| -rw-r--r-- | driver/als_si114x.c | 4 |
1 files 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; } |