[PATCH v3] Add Clickpad support
Takashi Iwai
tiwai at suse.de
Sun May 2 10:56:35 PDT 2010
At Thu, 29 Apr 2010 17:03:59 +1000,
Peter Hutterer wrote:
>
> On Fri, Apr 23, 2010 at 05:39:04PM +0200, Takashi Iwai wrote:
> > This patch adds the support for Synaptics Clickpad devices.
> > It requires the change in Linux kernel synaptics input driver, found in
> > https://patchwork.kernel.org/patch/92435/
> > The kernel patch is already included in linux-input GIT tree, which
> > will hit Linus tree sooner or later.
> >
> > When the kernel driver sets only the left-button bit evbit and no
> > multi-finger is possible, Clickpad mode is activated. In this mode,
> > the bottom touch area is used as button emulations. Clicking at the
> > bottom-left, bottom-center and bottom-right zone corresponds to a left,
> > center and right click.
> >
> > Signed-off-by: Takashi Iwai <tiwai at suse.de>
> > ---
> >
> > v2->v3: Fix the mis-detection of Clickpad device with double-tap feature
> > (e.g. MacBook)
> > Fix one forgotten spacing issue Peter suggested
> >
> > src/eventcomm.c | 6 ++++
> > src/synaptics.c | 72 +++++++++++++++++++++++++++++++++++++++++++++++++++-
> > src/synapticsstr.h | 2 +
> > 3 files changed, 79 insertions(+), 1 deletions(-)
> >
> > diff --git a/src/eventcomm.c b/src/eventcomm.c
> > index d00d810..6b44778 100644
> > --- a/src/eventcomm.c
> > +++ b/src/eventcomm.c
> > @@ -252,6 +252,12 @@ event_query_axis_ranges(LocalDevicePtr local)
> > if ((priv->has_triple = (TEST_BIT(BTN_TOOL_TRIPLETAP, keybits) != 0)))
> > strcat(buf, " triple");
> > xf86Msg(X_INFO, "%s: buttons:%s\n", local->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_INFO, "%s: is Clickpad device\n", local->name);
> > + }
> > }
> > }
> >
> > diff --git a/src/synaptics.c b/src/synaptics.c
> > index 091dbe1..554f7a3 100644
> > --- a/src/synaptics.c
> > +++ b/src/synaptics.c
> > @@ -457,6 +457,18 @@ static void set_default_parameters(LocalDevicePtr local)
> > vertResolution = priv->resy;
> > }
> >
> > + /* Clickpad mode -- bottom area is used as buttons */
> > + if (priv->is_clickpad) {
> > + int button_bottom;
> > + /* Clickpad devices usually the button area at the bottom, and
> > + * its size seems ca. 20% of the touchpad height no matter how
> > + * large the pad is.
> > + */
> > + button_bottom = priv->maxy - (abs(priv->maxy - priv->miny) * 20) / 100;
> > + if (button_bottom < b && button_bottom >= t)
> > + b = button_bottom;
> > + }
> > +
> > /* set the parameters */
> > pars->left_edge = xf86SetIntOption(opts, "LeftEdge", l);
> > pars->right_edge = xf86SetIntOption(opts, "RightEdge", r);
> > @@ -2052,6 +2064,59 @@ HandleClickWithFingers(SynapticsParameters *para, struct SynapticsHwState *hw)
> > }
> > }
> >
> > +/* clickpad event handling */
> > +static void
> > +HandleClickpad(LocalDevicePtr local, struct SynapticsHwState *hw, edge_type edge)
> > +{
> > + SynapticsPrivate *priv = (SynapticsPrivate *) (local->private);
> > + SynapticsParameters *para = &priv->synpara;
> > +
> > + if (edge & BOTTOM_EDGE) {
> > + /* button area */
> > + int width = priv->maxx - priv->minx;
> > + int left_button_x, right_button_x;
> > +
> > + /* left and right clickpad button ranges;
> > + * the gap between them is interpreted as a middle-button click
> > + */
> > + left_button_x = width * 2 / 5 + priv->minx;
> > + right_button_x = width * 3 / 5 + priv->minx;
> > +
> > + /* clickpad reports only one button, and we need
> > + * to fake left/right buttons depending on the touch position
> > + */
> > + if (hw->left) { /* clicked? */
> > + hw->left = 0;
> > + if (hw->x < left_button_x)
> > + hw->left = 1;
> > + else if (hw->x > right_button_x)
> > + hw->right = 1;
> > + else
> > + hw->middle = 1;
> > + }
> > +
> > + /* Don't move pointer position in the button area during clicked,
> > + * except for horiz/vert scrolling is enabled.
> > + *
> > + * The synaptics driver tends to be pretty sensitive. This hack
> > + * is to avoid that the pointer moves slightly and misses the
> > + * poistion you aimed to click.
> > + *
> > + * Also, when the pointer movement is reported, the dragging
> > + * (with a sort of multi-touching) doesn't work well, too.
> > + */
> > + if (hw->left || !(para->scroll_edge_horiz ||
> > + ((edge & RIGHT_EDGE) && para->scroll_edge_vert)))
> > + hw->z = 0; /* don't move pointer */
> > +
> > + } else if (hw->left) {
> > + /* dragging */
> > + hw->left = priv->prev_hw.left;
> > + hw->right = priv->prev_hw.right;
> > + hw->middle = priv->prev_hw.middle;
> > + }
> > + priv->prev_hw = *hw;
> > +}
> >
> > /*
> > * React on changes in the hardware state. This function is called every time
> > @@ -2102,6 +2167,12 @@ HandleState(LocalDevicePtr local, struct SynapticsHwState *hw)
> > if (para->touchpad_off == 1)
> > return delay;
> >
> > + edge = edge_detection(priv, hw->x, hw->y);
> > +
> > + /* Clickpad handling for button area */
> > + if (priv->is_clickpad)
> > + HandleClickpad(local, hw, edge);
> > +
> > /* Treat the first two multi buttons as up/down for now. */
> > hw->up |= hw->multi[0];
> > hw->down |= hw->multi[1];
> > @@ -2152,7 +2223,6 @@ HandleState(LocalDevicePtr local, struct SynapticsHwState *hw)
> > hw->multi[2] = hw->multi[3] = FALSE;
> > }
> >
> > - edge = edge_detection(priv, hw->x, hw->y);
> > inside_active_area = is_inside_active_area(priv, hw->x, hw->y);
> >
> > finger = SynapticsDetectFinger(priv, hw);
> > diff --git a/src/synapticsstr.h b/src/synapticsstr.h
> > index bd19c79..05e43d3 100644
> > --- a/src/synapticsstr.h
> > +++ b/src/synapticsstr.h
> > @@ -232,6 +232,8 @@ typedef struct _SynapticsPrivateRec
> > Bool has_double; /* double click detected for this device */
> > Bool has_triple; /* triple click detected for this device */
> > Bool has_pressure; /* device reports pressure */
> > + Bool is_clickpad; /* is Clickpad device (one-button) */
> > + struct SynapticsHwState prev_hw; /* previous h/w state (for clickpad) */
> >
> > enum TouchpadModel model; /* The detected model */
> > } SynapticsPrivate;
> > --
> > 1.7.0.4
>
> can you send me a man page update for this change as well? Just so something
> is in the man page informing the user what ClickPad in the log means. I'll
> squash it on top of this one then.
OK, I'll update in the next week after I back from my vacation.
thanks,
Takashi
More information about the xorg-devel
mailing list