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: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 23 Feb 2016 01:43:16 +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

Hi Dmitry,

Is there any update on this patch?

Thanks,
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.

Powered by blists - more mailing lists