| From 421a26f3d7a7c3ca43f3a9dc0f3cb0f562d5bd95 Mon Sep 17 00:00:00 2001 |
| From: Oliver Lang <Oliver.Lang@gossenmetrawatt.com> |
| Date: Thu, 10 Jun 2021 15:46:17 +0200 |
| Subject: iio: ltr501: ltr559: fix initialization of LTR501_ALS_CONTR |
| |
| From: Oliver Lang <Oliver.Lang@gossenmetrawatt.com> |
| |
| commit 421a26f3d7a7c3ca43f3a9dc0f3cb0f562d5bd95 upstream. |
| |
| The ltr559 chip uses only the lowest bit of the ALS_CONTR register to |
| configure between active and stand-by mode. In the original driver |
| BIT(1) is used, which does a software reset instead. |
| |
| This patch fixes the problem by using BIT(0) as als_mode_active for |
| the ltr559 chip. |
| |
| Fixes: 8592a7eefa54 ("iio: ltr501: Add support for ltr559 chip") |
| Signed-off-by: Oliver Lang <Oliver.Lang@gossenmetrawatt.com> |
| Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com> |
| Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> |
| Tested-by: Nikita Travkin <nikita@trvn.ru> # ltr559 |
| Link: https://lore.kernel.org/r/20210610134619.2101372-3-mkl@pengutronix.de |
| Cc: <Stable@vger.kernel.org> |
| Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/iio/light/ltr501.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/iio/light/ltr501.c |
| +++ b/drivers/iio/light/ltr501.c |
| @@ -1208,7 +1208,7 @@ static struct ltr501_chip_info ltr501_ch |
| .als_gain_tbl_size = ARRAY_SIZE(ltr559_als_gain_tbl), |
| .ps_gain = ltr559_ps_gain_tbl, |
| .ps_gain_tbl_size = ARRAY_SIZE(ltr559_ps_gain_tbl), |
| - .als_mode_active = BIT(1), |
| + .als_mode_active = BIT(0), |
| .als_gain_mask = BIT(2) | BIT(3) | BIT(4), |
| .als_gain_shift = 2, |
| .info = <r501_info, |
