diff options
| author | Tom Hughes <tomhughes@chromium.org> | 2022-10-12 12:48:14 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-10-17 22:57:31 +0000 |
| commit | 4f320abaf6c88861573830284fec79c1f356ae44 (patch) | |
| tree | 32300b650e9a6e9df88c7cd4aa0f103f72c60ec0 | |
| parent | b5ec6ecf73d287551fb2e2def5c23022c9533b43 (diff) | |
| download | chrome-ec-4f320abaf6c88861573830284fec79c1f356ae44.tar.gz | |
driver/als_si114x: Add missing case statement
When building with clang it warns:
driver/als_si114x.c:317:10: error: no case matching constant switch
condition '0' [-Werror]
switch (SI114X_NUM_LEDS) {
^~~~~~~~~~~~~~~
./driver/als_si114x.h:35:25: note: expanded from macro 'SI114X_NUM_LEDS'
^~~~~~~~~~~~~~~~~~~~~~~~~~
BRANCH=none
BUG=b:172020503
TEST=CC=clang make BOARD=eve
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Change-Id: I94db5530c681c438db53bb065c45b223caea7917
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3949622
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
| -rw-r--r-- | driver/als_si114x.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/driver/als_si114x.c b/driver/als_si114x.c index fb54a7e4ac..30f92e23d1 100644 --- a/driver/als_si114x.c +++ b/driver/als_si114x.c @@ -321,6 +321,7 @@ static int si114x_set_chlist(const struct motion_sensor_t *s) reg |= SI114X_PARAM_CHLIST_EN_PS2; case 1: reg |= SI114X_PARAM_CHLIST_EN_PS1; + case 0: break; } @@ -417,6 +418,8 @@ static int si114x_initialize(const struct motion_sensor_t *s) ret = raw_write8(s->port, s->i2c_spi_addr_flags, SI114X_PS_LED21, 0x0f); break; + case 0: + break; } if (ret != EC_SUCCESS) return ret; |