| Message ID | 20200616234130.814499-1-jeffrey@icurse.nl |
|---|---|
| State | Accepted |
| Commit | 62df579a86741436efae3d1bae29d0281078a7d3 |
| Headers | show |
| Series | i2c: i801: Register lis3lv02d I2C device on Dell Latitude 5480 | expand |
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 <jeffrey@icurse.nl> 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 >
Hi Wolfram, On Wed, 05 Aug 2020 11:33:47 +0200, wsa@kernel.org 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 <jeffrey@icurse.nl> > > 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 <jdelvare@suse.de> -- Jean Delvare SUSE L3 Support
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 <jeffrey@icurse.nl> > Reviewed-by: Jean Delvare <jdelvare@suse.de> Applied to for-next, thanks!
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 }, };
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 <jeffrey@icurse.nl> --- drivers/i2c/busses/i2c-i801.c | 1 + 1 file changed, 1 insertion(+)