On Sat, Aug 30, 2014 at 04:10:44PM +0200, Ulrik De Bie wrote:
> In the past, no elantech was known with 3 touchpad mouse buttons.
> Fujitsu H730 is the first known elantech with a middle button. This commit
> enables this middle button.
> 
> Reported-by: Stefan Valouch <[email protected]>
> Tested-by: Stefan Valouch <[email protected]>
> Tested-by: Alfredo Gemma <[email protected]>
> Signed-off-by: Ulrik De Bie <[email protected]>
> ---
>  drivers/input/mouse/elantech.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/input/mouse/elantech.c b/drivers/input/mouse/elantech.c
> index 67d56c0..e86bbd7 100644
> --- a/drivers/input/mouse/elantech.c
> +++ b/drivers/input/mouse/elantech.c
> @@ -563,6 +563,7 @@ static void elantech_input_sync_v4(struct psmouse 
> *psmouse)
>       } else {
>               input_report_key(dev, BTN_LEFT, packet[0] & 0x01);
>               input_report_key(dev, BTN_RIGHT, packet[0] & 0x02);
> +             input_report_key(dev, BTN_MIDDLE, packet[0] & 0x04);
>       }
>  
>       input_mt_report_pointer_emulation(dev, true);
> @@ -1150,6 +1151,7 @@ static int elantech_set_input_params(struct psmouse 
> *psmouse)
>       __clear_bit(EV_REL, dev->evbit);
>  
>       __set_bit(BTN_LEFT, dev->keybit);
> +     __set_bit(BTN_MIDDLE, dev->keybit);

No, we should not advertise presence of middle button unconditionally. I
guess we need another DMI, at least for now.

>       __set_bit(BTN_RIGHT, dev->keybit);
>  
>       __set_bit(BTN_TOUCH, dev->keybit);
> -- 
> 2.1.0
> 

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to