[PATCH v2 0/8] platform/x86: wmi: Rework WMI device enabling

Armin Wolf posted 8 patches 10 months ago
Documentation/wmi/acpi-interface.rst          |   3 +
.../wmi/driver-development-guide.rst          |   4 +
drivers/hwmon/hp-wmi-sensors.c                |   4 +-
drivers/platform/x86/hp/hp-bioscfg/bioscfg.c  |  13 +-
drivers/platform/x86/think-lmi.c              |  51 +++----
drivers/platform/x86/think-lmi.h              |   2 +
drivers/platform/x86/wmi.c                    | 141 ++++++++++--------
7 files changed, 116 insertions(+), 102 deletions(-)
[PATCH v2 0/8] platform/x86: wmi: Rework WMI device enabling
Posted by Armin Wolf 10 months ago
This patch series reworks how WMI devices are enabled and disabled
to improve the compatibility with various firmware implementations.

The first three patches make sure that no WMI driver using the WMI bus
infrastructure is using the deprecated GUID-based API to access the
underlying WMI device.

The fourth patch is a unrelated cleanup patch.

The last four patches finally rework the WMI device enabling inside
the WMI core and update the documentation.

The WMI core patches have been tested on a Dell Inspiron 3505, but
the remaining patches are mostly compile-tested only.

Changes since v2:
- add Reviewed-by, Acked-by and Tested-by tags
- use devres to disable the WMI device

Armin Wolf (8):
  hwmon: (hp-wmi-sensors) Use the WMI bus API when accessing sensors
  platform/x86: think-lmi: Use ACPI object when extracting strings
  platform/x86: think-lmi: Use WMI bus API when accessing BIOS settings
  platform/x86: hp-bioscfg: Use wmi_instance_count()
  platform/x86: wmi: Use devres to disable the WMI device
  platform/x86: wmi: Rework WCxx/WExx ACPI method handling
  platform/x86: wmi: Call WCxx methods when setting data blocks
  platform/x86: wmi: Update documentation regarding the GUID-based API

 Documentation/wmi/acpi-interface.rst          |   3 +
 .../wmi/driver-development-guide.rst          |   4 +
 drivers/hwmon/hp-wmi-sensors.c                |   4 +-
 drivers/platform/x86/hp/hp-bioscfg/bioscfg.c  |  13 +-
 drivers/platform/x86/think-lmi.c              |  51 +++----
 drivers/platform/x86/think-lmi.h              |   2 +
 drivers/platform/x86/wmi.c                    | 141 ++++++++++--------
 7 files changed, 116 insertions(+), 102 deletions(-)

--
2.39.5
Re: [PATCH v2 0/8] platform/x86: wmi: Rework WMI device enabling
Posted by Ilpo Järvinen 9 months, 4 weeks ago
On Sun, 16 Feb 2025 20:32:43 +0100, Armin Wolf wrote:

> This patch series reworks how WMI devices are enabled and disabled
> to improve the compatibility with various firmware implementations.
> 
> The first three patches make sure that no WMI driver using the WMI bus
> infrastructure is using the deprecated GUID-based API to access the
> underlying WMI device.
> 
> [...]


Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/8] hwmon: (hp-wmi-sensors) Use the WMI bus API when accessing sensors
      commit: 27cc2914e495d314d19c7c1f74687bb28bc292e8
[2/8] platform/x86: think-lmi: Use ACPI object when extracting strings
      commit: 82d3af6b30ac2c6031428dd506eab9fd416cb825
[3/8] platform/x86: think-lmi: Use WMI bus API when accessing BIOS settings
      commit: 126a53a94a3fb3171d35f3c87820aafbafa08430
[4/8] platform/x86: hp-bioscfg: Use wmi_instance_count()
      commit: 0fcc3162e3fbe4ca9347b2f42313dbf077207bb1
[5/8] platform/x86: wmi: Use devres to disable the WMI device
      commit: b6b566908cd75bd99eb0e4b396c89704588261b3
[6/8] platform/x86: wmi: Rework WCxx/WExx ACPI method handling
      commit: 656f0961d126f5d1f0bcc62fa8bb7598d8f89207
[7/8] platform/x86: wmi: Call WCxx methods when setting data blocks
      commit: 08c9f4029007325e0bbd4fdda05650b03e43f071
[8/8] platform/x86: wmi: Update documentation regarding the GUID-based API
      commit: 8d5316c6c6633c18c4ebeb2dd038933549d404de

--
 i.