Re: [PATCH 3/3] input: alps: Reset mouse and ALPS driver immediately after first invalid packet

From: Hans de Goede
Date: Fri Oct 03 2014 - 05:56:08 EST


Hi,

On 10/03/2014 11:43 AM, Pali RohÃr wrote:
> For unknown reasons linux psmouse alps driver sometimes receive totally invalid
> packet sequences on Dell Latitude laptops. According to ALPS HW engineers these
> invalid packets do not come from ALPS devices. So it looks like bug in BIOS and
> EC incorrectly split keyboard and touchpad PS/2 data when laptops are under
> heavy loads (big I/O together with powersave governor, running on battery).
>
> There are sequences of invalid packets (which are dropeed) and some sequences
> which look like valid. But these valid packets cause random trackstick button
> pressing, random cursor moving/jumping and in these condition it is not possible
> to use ALPS device (trackstick+touchpad).
>
> To prevent random button press and random cursor jumps immediately reset ALPS
> device after first invalid packet. This will cause that touchpad and trackstick
> will not respond for one or two seconds and it better then random cursor jumps.
>

This one probably should have:

Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1145954

And you may want to add Bug: tags to the relevant patches for the
launchpad issues too.

While on the topic of tags, once we've agreed upon the return value to use for the
2nd patch, can you please resend with a "Cc: stable@xxxxxxxxxxxxxxx" added to all
3 patches?

> Signed-off-by: Pali RohÃr <pali.rohar@xxxxxxxxx>
> Tested-by: Pali RohÃr <pali.rohar@xxxxxxxxx>

Looks good and seems sensible:

Acked-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Thanks & Regards,

Hans
--
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/