INPUT_WISTRON_BTNS should be disabled when ACPI enabled
Created attachment 6751 [details] disabel INPUT_WISTRON_BTNS when ACPI enabled
Quoting lkml mail: Yu, Luming wrote: >> I just tested module wistron_btn on one Acer Aspire laptop after >> adding one dmi entry. The wistron_btn found BIOS interfaces. >> One visible error is the bluetooth light won't turn on upon >> stroking bluetooth button. >> Without wistron_btn module, the bluetooth light works. >> with acpi enabled, I didn't try acpi disabled) >> >> wistron_btn polls a cmos address to detect hotkey event. It >> is not necessary, because there do have ACPI interrupt triggered upon >> hotkeys. There are many different laptops using similar interfaces. It is a mess :( If your laptop provides the hotkey events via ACPI, simply don't use wistron_btns. >> So, my suggestion is to disable this module when ACPI enabled. I have a laptop that needs this module (hotkeys are not supported via ACPI), but supports ACPI. Mirek
Created attachment 6778 [details] acpidump of Fujitsu-Siemens Amilo Pro V200
since ACPI is a run-time mode, doesn't this need to be a run-time check, rather than a build-time check?
Len, any idea of how to do the run-time check?
So the problem is that there may be some potential conflicts between ACPI and wistron driver. The laptops listed in the wistron dmi_ids table may also route the hotkey events to ACPI. This can't be detected by ACPI or wistron driver. Can we add a warning message when wistron driver is loaded? Just tell the users to send an email with dmidecode attached if wistron driver breaks the laptop (some keys doesn't work like described in comment #2) so that we can remove the dmi entry for this box.
Created attachment 13433 [details] patch-add-warning-message-when-loading-wistron-driver
I'm sorry, I don't see the point of warning users like this - is it at all likely that such a conflict will occur? Was there ever a conflict between ACPI and the DMI mappings in winstron_btns? Or are you going to add such a warning message to every non-ACPI driver that provides similar functionality to a separate ACPI driver?
Well, I did some tests on an acer aspire 1520. All the hotkey events are routed to the i8042 driver although the wireless and bluetooth are recognized as unknown key press. But the wireless and bluetooth lights are turned on while pressing them. IMO, it works well with atkbd driver, and the wireless and bluetooth hotkey are probably handled by the bios, so there is no need to support this latop in wistron driver. I didn't find any conflicts between ACPI and the current DMI mappings in winstron_btns. So you're right and please ignore the patch in comment #7. :) I think we can close this bug and mark it as INVALID. Thanks.