Re: [PATCH] i2c: i801: add lis3lv02d's I2C address for Vostro 5568

From: Jean Delvare
Date: Tue Oct 18 2022 - 13:40:04 EST


On Thu, 06 Oct 2022 16:54:40 +0200, Nam Cao wrote:
> Dell Vostro 5568 laptop has lis3lv02d, but its i2c address is not known
> to the kernel. Add this address.
>
> Output of "cat /sys/devices/platform/lis3lv02d/position" on Dell Vostro
> 5568 laptop:
> - Horizontal: (-18,0,1044)
> - Front elevated: (522,-18,1080)
> - Left elevated: (-18,-360,1080)
> - Upside down: (36,108,-1134)
>
> Signed-off-by: Nam Cao <namcaov@xxxxxxxxx>
> ---
> drivers/i2c/busses/i2c-i801.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> index a176296f4fff..e46561e095c6 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1243,6 +1243,7 @@ static const struct {
> */
> { "Latitude 5480", 0x29 },
> { "Vostro V131", 0x1d },
> + { "Vostro 5568", 0x29 },
> };
>
> static void register_dell_lis3lv02d_i2c_device(struct i801_priv *priv)

Fine with me.

Reviewed-by: Jean Delvare <jdelvare@xxxxxxx>

Pali, OK with you?

--
Jean Delvare
SUSE L3 Support