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

[Dudley Du]

Hi Dmitry,

Is there any update on this patch?

Thanks,
Dudley

> -----Original Message-----
> From: Jeremiah Mahler [mailto:jmmahler@xxxxxxxxx]
> Sent: 2016?1?15? 13:34
> To: Dudley Du
> Cc: dmitry.torokhov@xxxxxxxxx; eugenesan@xxxxxxxxx; bleung@xxxxxxxxxx;
> linux-input@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxx>
> > ---
> >  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@xxxxxxxxx>
>
> --
> - 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.