At Sat, 09 Oct 2010 09:29:44 +0200,
Henrik Rydberg wrote:
>
> Hi Takashi,
>
> > diff --git a/src/eventcomm.c b/src/eventcomm.c
> > index 85dfd09..fc5055b 100644
> > --- a/src/eventcomm.c
> > +++ b/src/eventcomm.c
> > @@ -269,6 +269,12 @@ event_query_axis_ranges(InputInfoPtr pInfo)
> > }
> >
> > xf86Msg(X_PROBED, "%s: buttons:%s\n", pInfo->name, buf);
> > +
> > + /* clickpad device reports only the single left button mask */
> > + if (priv->has_left && !priv->has_right && !priv->has_middle &&
> > !priv->has_double) {
> > + priv->is_clickpad = TRUE;
> > + xf86Msg(X_PROBED, "%s: is Clickpad device\n", local->name);
> > + }
> > }
> > }
>
>
> The variable has_double is true when the kernel reports multiple fingers via
> the
> BTN_*TAP keys. This is the case for the macbook pads, and I would imagine it
> to
> be true also for the clickpad soon. The clicking-on-lower-portion-of-pad is
> interesting for both devices. In other words, perhaps the logic should be
> reiterated?
Right, the logic in *this* patch is incomplete. The patch series are
queued in the historical order, so it doesn't contain any MT things.
The clickpad detection with MT enablement appears in the later patch.
But, the clickpad detection in the later patch is still questionable,
and I personally don't like that. It'd be really better if the
clickpad capability bit is exposed somehow from the driver
directly...
thanks,
Takashi
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
