[PATCH libinput] udev: add the Dell Lattitude E5420 to the wobbly touchpads

[Hans de Goede]

Hi,

On 20-05-16 12:28, Hans de Goede wrote:
> Hi,
>
> On 20-05-16 02:29, Peter Hutterer wrote:
>> https://bugzilla.redhat.com/show_bug.cgi?id=1336084
>>
>> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
>> ---
>>  udev/90-libinput-model-quirks.hwdb | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/udev/90-libinput-model-quirks.hwdb b/udev/90-libinput-model-quirks.hwdb
>> index 147fec2..22179bb 100644
>> --- a/udev/90-libinput-model-quirks.hwdb
>> +++ b/udev/90-libinput-model-quirks.hwdb
>> @@ -30,6 +30,9 @@ libinput:name:*AlpsPS/2 ALPS DualPoint TouchPad:fwversion:310
>>  libinput:name:*AlpsPS/2 ALPS GlidePoint:fwversion:310
>>   LIBINPUT_MODEL_ALPS_RUSHMORE=1
>>
>> +libinput:name:*AlpsPS/2 ALPS DualPoint TouchPad:dmi:*svnDellInc.:pnLatitudeE5420*
>> + LIBINPUT_MODEL_WOBBLY_TOUCHPAD=1
>> +
>>  ##########################################
>>  # Apple
>>  ##########################################
>
> It is likely a better idea to just mark all alps fwversion 300 devices as wobbly,
> talking about that I see that we only use the RUSHMORE flag to do WOBBLY handling,
> so we might drop that and just set WOBBLY on them instead.

To clarify to the best of my knowledge all alps touchpads of a given version are
created equally, they do not have fw difference per laptop model (or so I believe),
so this likely applies to all of them, and doing this base don the fwversion
avoids needing it for LatitudeE??20 and likely others.

Regards,

Hans