platform/x86: ideapad-laptop: Make touchpad_ctrl_via_ec a module option
authorHans de Goede <hdegoede@redhat.com>
Thu, 17 Nov 2022 11:02:44 +0000 (12:02 +0100)
committerHans de Goede <hdegoede@redhat.com>
Mon, 21 Nov 2022 10:16:49 +0000 (11:16 +0100)
commit301e0d766d7018a579063f539837f0f1fa1b2948
treedce129cd07300d5e2f147ced5574a53533d529ec
parenta10ba160d427e78ffa2ab15a86cacaec291fa58a
platform/x86: ideapad-laptop: Make touchpad_ctrl_via_ec a module option

Remove the ACPI-HID + DMI-id deny-lists for touchpad_ctrl_via_ec and
instead make it a module option which defaults to false.

The touchpad sysfs attribute allowing directly writing VPCCMD_W_TOUCHPAD
from userspace has been leading to a lot of bug-reports / patches adding
both ACPI HID + dmi-id based deny-lists for it which then need to be
expanded all the time going forward leading to a high maintenance load.

At the same time the touchpad sysfs attribute is not a standard Linux
userspace API. So it is not used in standard desktop-enviroments, instead
it is only used in the following 2 rare circumstances:

1. Ideapad specific control-panel like applets
2. Custom scripts written by users

For 1. these applets need to already deal with the touchpad sysfs attr
sometimes not being there because of the existing deny lists so hiding
it be default should not cause an issue; and most desktop environments
already have a touchpad-disable option in their native control-panel,
so having an ideapad specific toggle for this is not necessary.

For 2. since these users are already customizing their systems they
can add the module option if they want to keep using the touchpad
sysfs attribute.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Tested-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
Tested-by: Maxim Mikityanskiy <maxtram95@gmail.com>
Link: https://lore.kernel.org/r/20221117110244.67811-7-hdegoede@redhat.com
drivers/platform/x86/ideapad-laptop.c