Received-SPF: pass (google.com: domain of linux-kernel+bounces-122272-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: Armin Wolf <W_Armin@gmx.de>
To: hdegoede@redhat.com,
	ilpo.jarvinen@linux.intel.com
Cc: corbet@lwn.net,
	linux-doc@vger.kernel.org,
	platform-driver-x86@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 4/4] platform/x86: wmi: Add driver development guide
Date: Thu, 28 Mar 2024 02:23:36 +0100
Message-Id: <20240328012336.145612-4-W_Armin@gmx.de>
In-Reply-To: <20240328012336.145612-1-W_Armin@gmx.de>
References: <20240328012336.145612-1-W_Armin@gmx.de>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
UI-OutboundReport: notjunk:1;M01:P0:BYwcXPbrfJs=;0QW8kRtKbBNqknne2ZHJAFkIMqA
 Tx0D7UsVUmbRh9E0Xi10v8XGvrHL3sqlTMFwvxHiXB50ypwoF17TugnUXlSAuygxdziahb4nD
 sRmlsgVfMiGSETZ5G3p62FKpmEBfUId3mo2o/SwJFGW2aYB3FeZqpdKHLuBZ1I5Id9jIocBb/
 rm3YWwsRNNhkzPBTvAXrXZU6ABS4hV8ieIvSzl/lPACHmMgNOJ06CaHWJY8cQog8eonWVJ8LF
 1/EqSKgqNzGMT3imSUWwV1ScWzbCxnRP5Yal0JxjrzAEPvt714YFqO8qFeA4B2804O1ZjwXfw
 h6n9E2h6N0W++Ouu2f8M3Udc6MbfN+/EOngxWk3cqvKoUJEP6l6azIvxFnDqGVFQ3Zxtw4rWm
 ptdc4TMoHBCj0HloqCGpZ/jY5lPEG3nXvRhwwSaE5FfppiUlCIGAqYJ2tOYf5EEwBiXm2RnSI
 v2LlJz+g7SPnmGWCGSSXWdMZxxjGcUSFVjMOniXZ67X5iev3pcZSIMd6AWDca2JyI3Cslw7Rh
 ciFYuOwqlZ+XEN99z3Z75VsgmJNzr1v5JVNCqKj36+7+l0WxF03atkpnjau6Bc5/w4Mwc5xLZ
 46tBCYjEjENgFMN4bBefHLnIrGMgnKbCtUUJwq+JhB4R7XzDOGQbbIlAtnwmuaqKhperkRWh5
 Kts1nEcgZjtY/3GZrvsM6Gfv7IblwqJqz16l2G34dBUiEqrYXz1rAUWluxtkCNx/HLi95Qwxq
 CpV6EzihByErFpxsfUyh22jzOMOgeIOXqa7nanEI855v8Xn+IwuDLTm2hcPgiHEkWyn9+0IeX
 odMq5wPh+dZ5DlhvzSROjkXho31xSkCuvm4ZINA+0Mf1o=

Since 2010, an LWN article covering WMI drivers exists:

	https://lwn.net/Articles/391230/

Since the introduction of the modern bus-based interface
and other userspace tooling (bmfdec, lswmi, ...), this
article is outdated and causes people to still submit new
WMI drivers using the deprecated GUID-based interface.
Fix this by adding a short guide on how to develop WMI drivers
using the modern bus-based interface.

Signed-off-by: Armin Wolf <W_Armin@gmx.de>
=2D--
 .../wmi/driver-development-guide.rst          | 173 ++++++++++++++++++
 Documentation/wmi/index.rst                   |   1 +
 2 files changed, 174 insertions(+)
 create mode 100644 Documentation/wmi/driver-development-guide.rst

