Re: [PATCH] Input: synaptics - add retry mechanism for reconnect

From: James M Leddy
Date: Wed Apr 10 2013 - 09:24:26 EST


I'll have someone test it out. Thanks for the patch.

On 04/10/2013 01:40 AM, Chung-Yih Wang (çåæ) wrote:
> Hi Eric Miao, James Leddy and Daniel Manrique,
>
> If possible, could you give it a try for the patch I have on your device(s)?
> https://patchwork.kernel.org/patch/2156601/
>
> I think we may address similar issue here. The synaptics device may not
> respond to the commands in synaptics_reconnect() for stress test of
> suspend/resume, so my patch is trying to do
> the 'retry' for all requests in the reconnect path if one fails.
>
>
>
> On Wed, Feb 20, 2013 at 2:50 AM, Dmitry Torokhov
> <dmitry.torokhov@xxxxxxxxx <mailto:dmitry.torokhov@xxxxxxxxx>> wrote:
>
> Hi Chung-yih,
>
> On Mon, Feb 18, 2013 at 04:40:21PM +0800, Chung-yih Wang wrote:
> > On the Samsung Series 5 Chromebook, the Synaptics ClickPad will
> sometimes not
> > send an "0xFA" PS/2 ACK in response to a command byte during the
> reconnect
> > sequence following a resume from suspend. From our experiments,
> the failure can
> > happen during any byte of any of the commands in the reconnect batch.
> >
> > This failure results in a timeout in the psmouse driver. In
> addition, the
> > ClickPad will often also respond to the next PS/2 command byte
> with an "0xFE"
> > PS/2 RESEND response.
>
> Since you control the firmware of the device can you figure out under
> what condition the touchpad does not acknowledge the commands sent to it
> and fix it there?
>
> Thanks.
>
> --
> Dmitry
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/