Hi,
On 04/09/2014 03:47 AM, Peter Hutterer wrote:
> last_mt_vals_slot is only used in one location and there we can just use
> cur_slot
>
> Signed-off-by: Peter Hutterer <[email protected]>
Series looks good, series is:
Reviewed-by: Hans de Goede <[email protected]>
One remark about patches 5/6 and 6/6, in the mean time we've gotten
more pnp-ids / dmi strings, so you may wish to add those too and
squash them in.
I've been keeping a little text file with up2date info on these
laptops, I've attached it to this mail.
Note the T430u does not have top buttons, it is in the file
because it too advertises wrong min values.
Regards,
Hans
> ---
> src/eventcomm.c | 12 +-----------
> 1 file changed, 1 insertion(+), 11 deletions(-)
>
> diff --git a/src/eventcomm.c b/src/eventcomm.c
> index 858efeb..5030150 100644
> --- a/src/eventcomm.c
> +++ b/src/eventcomm.c
> @@ -110,16 +110,6 @@ EventProtoDataAlloc(int fd)
> return proto_data;
> }
>
> -static int
> -last_mt_vals_slot(const SynapticsPrivate * priv)
> -{
> - struct eventcomm_proto_data *proto_data =
> - (struct eventcomm_proto_data *) priv->proto_data;
> - int value = proto_data->cur_slot;
> -
> - return value < priv->num_slots ? value : -1;
> -}
> -
> static void
> UninitializeTouch(InputInfoPtr pInfo)
> {
> @@ -549,7 +539,7 @@ EventProcessTouchEvent(InputInfoPtr pInfo, struct
> SynapticsHwState *hw,
> proto_data->cur_slot = ev->value;
> }
> else {
> - int slot_index = last_mt_vals_slot(priv);
> + int slot_index = proto_data->cur_slot;
>
> if (slot_index < 0)
> return;
>
dmi pnp-id range Firmware id
Thinkpad T430u LEN0051 {1254, 5702, 1228, 4714}* Touchpad model:
1, fw: 8.1, id: 0x1e2b1, caps: 0xd004a3/0x940300/0x126c00, board id: 2228, fw
id: 1323666
ThinkPad Helix* LEN0033 {1024, 5052, 2258, 4832}
ThinkPad X240* LEN0035 {1232, 5710, 1156, 4696}
ThinkPad S1 Yoga LEN0042 {1232, 5710, 1156, 4696}
ThinkPad T431* LEN0034 {1024, 5112, 2024, 4832}
ThinkPad L440* LEN2004 {1024, 5112, 2024, 4832} Touchpad model:
1, fw: 8.1, id: 0x1e2b1, caps: 0xd001a3/0x940300/0x127c00, board id: 2722, fw
id: 1484859
ThinkPad T440* LEN0036 {1024, 5112, 2024, 4832}
ThinkPad L540 LEN2004 {1024, 5112, 2024, 4832} Touchpad model:
1, fw: 8.1, id: 0x1e2b1, caps: 0xd001a3/0x940300/0x127c00, board id: 2722, fw
id: 1484859
ThinkPad T540* LEN0034 {1024, 5056, 2058, 4832} ** Touchpad model:
1, fw: 8.1, id: 0x1e2b1, caps: 0xd001a3/0x940300/0x127c00, board id: 2722, fw
id: 1484859
ThinkPad X1 Carbon 2nd* LEN0034 {1024, 5112, 2024, 4832}
* We don't have a quirk for this one in the kernel
** Should very likely be the same as all the others with a LEN0034 pnp-id
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
