[PATCH libinput 0/8] Add libinput_device_suspend() to disable devices

[Bill Spitzak]

On 08/24/2014 05:44 PM, Peter Hutterer wrote:

> this is *not* about palm detection. palm detection is already always enabled
> where possible and with the parameters sensible for that specific touchpad.
>
> This option is for the optional disabling of the touchpad when an external
> mouse is connected. And that option is only necessary because we can't
> guarantee that palm detection works 100% of the time.

I think I am still failing to explain what I am asking for.

1. Disable: good, for exactly the reason you state, so that you can do 
thinks like turn it off when the mouse is plugged in without that being 
part of the touchpad driver.

2. "smart" disable/enable: good. You certainly want to provide the 
ability to use any intelligence written into the driver.

3. Enable: THIS IS THE ONE I WANT TO REMOVE. It seems like there should 
be no reason to bypass what was put in the driver to intelligently 
disable the touchpad.

My recommendation is to rename "smart disable" to "enable" and eliminate 
the proposed "enable" setting.

If in fact there is some problem with the driver so that you want to 
adjust the smart settings, something more complicated (and probably more 
driver-specific) than a boolean is going to be needed.