[PATCH xf86-input-synaptics v2 2/2] Turn down the default sensitivity for unibody MacBook touchpads

[Peter Hutterer]

On Wed, Jun 06, 2012 at 03:34:40PM -0700, Chase Douglas wrote:
> The touchpads are too sensitive by default. A touch is activated when a
> finger hovers a millimeter off the device. This change turns down the
> default sensitivity so that accidental touchpad interactions are
> lessened and pointer interaction accuracy is increased.
> 
> Signed-off-by: Chase Douglas <chase.douglas at canonical.com>
> ---
>  src/synaptics.c |   12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
> 
> diff --git a/src/synaptics.c b/src/synaptics.c
> index 68d35e8..2144684 100644
> --- a/src/synaptics.c
> +++ b/src/synaptics.c
> @@ -319,17 +319,23 @@ static void
>  calculate_tap_hysteresis(SynapticsPrivate * priv, int range,
>                           int *fingerLow, int *fingerHigh)
>  {
> -    if (priv->model == MODEL_ELANTECH) {
> +    switch (priv->model) {
> +    case MODEL_ELANTECH:
>          /* All Elantech touchpads don't need the Z filtering to get the
>           * number of fingers correctly. See Documentation/elantech.txt
>           * in the kernel.
>           */
>          *fingerLow = priv->minp + 1;
>          *fingerHigh = priv->minp + 1;
> -    }
> -    else {
> +        break;
> +    case MODEL_UNIBODY_MACBOOK:
> +        *fingerLow = 70;
> +        *fingerHigh = 75;
> +        break;
> +    default:
>          *fingerLow = priv->minp + range * (25.0 / 256);
>          *fingerHigh = priv->minp + range * (30.0 / 256);
> +        break;
>      }
>  }
>  
> -- 
> 1.7.9.5

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net> though as discussed
on IRC I'd prefer a product ID range mapping instead of the strcmp(name) for
the other patch.
 
Cheers,
  Peter