[PATCH v2 0/5] platform/x86: wmi: Cleanup obsolete features

From: Armin Wolf
Date: Sun Dec 10 2023 - 15:25:36 EST


This patch series removes three features deemed obsolete:
- the debug_dump_wdg module param:
- suffers from garbled output due to pr_cont()
- functionality is better provided by "fwts wmi"
- the debug_event module param:
- pr_cont() usage
- uses the deprecated GUID-based API
- largely replaced by the ACPI netlink interface
- ioctl interface
- used only by a single driver, no adoption otherwise
- numerous design issues

Since the ioctl interface is actually used by userspace programs,
the only user (the dell-smbios-wmi driver) was modified to implement
the necessary pieces itself so that no regressions are expected.

The series depends on
commit cbf54f37600e ("platform/x86: wmi: Skip blocks with zero instances"),
which is currently in the "fixes" tree.

All patches where tested on a Dell Inspiron 3505 and work without
issues.

Changes since v1:
- add Reviewed-by to patches 1, 2 and 5
- drop patch adding the driver development guide
- rework error handling in dell-smbios-wmi

Armin Wolf (5):
platform/x86: wmi: Remove debug_dump_wdg module param
platform/x86: wmi: Remove debug_event module param
platform/x86: dell-smbios-wmi: Use devm_get_free_pages()
platform/x86: dell-smbios-wmi: Stop using WMI chardev
platform/x86: wmi: Remove chardev interface

drivers/platform/x86/dell/dell-smbios-wmi.c | 173 ++++++++----
drivers/platform/x86/wmi.c | 285 +-------------------
include/linux/wmi.h | 8 -
3 files changed, 132 insertions(+), 334 deletions(-)

--
2.39.2