Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-23 Thread Dmitry Torokhov
On Thu, Jun 23, 2016 at 11:46:55PM +0200, Wolfram Sang wrote: > On Thu, Jun 23, 2016 at 01:55:52PM -0700, Dmitry Torokhov wrote: > > On Thu, Jun 16, 2016 at 08:09:42AM +0200, Wolfram Sang wrote: > > > > > - removed the .resume hook as upstream changed suspend/resume hooks > > > > > and there > > >

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-23 Thread Wolfram Sang
On Thu, Jun 23, 2016 at 01:55:52PM -0700, Dmitry Torokhov wrote: > On Thu, Jun 16, 2016 at 08:09:42AM +0200, Wolfram Sang wrote: > > > > - removed the .resume hook as upstream changed suspend/resume hooks and > > > > there > > > > is no need in the end to re-enable host notify on resume (tested

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-23 Thread Dmitry Torokhov
On Thu, Jun 16, 2016 at 08:09:42AM +0200, Wolfram Sang wrote: > > > - removed the .resume hook as upstream changed suspend/resume hooks and > > > there > > > is no need in the end to re-enable host notify on resume (tested on > > > Lenovo > > > t440 and t450). > > > > Actually, this hook see

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-16 Thread Wolfram Sang
> (i2c-i801) can be carried over through the input tree. So as long as > you don't need to have a new feature without users for a short period > of time, that's fine by me :) That's fine. I have extremly high trust that the user of the feature will be added soon ;) signature.asc Description: P

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-16 Thread Benjamin Tissoires
On Jun 16 2016 or thereabouts, Wolfram Sang wrote: > > > - removed the .resume hook as upstream changed suspend/resume hooks and > > > there > > > is no need in the end to re-enable host notify on resume (tested on > > > Lenovo > > > t440 and t450). > > > > Actually, this hook seemed to be r

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-15 Thread Wolfram Sang
> > - removed the .resume hook as upstream changed suspend/resume hooks and > > there > > is no need in the end to re-enable host notify on resume (tested on Lenovo > > t440 and t450). > > Actually, this hook seemed to be required on the Lenovo T440 (Haswell) > but not on the T450 (Broadwell)

Re: [PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-15 Thread Benjamin Tissoires
On Jun 09 2016 or thereabouts, Benjamin Tissoires wrote: > The i801 chip can handle the Host Notify feature since ICH 3 as mentioned > in > http://www.intel.com/content/dam/doc/datasheet/82801ca-io-controller-hub-3-datasheet.pdf > > Enable the functionality unconditionally and propagate the alert

[PATCH v8 3/4] i2c: i801: add support of Host Notify

2016-06-09 Thread Benjamin Tissoires
The i801 chip can handle the Host Notify feature since ICH 3 as mentioned in http://www.intel.com/content/dam/doc/datasheet/82801ca-io-controller-hub-3-datasheet.pdf Enable the functionality unconditionally and propagate the alert on each notification. With a T440s and a Synaptics touchpad that