diff --git a/Documentation/wmi/driver-development-guide.rst b/Documentatio=
n/wmi/driver-development-guide.rst
new file mode 100644
index 000000000000..9a1b0a8490bb
=2D-- /dev/null
+++ b/Documentation/wmi/driver-development-guide.rst
@@ -0,0 +1,173 @@
+.. SPDX-License-Identifier: GPL-2.0-or-later
+
+=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
+WMI driver development guide
+=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
+
+The WMI subsystem provides a rich driver API for implementing WMI drivers=
,
+documented at Documentation/driver-api/wmi.rst. This document will serve
+as an introductory guide for WMI driver writers using this API. It is sup=
posed
+to be an successor to the original `LWN article <https://lwn.net/Articles=
/391230/>`_
+which deals with WMI drivers using the deprecated GUID-based WMI interfac=
e.
+
+Obtaining WMI device information
+--------------------------------
+
+Before developing an WMI driver, information about the WMI device in ques=
tion
+must be obtained. The `lswmi <https://pypi.org/project/lswmi>`_ utility c=
an be
+used to display detailed WMI device information using the following comma=
nd:
+
+::
+
+  lswmi -V
+
+The resulting output will contain information about all WMI devices avail=
able on
+a given machine, plus some extra information.
+
+In order to find out more about the interface used to communicate with a =
WMI device,
+the `bmfdec <https://github.com/pali/bmfdec>`_ utilities can be used to d=
ecode
+the Binary MOF (Managed Object Format) information used to describe WMI d=
evices.
+The ``wmi-bmof`` driver exposes this information to userspace, see
+Documentation/wmi/devices/wmi-bmof.rst.
+
+In order to retrieve the decoded Binary MOF information, use the followin=
g command (requires root):
+
+::
+
+  ./bmf2mof /sys/bus/wmi/devices/05901221-D566-11D1-B2F0-00A0C9062910[-X]=
/bmof
+
+Sometimes, looking at the disassembled ACPI tables used to describe the W=
MI device
+helps in understanding how the WMI device is supposed to work. The path o=
f the ACPI
+method associated with a given WMI device can be retrieved using the ``ls=
wmi`` utility
+as mentioned above.
+
+Basic WMI driver structure
+--------------------------
+
+The basic WMI driver is build around the struct wmi_driver, which is then=
 bound
