Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Rafael J. Wysocki
On Tuesday, May 20, 2014 08:25:51 AM Dmitry Torokhov wrote: > On Tue, May 20, 2014 at 02:40:12PM +0200, Rafael J. Wysocki wrote: > > On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: > > > Hi Dudley, > > > > > > On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: > > > > Rely on

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Benson Leung
Hi Dmitry and Rafael, I can provide more insight here since I implemented an early version of this in the atmel and cyapa drivers that are used in the chromeos kernel. On Tue, May 20, 2014 at 8:25 AM, Dmitry Torokhov wrote: > On Tue, May 20, 2014 at 02:40:12PM +0200, Rafael J. Wysocki wrote: >>

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Dmitry Torokhov
On Tue, May 20, 2014 at 02:40:12PM +0200, Rafael J. Wysocki wrote: > On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: > > Hi Dudley, > > > > On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: > > > Rely on EV_SW and SW_LID bits to identify a LID device, and hook > > > up our

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Rafael J. Wysocki
On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: > Hi Dudley, > > On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: > > Rely on EV_SW and SW_LID bits to identify a LID device, and hook > > up our filter to listen for SW_LID events to enable/disable touchpad when > > LID is

RE: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Dudley Du
Torokhov [mailto:dmitry.torok...@gmail.com] > Sent: Tuesday, May 20, 2014 11:43 AM > To: Dudley Du > Cc: Benson Leung; Daniel Kurtz; David Solda; linux-in...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based >

RE: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Dudley Du
[mailto:dmitry.torok...@gmail.com] Sent: Tuesday, May 20, 2014 11:43 AM To: Dudley Du Cc: Benson Leung; Daniel Kurtz; David Solda; linux-in...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state Hi Dudley, On Wed

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Rafael J. Wysocki
On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: Hi Dudley, On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: Rely on EV_SW and SW_LID bits to identify a LID device, and hook up our filter to listen for SW_LID events to enable/disable touchpad when LID is open/closed.

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Dmitry Torokhov
On Tue, May 20, 2014 at 02:40:12PM +0200, Rafael J. Wysocki wrote: On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: Hi Dudley, On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: Rely on EV_SW and SW_LID bits to identify a LID device, and hook up our filter to listen

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Benson Leung
Hi Dmitry and Rafael, I can provide more insight here since I implemented an early version of this in the atmel and cyapa drivers that are used in the chromeos kernel. On Tue, May 20, 2014 at 8:25 AM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: On Tue, May 20, 2014 at 02:40:12PM +0200,

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-20 Thread Rafael J. Wysocki
On Tuesday, May 20, 2014 08:25:51 AM Dmitry Torokhov wrote: On Tue, May 20, 2014 at 02:40:12PM +0200, Rafael J. Wysocki wrote: On Monday, May 19, 2014 08:43:02 PM Dmitry Torokhov wrote: Hi Dudley, On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: Rely on EV_SW and SW_LID

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-19 Thread Dmitry Torokhov
Hi Dudley, On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: > Rely on EV_SW and SW_LID bits to identify a LID device, and hook > up our filter to listen for SW_LID events to enable/disable touchpad when > LID is open/closed. > TEST=test on Chomebooks. This is a policy and it does not

Re: [PATCH 4/6] input: cyapa: enable/disable trackpad device based on LID state

2014-05-19 Thread Dmitry Torokhov
Hi Dudley, On Wed, Apr 16, 2014 at 08:39:34AM +, Dudley Du wrote: Rely on EV_SW and SW_LID bits to identify a LID device, and hook up our filter to listen for SW_LID events to enable/disable touchpad when LID is open/closed. TEST=test on Chomebooks. This is a policy and it does not