Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-07 Thread Wolfram Sang
> > > > > + if (!known_i2c_address) { > > > > > + dev_warn(&priv->pci_dev->dev, > > > > > + "Accelerometer lis3lv02d i2c device is present > > > > > " > > > > > + "but its i2c address is unknown, skipping > > > > > ...\n"); > > > > > > >

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-05 Thread Benjamin Tissoires
On Jan 05 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 09:54:56 Dmitry Torokhov wrote: > > On Wed, Jan 04, 2017 at 06:46:19PM +0100, Wolfram Sang wrote: > > > > > > > How about: > > > > --- > > > > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:00 2001 > >

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-05 Thread Pali Rohár
On Wednesday 04 January 2017 09:54:56 Dmitry Torokhov wrote: > On Wed, Jan 04, 2017 at 06:46:19PM +0100, Wolfram Sang wrote: > > > > > How about: > > > --- > > > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:00 2001 > > > From: Benjamin Tissoires > > > Date: Wed, 4 Jan 2017 18:26

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Dmitry Torokhov
On Wed, Jan 04, 2017 at 02:00:30PM -0800, Dmitry Torokhov wrote: > On Wed, Jan 04, 2017 at 10:55:47PM +0100, Wolfram Sang wrote: > > > > > From e362a0277fd1bd6112f258664d8831d9bc6b78da Mon Sep 17 00:00:00 2001 > > > From: Dmitry Torokhov > > > Date: Wed, 4 Jan 2017 09:33:43 -0800 > > > Subject: [

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Dmitry Torokhov
On Wed, Jan 04, 2017 at 10:49:06PM +0100, Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Dmitry Torokhov wrote: > > On Wed, Jan 04, 2017 at 06:46:19PM +0100, Wolfram Sang wrote: > > > > > > > How about: > > > > --- > > > > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Wolfram Sang
> From e362a0277fd1bd6112f258664d8831d9bc6b78da Mon Sep 17 00:00:00 2001 > From: Dmitry Torokhov > Date: Wed, 4 Jan 2017 09:33:43 -0800 > Subject: [PATCH] i2c: do not enable fall back to Host Notify by default > > Falling back unconditionally to HostNotify as primary client's interrupt > breaks

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Dmitry Torokhov
On Wed, Jan 04, 2017 at 10:55:47PM +0100, Wolfram Sang wrote: > > > From e362a0277fd1bd6112f258664d8831d9bc6b78da Mon Sep 17 00:00:00 2001 > > From: Dmitry Torokhov > > Date: Wed, 4 Jan 2017 09:33:43 -0800 > > Subject: [PATCH] i2c: do not enable fall back to Host Notify by default > > > > Fallin

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Dmitry Torokhov wrote: > On Wed, Jan 04, 2017 at 06:46:19PM +0100, Wolfram Sang wrote: > > > > > How about: > > > --- > > > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:00 2001 > > > From: Benjamin Tissoires > > > Date: Wed, 4 Jan 2017 18:26:54 +01

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Jean Delvare
On Tue, 3 Jan 2017 12:59:37 -0800, Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote: > > Some distributions blacklist i2c-i801.ko module... And > > Any particular reason for that? At some point in time, the i2c-i801 driver caused problems on a few systems. They

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Dmitry Torokhov
On Wed, Jan 04, 2017 at 06:46:19PM +0100, Wolfram Sang wrote: > > > How about: > > --- > > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:00 2001 > > From: Benjamin Tissoires > > Date: Wed, 4 Jan 2017 18:26:54 +0100 > > Subject: [PATCH] I2C: add the source of the IRQ in struct i2c

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Wolfram Sang
> How about: > --- > From daa7571bbf337704332c0cfeec9b8fd5aeae596f Mon Sep 17 00:00:00 2001 > From: Benjamin Tissoires > Date: Wed, 4 Jan 2017 18:26:54 +0100 > Subject: [PATCH] I2C: add the source of the IRQ in struct i2c_client > > With commit 4d5538f5882a ("i2c: use an IRQ to report Host Notif

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 14:02:52 Benjamin Tissoires wrote: > > > > > No. DT platforms won't have an issue: they don't change anything, and > > > > > there will be a new /dev/freefall misc device for the platforms that > > > > > > > > And this

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 14:02:52 Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > On Wednesday 04 January 2017 12:22:23 Pali Rohár wrote: > > > On Wednesday 04 January 2017 11:32:33 Benjamin Tissoires wrote: > > > > On Jan 04 2017 or thereabouts, Pali Rohár wrot

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 12:22:23 Pali Rohár wrote: > > On Wednesday 04 January 2017 11:32:33 Benjamin Tissoires wrote: > > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > > On Wednesday 04 January 2017 11:13:06 Benjamin Tissoires wrot

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 11:25:44 Benjamin Tissoires wrote: > > On Jan 04 2017 or thereabouts, Andy Shevchenko wrote: > > > On Tue, Jan 3, 2017 at 10:39 PM, Pali Rohár wrote: > > > > > > > > With Michał we already discussed about it, see email

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 12:22:23 Pali Rohár wrote: > On Wednesday 04 January 2017 11:32:33 Benjamin Tissoires wrote: > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > On Wednesday 04 January 2017 11:13:06 Benjamin Tissoires wrote: > > > > On Jan 04 2017 or thereabouts, Pali Rohár wrot

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 11:25:44 Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Andy Shevchenko wrote: > > On Tue, Jan 3, 2017 at 10:39 PM, Pali Rohár wrote: > > > > > > With Michał we already discussed about it, see emails. Basically you can > > > enable/disable kernel modules at

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 11:32:33 Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > On Wednesday 04 January 2017 11:13:06 Benjamin Tissoires wrote: > > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > > On Wednesday 04 January 2017 10:05:22 Benjamin Tissoi

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 11:13:06 Benjamin Tissoires wrote: > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > On Wednesday 04 January 2017 10:05:22 Benjamin Tissoires wrote: > > > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > >

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 11:13:06 Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > On Wednesday 04 January 2017 10:05:22 Benjamin Tissoires wrote: > > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > > On Tuesday 03 January 2017 12:59:37 Dmitry Torokhov w

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Andy Shevchenko wrote: > On Tue, Jan 3, 2017 at 10:39 PM, Pali Rohár wrote: > > > > With Michał we already discussed about it, see emails. Basically you can > > enable/disable kernel modules at compile time or blacklist at runtime > > (or even chose what will be comp

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Jean Delvare
On Tue, 3 Jan 2017 10:06:41 +0100, Benjamin Tissoires wrote: > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > So 4d5538f5882a is breaking lis3lv02d driver... > > Apologies for that. > > I could easily fix this by adding a kernel API to know whether the > provided irq is from Host Notify or

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Wednesday 04 January 2017 10:05:22 Benjamin Tissoires wrote: > > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > > On Tuesday 03 January 2017 12:59:37 Dmitry Torokhov wrote: > > > > On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Andy Shevchenko
On Tue, Jan 3, 2017 at 10:39 PM, Pali Rohár wrote: > > With Michał we already discussed about it, see emails. Basically you can > enable/disable kernel modules at compile time or blacklist at runtime > (or even chose what will be compiled into vmlinux and what as external > .ko module). Some distr

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Jean Delvare
On Wed, 28 Dec 2016 15:02:52 +0100, Wolfram Sang wrote: > On Tue, Dec 27, 2016 at 02:51:01PM +0100, Pali Rohár wrote: > > i2c_new_device() with lis3lv02d for i801 i2c bus needs to be called > > after initializing i2c-i801 bus driver. > > > > I have no idea how to do it (properly) outside of i2c-i80

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 10:13:38 Benjamin Tissoires wrote: > On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > > On Tue, Jan 03, 2017 at 10:29:34PM +0100, Benjamin Tissoires wrote: > > > On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > > > > > > > Yet another option: can we add a

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 10:29:34PM +0100, Benjamin Tissoires wrote: > > On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > > > > > Yet another option: can we add a new flag to i2c_board_info controlling > > > whether we want to enable/dis

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Wednesday 04 January 2017 10:05:22 Benjamin Tissoires wrote: > On Jan 04 2017 or thereabouts, Pali Rohár wrote: > > On Tuesday 03 January 2017 12:59:37 Dmitry Torokhov wrote: > > > On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote: > > > > On Tuesday 03 January 2017 21:24:18 Dmitry Toro

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Benjamin Tissoires
On Jan 04 2017 or thereabouts, Pali Rohár wrote: > On Tuesday 03 January 2017 12:59:37 Dmitry Torokhov wrote: > > On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote: > > > On Tuesday 03 January 2017 21:24:18 Dmitry Torokhov wrote: > > > > On Tue, Jan 03, 2017 at 09:05:51PM +0100, Pali Rohár

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-04 Thread Pali Rohár
On Tuesday 03 January 2017 12:59:37 Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote: > > On Tuesday 03 January 2017 21:24:18 Dmitry Torokhov wrote: > > > On Tue, Jan 03, 2017 at 09:05:51PM +0100, Pali Rohár wrote: > > > > On Tuesday 03 January 2017 20:48:12 Dmitr

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Dmitry Torokhov
On Tue, Jan 03, 2017 at 10:29:34PM +0100, Benjamin Tissoires wrote: > On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > > > Yet another option: can we add a new flag to i2c_board_info controlling > > whether we want to enable/disable wiring up host notify interrupt? > > That should be fairly

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Benjamin Tissoires
On Jan 03 2017 or thereabouts, Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 07:50:17PM +0100, Pali Rohár wrote: > > On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: > > > On Tue, Jan 03, 2017 at 10:06:41AM +0100, Benjamin Tissoires wrote: > > > > On Dec 29 2016 or thereabouts, Pali Ro

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Dmitry Torokhov
On Tue, Jan 03, 2017 at 09:39:13PM +0100, Pali Rohár wrote: > On Tuesday 03 January 2017 21:24:18 Dmitry Torokhov wrote: > > On Tue, Jan 03, 2017 at 09:05:51PM +0100, Pali Rohár wrote: > > > On Tuesday 03 January 2017 20:48:12 Dmitry Torokhov wrote: > > > > On Tue, Jan 03, 2017 at 07:50:17PM +0100,

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Pali Rohár
On Tuesday 03 January 2017 21:24:18 Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 09:05:51PM +0100, Pali Rohár wrote: > > On Tuesday 03 January 2017 20:48:12 Dmitry Torokhov wrote: > > > On Tue, Jan 03, 2017 at 07:50:17PM +0100, Pali Rohár wrote: > > > > On Tuesday 03 January 2017 19:38:43 Dmitr

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Dmitry Torokhov
On Tue, Jan 03, 2017 at 09:05:51PM +0100, Pali Rohár wrote: > On Tuesday 03 January 2017 20:48:12 Dmitry Torokhov wrote: > > On Tue, Jan 03, 2017 at 07:50:17PM +0100, Pali Rohár wrote: > > > On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: > > > > On Tue, Jan 03, 2017 at 10:06:41AM +0100,

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Andy Shevchenko
On Tue, Jan 3, 2017 at 8:50 PM, Pali Rohár wrote: > On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: >> "IIRC both dell-smo8800 and lis3lv02d represent one HW device (that >> ST microelectronics accelerometer) but due to complicated HW >> abstraction and layers on Dell laptops it is han

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Pali Rohár
On Tuesday 03 January 2017 20:48:12 Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 07:50:17PM +0100, Pali Rohár wrote: > > On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: > > > On Tue, Jan 03, 2017 at 10:06:41AM +0100, Benjamin Tissoires > > > wrote: > > > > On Dec 29 2016 or thereabou

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Dmitry Torokhov
On Tue, Jan 03, 2017 at 07:50:17PM +0100, Pali Rohár wrote: > On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: > > On Tue, Jan 03, 2017 at 10:06:41AM +0100, Benjamin Tissoires wrote: > > > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > > > On Thursday 29 December 2016 22:09:32 Micha

RE: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Mario.Limonciello
en Eisinger ; > Gabriele Mazzotta ; Andy Lutomirski > ; Limonciello, Mario ; Alex > Hung ; Takashi Iwai ; linux- > i...@vger.kernel.org; linux-kernel@vger.kernel.org; platform-driver- > x...@vger.kernel.org > Subject: Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Pali Rohár
On Tuesday 03 January 2017 19:38:43 Dmitry Torokhov wrote: > On Tue, Jan 03, 2017 at 10:06:41AM +0100, Benjamin Tissoires wrote: > > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > > On Thursday 29 December 2016 22:09:32 Michał Kępień wrote: > > > > > On Thursday 29 December 2016 14:47:19 Mich

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Dmitry Torokhov
On Tue, Jan 03, 2017 at 10:06:41AM +0100, Benjamin Tissoires wrote: > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > On Thursday 29 December 2016 22:09:32 Michał Kępień wrote: > > > > On Thursday 29 December 2016 14:47:19 Michał Kępień wrote: > > > > > > On Thursday 29 December 2016 09:29:36

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Pali Rohár
On Tuesday 03 January 2017 10:06:41 Benjamin Tissoires wrote: > On Dec 29 2016 or thereabouts, Pali Rohár wrote: > > On Thursday 29 December 2016 22:09:32 Michał Kępień wrote: > > > Also, just to make sure we do not overthink this, I understand that > > > not every unit of the models from the white

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2017-01-03 Thread Benjamin Tissoires
On Dec 29 2016 or thereabouts, Pali Rohár wrote: > On Thursday 29 December 2016 22:09:32 Michał Kępień wrote: > > > On Thursday 29 December 2016 14:47:19 Michał Kępień wrote: > > > > > On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > > > > > > Dell platform team told us that some (DMI

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Pali Rohár
On Thursday 29 December 2016 22:09:32 Michał Kępień wrote: > > On Thursday 29 December 2016 14:47:19 Michał Kępień wrote: > > > > On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > > > > > Dell platform team told us that some (DMI whitelisted) Dell > > > > > > Latitude machines have ST

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Michał Kępień
> On Thursday 29 December 2016 14:47:19 Michał Kępień wrote: > > > On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > > > > Dell platform team told us that some (DMI whitelisted) Dell > > > > > Latitude machines have ST microelectronics accelerometer at i2c > > > > > address 0x29. That

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Pali Rohár
On Thursday 29 December 2016 14:47:19 Michał Kępień wrote: > > On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > > > Dell platform team told us that some (DMI whitelisted) Dell > > > > Latitude machines have ST microelectronics accelerometer at i2c > > > > address 0x29. That i2c addres

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Michał Kępień
> On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > > Dell platform team told us that some (DMI whitelisted) Dell > > > Latitude machines have ST microelectronics accelerometer at i2c > > > address 0x29. That i2c address is not specified in DMI or ACPI, so > > > runtime detection witho

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Pali Rohár
On Thursday 29 December 2016 09:29:36 Michał Kępień wrote: > > Dell platform team told us that some (DMI whitelisted) Dell > > Latitude machines have ST microelectronics accelerometer at i2c > > address 0x29. That i2c address is not specified in DMI or ACPI, so > > runtime detection without whiteli

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-29 Thread Michał Kępień
> Dell platform team told us that some (DMI whitelisted) Dell Latitude > machines have ST microelectronics accelerometer at i2c address 0x29. That > i2c address is not specified in DMI or ACPI, so runtime detection without > whitelist which is below is not possible. > > Presence of that ST microel

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-28 Thread Valdis . Kletnieks
On Wed, 28 Dec 2016 15:03:02 +0100, Wolfram Sang said: > > I have absolutely no idea how to you want to achieve calling that > > i2c_new_device() registration > > without kernel patches. > > Documentation/i2c/instantiating-devices lists all supported methods. > Method 4 is userspace instantiation.

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-28 Thread Wolfram Sang
I have absolutely no idea how to you want to achieve calling that i2c_new_device() registration without kernel patches. Documentation/i2c/instantiating-devices lists all supported methods. Method 4 is userspace instantiation.

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-28 Thread Wolfram Sang
On Tue, Dec 27, 2016 at 02:51:01PM +0100, Pali Rohár wrote: On Tuesday 27 December 2016 14:43:49 Wolfram Sang wrote: > > Dell platform team told us that some (DMI whitelisted) Dell > > Latitude machines have ST microelectronics accelerometer at i2c > > address 0x29. That > > i2c address is not sp

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-28 Thread Pali Rohár
On Wednesday 28 December 2016 08:55:18 Andy Shevchenko wrote: > On Wed, Dec 28, 2016 at 12:41 AM, wrote: > > On Wed, 28 Dec 2016 00:15:30 +0200, Andy Shevchenko said: > >> On Tue, Dec 27, 2016 at 3:51 PM, Pali Rohár wrote: > > > >> > I have no idea how to do it (properly) outside of i2c-i801.c f

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-28 Thread Pali Rohár
On Tuesday 27 December 2016 23:15:30 Andy Shevchenko wrote: > On Tue, Dec 27, 2016 at 3:51 PM, Pali Rohár > wrote: > > On Tuesday 27 December 2016 14:43:49 Wolfram Sang wrote: > >> > Dell platform team told us that some (DMI whitelisted) Dell > >> > Latitude machines have ST microelectronics accel

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Andy Shevchenko
On Wed, Dec 28, 2016 at 12:41 AM, wrote: > On Wed, 28 Dec 2016 00:15:30 +0200, Andy Shevchenko said: >> On Tue, Dec 27, 2016 at 3:51 PM, Pali Rohár wrote: > >> > I have no idea how to do it (properly) outside of i2c-i801.c file. >> >> I doubt we need a single line of code for this. See [1] and p

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Valdis . Kletnieks
On Wed, 28 Dec 2016 00:15:30 +0200, Andy Shevchenko said: > On Tue, Dec 27, 2016 at 3:51 PM, Pali Rohár wrote: > > I have no idea how to do it (properly) outside of i2c-i801.c file. > > I doubt we need a single line of code for this. See [1] and perhaps > create an EFI variable with necessary upg

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Andy Shevchenko
On Tue, Dec 27, 2016 at 3:51 PM, Pali Rohár wrote: > On Tuesday 27 December 2016 14:43:49 Wolfram Sang wrote: >> > Dell platform team told us that some (DMI whitelisted) Dell >> > Latitude machines have ST microelectronics accelerometer at i2c >> > address 0x29. That >> > i2c address is not specif

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Pali Rohár
On Tuesday 27 December 2016 14:43:49 Wolfram Sang wrote: > > Dell platform team told us that some (DMI whitelisted) Dell > > Latitude machines have ST microelectronics accelerometer at i2c > > address 0x29. That > > i2c address is not specified in DMI or ACPI, so runtime detection > > without > > w

Re: [PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Wolfram Sang
Dell platform team told us that some (DMI whitelisted) Dell Latitude machines have ST microelectronics accelerometer at i2c address 0x29. That i2c address is not specified in DMI or ACPI, so runtime detection without whitelist which is below is not possible. I'd think this should rather live

[PATCH] i2c: i801: Register optional lis3lv02d i2c device on Dell machines

2016-12-27 Thread Pali Rohár
Dell platform team told us that some (DMI whitelisted) Dell Latitude machines have ST microelectronics accelerometer at i2c address 0x29. That i2c address is not specified in DMI or ACPI, so runtime detection without whitelist which is below is not possible. Presence of that ST microelectronics ac