diff options
author | Marco Chen <marcochen@google.com> | 2017-11-09 11:27:24 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-11-09 21:46:04 -0800 |
commit | d972385836212826d9179f4532e33e41b85f185f (patch) | |
tree | 3a8d75049d94de7f55788ca471831170d00057ea | |
parent | d51eb529b611d31a8827c5a6c0f13aeae22fd47d (diff) | |
download | chrome-ec-d972385836212826d9179f4532e33e41b85f185f.tar.gz |
poppy/soraka: set attenuation_factor of opt3001 to 1.
attenuation_factor is set to 1 because there is a calibration in iio
framework of kernel already which would be configured in the factory
flow.
BRANCH=None
BUG=b:69025351
TEST=Manually test on the DUT to make sure there is no additional
factor applied to raw data.
Change-Id: I25c5a3b341573bb82828164d3243f9adbac05372
Signed-off-by: Marco Chen <marcochen@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/759765
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
-rw-r--r-- | board/poppy/board.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/board/poppy/board.c b/board/poppy/board.c index 513f17441c..58b108bb26 100644 --- a/board/poppy/board.c +++ b/board/poppy/board.c @@ -536,8 +536,12 @@ BUILD_ASSERT(ARRAY_SIZE(temp_sensors) == TEMP_SENSOR_COUNT); /* ALS instances. Must be in same order as enum als_id. */ struct als_t als[] = { - /* TODO(crosbug.com/p/61098): verify attenuation_factor */ - {"TI", opt3001_init, opt3001_read_lux, 5}, + /* + * attenuation_factor is set to 1 because there would be a calibration + * in iio framework of kernel which would be configured in factory + * flow. + */ + {"TI", opt3001_init, opt3001_read_lux, 1}, }; BUILD_ASSERT(ARRAY_SIZE(als) == ALS_COUNT); |