Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31;
From:   Armin Wolf <W_Armin@gmx.de>
To:     hdegoede@redhat.com, ilpo.jarvinen@linux.intel.com, corbet@lwn.net
Cc:     Dell.Client.Kernel@dell.com, linux-doc@vger.kernel.org,
        platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 0/5] platform/x86: wmi: Cleanup obsolete features
Date:   Thu,  7 Dec 2023 23:26:18 +0100
Message-Id: <20231207222623.232074-1-W_Armin@gmx.de>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:Dhn/0EMwgHQ=;bxz3HihOqtq6cwc3l0KTYBCsPSW
 wf72mfYMm7eX55hx+eeTQJGxrhz1ktttzeesaMkMgV0Fc7T30HOXyDr8gD7tbbYRdpNpIAOI9
 ZgfbTkxdeQ2awZWbNxUPy1SDJWs6NPARPJr32iYLVRCeqIbBVDoAXl/Ob2XI3zuKywkTRIrMZ
 YLkt+A6zaSRuT/4ovnGGF2b0aPMJRPN3yCJpMOjjKOfS3/BQYlhJ+eKmfdZnfSVaBAXwODct7
 FkMV2oK2Agc4A0VyAaua6NxL+kvXAwqiFYIFtmvfKpWWvbK7Gb7uGMSzfUmVyjdUyxiGRGie5
 GXxnYFXViir9LdXZcITCRZxNO0py8VeSxjKiCGBiFBiLUZhDlvb0sHo0hTwEMLEtSW8Co36ja
 n9o53ZSeo8g0QW0X6sWzEIhKlp6tSJloKfe0rK4ZSscUczneUW3UvXhMlOYwYjFkTircH2FoJ
 W0QUIeRdNFeEQ1qjNrakaTrlafdJVbwR8+J5lXgg1q9wkOvpM+YVdvnWOdoH4zFJUX7ycFPgA
 apQx9uHk2oi0rRmJsynYYxk5NTQhHObLfUvvLVbwNtzbAe32R0U4v4dEzFyhx8/3MBmzGRGtZ
 NIlJCTY6VgkzTb8BtoKPltgVk8Tpqzlm7KQ4ejViik3UKOqA1zkUZmw6h+VPmZtiR58/MmCED
 mw6QWohWpbSenwDpPLPnKp+BCQxEKNddZZQRELDj0W9afSZiLSuk33XfKZIbTL3Blrj6jlhFc
 fcKjCg1fFO986Y38KL0mD3cH2OyBYrVsFr+MaY6kvA/2r/+FDWCgOznQqtiDh6xlPQ+J0KweS
 HZM6UBSLda5ghrGgtaMJG6hrt2a42HlhMaHc73PXRGSxR6xTcOAqcTc45nb1O+T9/hMlvXa5m
 GOsWU0T6aMwWchxlp3dOsTxmk0N+de8VJKw6zdjjk5aO3nFAyMqGPOySPjPTAn0GbX4dnTZ3W
 qVPJWbHzNN9AGpGcSochRoZ6H9k=
Precedence: bulk

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 last patch in contrast adds a short WMI driver development guide
to the WMI subsystem documentation, so that driver developers stop
submitting WMI drivers using the deprecated GUID-based interface.

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.

Armin Wolf (5):
  platform/x86: wmi: Remove debug_dump_wdg module param
  platform/x86: wmi: Remove debug_wmi module param
  platform/x86: dell-smbios-wmi: Stop using WMI chardev
  platform/x86: wmi: Remove chardev interface
  platform/x86: wmi: Add driver development guide

 .../wmi/driver-development-guide.rst          | 126 ++++++++
 Documentation/wmi/index.rst                   |   1 +
 drivers/platform/x86/dell/dell-smbios-wmi.c   | 163 +++++++---
 drivers/platform/x86/wmi.c                    | 285 +-----------------
 include/linux/wmi.h                           |   8 -
 5 files changed, 256 insertions(+), 327 deletions(-)
 create mode 100644 Documentation/wmi/driver-development-guide.rst

=2D-
2.39.2