linux-kernel - RE: [PATCH] input: cyapa: fix the report events may lost issue during set power mode

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <SN1PR0601MB19033C37A19EED52A9AF517BD1CD0@SN1PR0601MB1903.namprd06.prod.outlook.com>
Date:	Fri, 15 Jan 2016 06:44:40 +0000
From:	Dudley Du <dudl@...ress.com>
To:	Jeremiah Mahler <jmmahler@...il.com>
CC:	"dmitry.torokhov@...il.com" <dmitry.torokhov@...il.com>,
	"eugenesan@...il.com" <eugenesan@...il.com>,
	"bleung@...gle.com" <bleung@...gle.com>,
	"linux-input@...r.kernel.org" <linux-input@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] input: cyapa: fix the report events may lost issue during
 set power mode

Jeremiah,

Thanks for your help to do test and verification.

Regards,
Dudley

> -----Original Message-----
> From: Jeremiah Mahler [mailto:jmmahler@...il.com]
> Sent: 2016?1?15? 13:34
> To: Dudley Du
> Cc: dmitry.torokhov@...il.com; eugenesan@...il.com; bleung@...gle.com;
> linux-input@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] input: cyapa: fix the report events may lost issue during set
> power mode
>
> Dudley,
>
> On Thu, Jan 14, 2016 at 01:40:33PM +0800, Dudley Du wrote:
> > When driver set the power mode to change the scan rate in auto suspend
> > process, some events may be lost because
> > 1) for gen3 trackpad, the driver must msleep() some time to avoid issuing
> > next command encounter error;
> > 2) for gen5 and later trackpads, the queue dumping function will simply
> > ignore the events when waiting for the set power mode command response.
> > so, when auto resume process will be executed, the set power mode command
> > will be issued can cause the events may be lost.
> > The solution is kept polling and report those valid events when the set
> > power mode command is in progress.
> >
> > TEST=test on Acer C720P Chromebook
> >
> > Signed-off-by: Dudley Du <dudl@...ress.com>
> > ---
> >  drivers/input/mouse/cyapa.c      |  22 ++++----
> >  drivers/input/mouse/cyapa.h      |  14 ++++-
> >  drivers/input/mouse/cyapa_gen3.c | 108
> ++++++++++++++++++++++++++++-----------
> >  drivers/input/mouse/cyapa_gen5.c |  99
> +++++++++++++++++++++++++++++------
> >  drivers/input/mouse/cyapa_gen6.c |   4 +-
> >  5 files changed, 188 insertions(+), 59 deletions(-)
> >
> [...]
>
> I tested this on my Acer C720 and it appears to work properly.
>
> Tested-by: Jeremiah Mahler <jmmahler@...il.com>
>
> --
> - Jeremiah Mahler

This message and any attachments may contain Cypress (or its subsidiaries) confidential information. If it has been received in error, please advise the sender and immediately delete this message.