2020-06-17 00:03:41

by Jeffrey Lin

[permalink] [raw]
Subject: [PATCH] i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480

Value of /sys/devices/platform/lis3lv02d/position when
Horizontal: (36,-108,-1152)
Left elevated: (-432,-126,-1062)
Front elevated: (36,594,-936)
Upside down: (-126,-252,1098)

Signed-off-by: Jeffrey Lin <[email protected]>
---
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 fea644921a76..d7c802e20ae6 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1268,6 +1268,7 @@ static const struct {
/*
* Additional individual entries were added after verification.
*/
+ { "Latitude 5480", 0x29 },
{ "Vostro V131", 0x1d },
};

--
2.27.0


2020-08-05 09:34:57

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH] i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480

On Tue, Jun 16, 2020 at 07:41:30PM -0400, Jeffrey Lin wrote:
> Value of /sys/devices/platform/lis3lv02d/position when
> Horizontal: (36,-108,-1152)
> Left elevated: (-432,-126,-1062)
> Front elevated: (36,594,-936)
> Upside down: (-126,-252,1098)
>
> Signed-off-by: Jeffrey Lin <[email protected]>

Jean?

> ---
> 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 fea644921a76..d7c802e20ae6 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1268,6 +1268,7 @@ static const struct {
> /*
> * Additional individual entries were added after verification.
> */
> + { "Latitude 5480", 0x29 },
> { "Vostro V131", 0x1d },
> };
>
> --
> 2.27.0
>

Attachments:
(No filename) (867.00 B)
 signature.asc (849.00 B)
 Download all attachments

2020-08-31 08:16:51

by Jean Delvare

[permalink] [raw]
Subject: Re: [PATCH] i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480

Hi Wolfram,

On Wed, 05 Aug 2020 11:33:47 +0200, [email protected] wrote:
> On Tue, Jun 16, 2020 at 07:41:30PM -0400, Jeffrey Lin wrote:
> > Value of /sys/devices/platform/lis3lv02d/position when
> > Horizontal: (36,-108,-1152)
> > Left elevated: (-432,-126,-1062)
> > Front elevated: (36,594,-936)
> > Upside down: (-126,-252,1098)
> >
> > Signed-off-by: Jeffrey Lin <[email protected]>
>
> Jean?
>
> > ---
> > 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 fea644921a76..d7c802e20ae6 100644
> > --- a/drivers/i2c/busses/i2c-i801.c
> > +++ b/drivers/i2c/busses/i2c-i801.c
> > @@ -1268,6 +1268,7 @@ static const struct {
> > /*
> > * Additional individual entries were added after verification.
> > */
> > + { "Latitude 5480", 0x29 },
> > { "Vostro V131", 0x1d },
> > };
> >

No actual code change and not something I can test, so pretty much any
kernel developer is as qualified as I am to deal with it. But yes, OK,
I'm listed as the maintainer for this driver, so I should have replied
earlier, sorry.

Reviewed-by: Jean Delvare <[email protected]>

--
Jean Delvare
SUSE L3 Support

2020-09-02 05:02:08

by Jeffrey Lin

[permalink] [raw]
Subject: [PATCH v2] i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480

Value of /sys/devices/platform/lis3lv02d/position when
Horizontal: (36,-108,-1152)
Left elevated: (-432,-126,-1062)
Front elevated: (36,594,-936)
Upside down: (-126,-252,1098)

Signed-off-by: Jeffrey Lin <[email protected]>
Reviewed-by: Jean Delvare <[email protected]>
---

Changes in v2:
- Added Jean's Reviewed-by

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 e32ef3f01fe8..efab1e71ad6a 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1274,6 +1274,7 @@ static const struct {
/*
* Additional individual entries were added after verification.
*/
+ { "Latitude 5480", 0x29 },
{ "Vostro V131", 0x1d },
};

--
2.28.0

2020-09-09 08:30:56

by Wolfram Sang

[permalink] [raw]
Subject: Re: [PATCH v2] i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480

On Wed, Sep 02, 2020 at 12:51:37AM -0400, Jeffrey Lin wrote:
> Value of /sys/devices/platform/lis3lv02d/position when
> Horizontal: (36,-108,-1152)
> Left elevated: (-432,-126,-1062)
> Front elevated: (36,594,-936)
> Upside down: (-126,-252,1098)
>
> Signed-off-by: Jeffrey Lin <[email protected]>
> Reviewed-by: Jean Delvare <[email protected]>

Applied to for-next, thanks!

Attachments:
(No filename) (413.00 B)
 signature.asc (849.00 B)
 Download all attachments