+to matching WMI devices using a struct wmi_device_id table:
+
+::
+
+  static const struct wmi_device_id foo_id_table[] =3D {
+         { "936DA01F-9ABD-4D9D-80C7-02AF85C822A8", NULL },
+         { }
+  };
+  MODULE_DEVICE_TABLE(wmi, foo_id_table);
+
+  static struct wmi_driver foo_driver =3D {
+        .driver =3D {
+                .name =3D "foo",
+                .probe_type =3D PROBE_PREFER_ASYNCHRONOUS,        /* reco=
mmended */
+                .pm =3D pm_sleep_ptr(&foo_dev_pm_ops),            /* opti=
onal */
+        },
+        .id_table =3D foo_id_table,
+        .probe =3D foo_probe,
+        .remove =3D foo_remove,         /* optional, devres is preferred =
*/
+        .notify =3D foo_notify,         /* optional, for event handling *=
/
+        .no_notify_data =3D true,       /* optional, enables events conta=
ining no additional data */
+        .no_singleton =3D true,         /* required for new WMI drivers *=
/
+  };
+  module_wmi_driver(foo_driver);
+
+The probe() callback is called when the WMI driver is bound to a matching=
 WMI device. Allocating
+driver-specific data structures and initialising interfaces to other kern=
el subsystems should
+normally be done in this function.
+
+The remove() callback is then called when the WMI driver is unbound from =
a WMI device. In order
+to unregister interfaces to other kernel subsystems and release resources=
, devres should be used.
+This simplifies error handling during probe and often allows to omit this=
 callback entirely, see
+Documentation/driver-api/driver-model/devres.rst for details.
+
+Please note that new WMI drivers are required to be able to be instantiat=
ed multiple times,
+and are forbidden from using any deprecated GUID-based WMI functions. Thi=
s means that the
+WMI driver should be prepared for the scenario that multiple matching WMI=
 devices are present
+on a given machine.
+
+Because of this, WMI drivers should use the state container design patter=
n as described in
+Documentation/driver-api/driver-model/design-patterns.rst.
+
+WMI method drivers
+------------------
+
+WMI drivers can call WMI device methods using wmidev_evaluate_method(), t=
he
+structure of the ACPI buffer passed to this function is device-specific a=
nd usually
+needs some tinkering to get right. Looking at the ACPI tables containing =
the WMI
+device usually helps here. The method id and instance number passed to th=
is function
+are also device-specific, looking at the decoded Binary MOF is usually en=
ough to
+find the right values.
+
+The maximum instance number can be retrieved during runtime using wmidev_=
instance_count().
+
+Take a look at drivers/platform/x86/inspur_platform_profile.c for an exam=
ple WMI method driver.
+
+WMI data block drivers
+----------------------
+
+WMI drivers can query WMI device data blocks using wmidev_block_query(), =
the
+structure of the returned ACPI object is again device-specific. Some WMI =
devices
+also allow for setting data blocks using wmidev_block_set().
+
+The maximum instance number can also be retrieved using wmidev_instance_c=
ount().
+
+Take a look at drivers/platform/x86/intel/wmi/sbl-fw-update.c for an exam=
ple
+WMI data block driver.
+
+WMI event drivers
+-----------------
+
+WMI drivers can receive WMI events by providing the notify() callback ins=
ide the struct wmi_driver.
+The WMI subsystem will then take care of setting up the WMI event accordi=
ngly. Please note that
+the structure of the ACPI object passed to this callback is device-specif=
ic, and freeing the
+ACPI object is being done by the WMI subsystem, not the driver.
+
+The WMI driver core will take care that the notify() callback will only b=
e called after
+the probe() callback has been called, and that no events are being receiv=
ed by the driver
+right before and after calling its remove() callback.
+
+However WMI driver developers should be aware that multiple WMI events ca=
n be received concurrently,
+so any locking (if necessary) needs to be provided by the WMI driver itse=
lf.
+
+In order to be able to receive WMI events containg no additional event da=
ta,
+the ``no_notify_data`` flag inside struct wmi_driver should be set to ``t=
rue``.
+
+Take a look at drivers/platform/x86/xiaomi-wmi.c for an example WMI event=
 driver.
+
+Handling multiple WMI devices at once
+-------------------------------------
+
+There are many cases of firmware vendors using multiple WMI devices to co=
ntrol different aspects
+of a single physical device. This can make developing WMI drivers complic=
ated, as those drivers
+might need to communicate with each other to present a unified interface =
to userspace.
+
+On such case involves a WMI event device which needs to talk to a WMI dat=
a block device or WMI
+method device upon receiving an WMI event. In such a case, two WMI driver=
s should be developed,
+one for the WMI event device and one for the other WMI device.
+
+The WMI event device driver has only one purpose: to receive WMI events, =
validate any additional
+event data and invoke a notifier chain. The other WMI driver adds itself =
to this notifier chain
+during probing and thus gets notified every time a WMI event is received.=
 This WMI driver might
+then process the event further for example by using an input device.
+
+For other WMI device constellations, similar mechanisms can be used.
+
+Things to avoid
+---------------
+
+When developing WMI drivers, there are a couple of things which should be=
 avoided:
+
+- usage of the deprecated GUID-based WMI interface which uses GUIDs inste=
ad of WMI device structs
+- bypassing of the WMI subsystem when talking to WMI devices
+- WMI drivers which cannot be instantiated multiple times.
+
+Many older WMI drivers violate one or more points from this list. The rea=
son for
+this is that the WMI subsystem evolved significantly over the last two de=
cades,
+so there is a lot of legacy cruft inside older WMI drivers.
+
+New WMI drivers are also required to conform to the linux kernel coding s=
tyle as specified in
+Documentation/process/coding-style.rst. The checkpatch utility can catch =
many common coding style
+violations, you can invoke it with the following command:
+
+::
+
+  ./scripts/checkpatch.pl --strict <path to driver file>
diff --git a/Documentation/wmi/index.rst b/Documentation/wmi/index.rst
index 537cff188e14..fec4b6ae97b3 100644
=2D-- a/Documentation/wmi/index.rst
+++ b/Documentation/wmi/index.rst
@@ -8,6 +8,7 @@ WMI Subsystem
    :maxdepth: 1

    acpi-interface
+   driver-development-guide
    devices/index

 .. only::  subproject and html
=2D-
2.39.2