Re: [PATCH v3 1/4] platform/x86: Add lenovo-wmi drivers Documentation

From: Derek John Clark
Date: Mon Mar 10 2025 - 17:36:45 EST

Next message: Paul Moore: "Re: [PATCH] MAINTAINERS: add Serge Hallyn as maintainer for creds"
Previous message: Vladimir Zapolskiy: "Re: [PATCH v4 1/4] dt-bindings: media: Add qcom,x1e80100-camss"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Fri, Mar 7, 2025 at 1:51 PM Armin Wolf <W_Armin@xxxxxx> wrote:
>
> Am 25.02.25 um 22:59 schrieb Derek J. Clark:
>
> > Adds documentation for all new lenovo-wmi drivers.
> >
> > v3:
> > - Split documentation into multiple files, one for each parent
> > driver for the Gamezone and Other Mode WMI interfaces.
> > - Add MOF data for all parent and child interfaces.
> > - Remove lenovo-wmi-camera.c driver from v2 documentation.
> > v2:
> > - Update description of Custom Profile to include the need to manually
> > set it.
> > - Remove all references to Legion hardware.
> > - Add section for lenovo-wmi-camera.c driver as it follows the same
> > naming convention.
> >
> > Signed-off-by: Derek J. Clark <derekjohn.clark@xxxxxxxxx>
> > ---
> > .../wmi/devices/lenovo-wmi-gamezone.rst | 355 ++++++++++++++++++
> > .../wmi/devices/lenovo-wmi-other-method.rst | 142 +++++++
> > MAINTAINERS | 7 +
> > 3 files changed, 504 insertions(+)
> > create mode 100644 Documentation/wmi/devices/lenovo-wmi-gamezone.rst
> > create mode 100644 Documentation/wmi/devices/lenovo-wmi-other-method.rst
> >
> > diff --git a/Documentation/wmi/devices/lenovo-wmi-gamezone.rst b/Documentation/wmi/devices/lenovo-wmi-gamezone.rst
> > new file mode 100644
> > index 000000000000..1dc281fdd99d
> > --- /dev/null
> > +++ b/Documentation/wmi/devices/lenovo-wmi-gamezone.rst
> > @@ -0,0 +1,355 @@
> > +.. SPDX-License-Identifier: GPL-2.0-or-later
> > +==========================================================
> > +Lenovo WMI Interface Gamezone Driver (lenovo-wmi-gamezone)
> > +==========================================================
> > +
> > +Introduction
> > +============
> > +The Lenovo WMI gamezone interface is broken up into multiple GUIDs,
> > +The priamry "Gamezone" GUID provides advanced features such as fan
> > +profiles and overclocking. It is paired with multiple event GUIDs
> > +and data block GUIDs that provide context for the various methods.
> > +
> > +Gamezone Data
> > +-------------
> > +WMI GUID "887B54E3-DDDC-4B2C-8B88-68A26A8835D0"
>
> Maybe formatting the GUID as monospace text would look better here.
>
Acked.

> > +
> > +The Gamezone Data WMI interface provides platform-profile and fan curve
> > +settings for devices that fall under the "Gaming Series" of Lenovo devices.
> > +It uses a notifier chain to inform other Lenovo WMI interface drivers of the
> > +current platform profile when it changes.
> > +
> > +The following platform profiles are supported:
> > + - low-power
> > + - balanced
> > + - balanced-performance
> > + - performance
> > + - custom
> > +
> > +Balanced-Perfornance and Performance Profiles
> > +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > +Some newer Lenovo "Gaming Series" laptops have an "Extreme Mode" profile
> > +enabled in their BIOS. For these devices, the performance platform profile
> > +will correspond to the BIOS Extreme Mode, while the balanced-performance
> > +platform profile will correspond to the BIOS Performance mode. For legacy
> > +devices, the performance platform prfile will correspond with the BIOS
> > +Performance mode.
> > +
> > +Custom Profile
> > +~~~~~~~~~~~~~~
> > +The custom profile represents a hardware mode on Lenovo devices that enables
> > +user modifications to Package Power Tracking (PPT) and fan curve settings.
> > +When an attribute exposed by the Other Mode WMI interface is to be modified,
> > +the Gamezone driver must first be switched to the "custom" profile manually,
> > +or the setting will have no effect. If another profile is set from the list
> > +of supported profiles, the BIOS will override any user PPT settings when
> > +switching to that profile.
> > +
> > +Gamezone Thermal Mode Event
> > +---------------------------
> > +WMI GUID "D320289E-8FEA-41E0-86F9-911D83151B5F"
> > +
>
> Same as above.
>
> > +The Gamezone Thermal Mode Event interface notifies the system when the platform
> > +profile has changed, either through the hardware event (Fn+Q for laptops or
> > +Legion + Y for Go Series), or through the Gamezone WMI interface.
> > +
> > +
> > +WMI interface description
> > +=========================
> > +
> > +The WMI interface description can be decoded from the embedded binary MOF (bmof)
> > +data using the `bmfdec <https://github.com/pali/bmfdec>`_ utility:
> > +
> > +::
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("LENOVO_GAMEZONE_DATA class"),
> > + guid("{887B54E3-DDDC-4B2C-8B88-68A26A8835D0}")]
> > + class LENOVO_GAMEZONE_DATA {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
>
> The MOF definition mixes tabs and spaces and is not properly indented. Please fix this.

Will be fixed, ty.

> > +
> > + [WmiMethodId(4), Implemented, Description("Is SupportGpu OverClock")]
> > + void IsSupportGpuOC([out, Description("Is SupportGpu OverClock")] uint32 Data);
> > +
> > + [WmiMethodId(11), Implemented, Description("Get AslCode Version")]
> > + void GetVersion ([out, Description("AslCode version")] UINT32 Data);
> > +
> > + [WmiMethodId(12), Implemented, Description("Fan cooling capability")]
> > + void IsSupportFanCooling([out, Description("Fan cooling capability")] UINT32 Data);
> > +
> > + [WmiMethodId(13), Implemented, Description("Set Fan cooling on/off")]
> > + void SetFanCooling ([in, Description("Set Fan cooling on/off")] UINT32 Data);
> > +
> > + [WmiMethodId(14), Implemented, Description("cpu oc capability")]
> > + void IsSupportCpuOC ([out, Description("cpu oc capability")] UINT32 Data);
> > +
> > + [WmiMethodId(15), Implemented, Description("bios has overclock capability")]
> > + void IsBIOSSupportOC ([out, Description("bios has overclock capability")] UINT32 Data);
> > +
> > + [WmiMethodId(16), Implemented, Description("enable or disable overclock in bios")]
> > + void SetBIOSOC ([in, Description("enable or disable overclock in bios")] UINT32 Data);
> > +
> > + [WmiMethodId(18), Implemented, Description("Get CPU temperature")]
> > + void GetCPUTemp ([out, Description("Get CPU temperature")] UINT32 Data);
> > +
> > + [WmiMethodId(19), Implemented, Description("Get GPU temperature")]
> > + void GetGPUTemp ([out, Description("Get GPU temperature")] UINT32 Data);
> > +
> > + [WmiMethodId(20), Implemented, Description("Get Fan cooling on/off status")]
> > + void GetFanCoolingStatus ([out, Description("Get Fan cooling on/off status")] UINT32 Data);
> > +
> > + [WmiMethodId(21), Implemented, Description("EC support disable windows key capability")]
> > + void IsSupportDisableWinKey ([out, Description("EC support disable windows key capability")] UINT32 Data);
> > +
> > + [WmiMethodId(22), Implemented, Description("Set windows key disable/enable")]
> > + void SetWinKeyStatus ([in, Description("Set windows key disable/enable")] UINT32 Data);
> > +
> > + [WmiMethodId(23), Implemented, Description("Get windows key disable/enable status")]
> > + void GetWinKeyStatus ([out, Description("Get windows key disable/enable status")] UINT32 Data);
> > +
> > + [WmiMethodId(24), Implemented, Description("EC support disable touchpad capability")]
> > + void IsSupportDisableTP ([out, Description("EC support disable touchpad capability")] UINT32 Data);
> > +
> > + [WmiMethodId(25), Implemented, Description("Set touchpad disable/enable")]
> > + void SetTPStatus ([in, Description("Set touchpad disable/enable")] UINT32 Data);
> > +
> > + [WmiMethodId(26), Implemented, Description("Get touchpad disable/enable status")]
> > + void GetTPStatus ([out, Description("Get touchpad disable/enable status")] UINT32 Data);
> > +
> > + [WmiMethodId(30), emented, Description("Get Keyboard feature list")]
> > + void GetKeyboardfeaturelist ([out, Description("Get Keyboard feature list")] UINT32 Data);
> > +
> > + [WmiMethodId(31), emented, Description("Get Memory OC Information")]
> > + void GetMemoryOCInfo ([out, Description("Get Memory OC Information")] UINT32 Data);
> > +
> > + [WmiMethodId(32), emented, Description("Water Cooling feature capability")]
> > + void IsSupportWaterCooling ([out, Description("Water Cooling feature capability")] UINT32 Data);
> > +
> > + [WmiMethodId(33), emented, Description("Set Water Cooling status")]
> > + void SetWaterCoolingStatus ([in, Description("Set Water Cooling status")] UINT32 Data);
> > +
> > + [WmiMethodId(34), emented, Description("Get Water Cooling status")]
> > + void GetWaterCoolingStatus ([out, Description("Get Water Cooling status")] UINT32 Data);
> > +
> > + [WmiMethodId(35), emented, Description("Lighting feature capability")]
> > + void IsSupportLightingFeature ([out, Description("Lighting feature capability")] UINT32 Data);
> > +
> > + [WmiMethodId(36), emented, Description("Set keyboard light off or on to max")]
> > + void SetKeyboardLight ([in, Description("keyboard light off or on switch")] UINT32 Data);
> > +
> > + [WmiMethodId(37), emented, Description("Get keyboard light on/off status")]
> > + void GetKeyboardLight ([out, Description("Get keyboard light on/off status")] UINT32 Data);
> > +
> > + [WmiMethodId(38), emented, Description("Get Macrokey scan code")]
> > + void GetMacrokeyScancode ([in, Description("Macrokey index")] UINT32 idx, [out, Description("Scan code")] UINT32 scancode);
> > +
> > + [WmiMethodId(39), emented, Description("Get Macrokey count")]
> > + void GetMacrokeyCount ([out, Description("Macrokey count")] UINT32 Data);
> > +
> > + [WmiMethodId(40), emented, Description("Support G-Sync feature")]
> > + void IsSupportGSync ([out, Description("Support G-Sync feature")] UINT32 Data);
> > +
> > + [WmiMethodId(41), emented, Description("Get G-Sync Statust")]
> > + void GetGSyncStatus ([out, Description("Get G-Sync Statust")] UINT32 Data);
> > +
> > + [WmiMethodId(42), emented, Description("Set G-Sync Statust")]
> > + void SetGSyncStatus ([in, Description("Set G-Sync Statust")] UINT32 Data);
> > +
> > + [WmiMethodId(43), emented, Description("Support Smart Fan feature")]
> > + void IsSupportSmartFan ([out, Description("Support Smart Fan feature")] UINT32 Data);
> > +
> > + [WmiMethodId(44), emented, Description("Set Smart Fan Mode")]
> > + void SetSmartFanMode ([in, Description("Set Smart Fan Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(45), emented, Description("Get Smart Fan Mode")]
> > + void GetSmartFanMode ([out, Description("Get Smart Fan Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(46), emented, Description("Get Smart Fan Setting Mode")]
> > + void GetSmartFanSetting ([out, Description("Get Smart Setting Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(47), emented, Description("Get Power Charge Mode")]
> > + void GetPowerChargeMode ([out, Description("Get Power Charge Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(48), emented, Description("Get Gaming Product Info")]
> > + void GetProductInfo ([out, Description("Get Gaming Product Info")] UINT32 Data);
> > +
> > + [WmiMethodId(49), emented, Description("Over Drive feature capability")]
> > + void IsSupportOD ([out, Description("Over Drive feature capability")] UINT32 Data);
> > +
> > + [WmiMethodId(50), emented, Description("Get Over Drive status")]
> > + void GetODStatus ([out, Description("Get Over Drive status")] UINT32 Data);
> > +
> > + [WmiMethodId(51), emented, Description("Set Over Drive status")]
> > + void SetODStatus ([in, Description("Set Over Drive status")] UINT32 Data);
> > +
> > + [WmiMethodId(52), emented, Description("Set Light Control Owner")]
> > + void SetLightControlOwner ([in, Description("Set Light Control Owner")] UINT32 Data);
> > +
> > + [WmiMethodId(53), emented, Description("Set DDS Control Owner")]
> > + void SetDDSControlOwner ([in, Description("Set DDS Control Owner")] UINT32 Data);
> > +
> > + [WmiMethodId(54), emented, Description("Get the flag of restore OC value")]
> > + void IsRestoreOCValue ([in, Description("Clean this flag")] UINT32 idx, [out, Description("Restore oc value flag")] UINT32 Data);
> > +
> > + [WmiMethodId(55), emented, Description("Get Real Thremal Mode")]
> > + void GetThermalMode ([out, Description("Real Thremal Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(56), emented, Description("Get the OC switch status in BIOS")]
> > + void GetBIOSOCMode ([out, Description("OC Mode")] UINT32 Data);
> > +
> > + [WmiMethodId(59), emented, Description("Get hardware info support version")]
> > + void GetHardwareInfoSupportVersion ([out, Description("version")] UINT32 Data);
> > +
> > + [WmiMethodId(60), emented, Description("Get Cpu core 0 max frequency")]
> > + void GetCpuFrequency ([out, Description("frequency")] UINT32 Data);
> > +
> > + [WmiMethodId(62), emented, Description("Check the Adapter type fit for OC")]
> > + void IsACFitForOC ([out, Description("AC check result")] UINT32 Data);
> > +
> > + [WmiMethodId(63), emented, Description("Is support IGPU mode")]
> > + void IsSupportIGPUMode ([out, Description("IGPU modes")] UINT32 Data);
> > +
> > + [WmiMethodId(64), emented, Description("Get IGPU Mode Status")]
> > + void GetIGPUModeStatus([out, Description("IGPU Mode Status")] UINT32 Data);
> > +
> > + [WmiMethodId(65), emented, Description("Set IGPU Mode")]
> > + void SetIGPUModeStatus([in, Description("IGPU Mode")] UINT32 mode, [out, Description("return code")] UINT32 Data);
> > +
> > + [WmiMethodId(66), emented, Description("Notify DGPU Status")]
> > + void NotifyDGPUStatus([in, Description("DGPU status")] UINT32 status, [out, Description("return code")] UINT32 Data);
> > +
> > + [WmiMethodId(67), emented, Description("Is changed Y log")]
> > + void IsChangedYLog([out, Description("Is changed Y Log")] UINT32 Data);
> > +
> > + [WmiMethodId(68), emented, Description("Get DGPU Hardwawre ID")]
> > + void GetDGPUHWId([out, Description("Get DGPU Hardware ID")] string Data);
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Definition of CPU OC parameter list"),
> > + guid("{B7F3CA0A-ACDC-42D2-9217-77C6C628FBD2}")]
> > + class LENOVO_GAMEZONE_CPU_OC_DATA {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("OC tune id.")]
> > + uint32 Tuneid;
> > +
> > + [WmiDataId(2), read, Description("Default value.")]
> > + uint32 DefaultValue;
> > +
> > + [WmiDataId(3), read, Description("OC Value.")]
> > + uint32 OCValue;
> > +
> > + [WmiDataId(4), read, Description("Min Value.")]
> > + uint32 MinValue;
> > +
> > + [WmiDataId(5), read, Description("Max Value.")]
> > + uint32 MaxValue;
> > +
> > + [WmiDataId(6), read, Description("Scale Value.")]
> > + uint32 ScaleValue;
> > +
> > + [WmiDataId(7), read, Description("OC Order id.")]
> > + uint32 OCOrderid;
> > +
> > + [WmiDataId(8), read, Description("NON-OC Order id.")]
> > + uint32 NOCOrderid;
> > +
> > + [WmiDataId(9), read, Description("Delay time in ms.")]
> > + uint32 Interval;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Definition of GPU OC parameter list"),
> > + guid("{887B54E2-DDDC-4B2C-8B88-68A26A8835D0}")]
> > + class LENOVO_GAMEZONE_GPU_OC_DATA {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("P-State ID.")]
> > + uint32 PStateID;
> > +
> > + [WmiDataId(2), read, Description("CLOCK ID.")]
> > + uint32 ClockID;
> > +
> > + [WmiDataId(3), read, Description("Default value.")]
> > + uint32 defaultvalue;
> > +
> > + [WmiDataId(4), read, Description("OC Offset freqency.")]
> > + uint32 OCOffsetFreq;
> > +
> > + [WmiDataId(5), read, Description("OC Min offset value.")]
> > + uint32 OCMinOffset;
> > +
> > + [WmiDataId(6), read, Description("OC Max offset value.")]
> > + uint32 OCMaxOffset;
> > +
> > + [WmiDataId(7), read, Description("OC Offset Scale.")]
> > + uint32 OCOffsetScale;
> > +
> > + [WmiDataId(8), read, Description("OC Order id.")]
> > + uint32 OCOrderid;
> > +
> > + [WmiDataId(9), read, Description("NON-OC Order id.")]
> > + uint32 NOCOrderid;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Fancooling finish event"),
> > + guid("{BC72A435-E8C1-4275-B3E2-D8B8074ABA59}")]
> > + class LENOVO_GAMEZONE_FAN_COOLING_EVENT: WMIEvent {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("Fancooling clean finish event")]
> > + uint32 EventId;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Smart Fan mode change event"),
> > + guid("{D320289E-8FEA-41E0-86F9-611D83151B5F}")]
> > + class LENOVO_GAMEZONE_SMART_FAN_MODE_EVENT: WMIEvent {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("Smart Fan Mode change event")]
> > + uint32 mode;
> > +
> > + [WmiDataId(2), read, Description("version of FN+Q")]
> > + uint32 version;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Smart Fan setting mode change event"),
> > + guid("{D320289E-8FEA-41E1-86F9-611D83151B5F}")]
> > + class LENOVO_GAMEZONE_SMART_FAN_SETTING_EVENT: WMIEvent {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("Smart Fan Setting mode change event")]
> > + uint32 mode;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("POWER CHARGE MODE Change EVENT"),
> > + guid("{D320289E-8FEA-41E0-86F9-711D83151B5F}")]
> > + class LENOVO_GAMEZONE_POWER_CHARGE_MODE_EVENT: WMIEvent {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("POWER CHARGE MODE Change EVENT")]
> > + uint32 mode;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("Thermal Mode Real Mode change event"),
> > + guid("{D320289E-8FEA-41E0-86F9-911D83151B5F}")]
> > + class LENOVO_GAMEZONE_THERMAL_MODE_EVENT: WMIEvent {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description("Thermal Mode Real Mode")]
> > + uint32 mode;
> > + };
> > diff --git a/Documentation/wmi/devices/lenovo-wmi-other-method.rst b/Documentation/wmi/devices/lenovo-wmi-other-method.rst
> > new file mode 100644
> > index 000000000000..d555f1ed9588
> > --- /dev/null
> > +++ b/Documentation/wmi/devices/lenovo-wmi-other-method.rst
> > @@ -0,0 +1,142 @@
> > +.. SPDX-License-Identifier: GPL-2.0-or-later
> > +===========================================================
> > +Lenovo WMI Interface Other Mode Driver (lenovo-wmi-other)
> > +===========================================================
> > +
> > +Introduction
> > +============
> > +Lenovo WMI Other Mode interface is broken up into multiple GUIDs,
> > +The primary Other Mode interface provides advanced power tuning features
> > +such as Package Power Tracking (PPT). It is paired with multiple data block
> > +GUIDs that provide context for the various methods.
> > +
> > +
> > +Other Mode
> > +------------
> > +WMI GUID "DC2A8805-3A8C-41BA-A6F7-092E0089CD3B"
>
> The suggestion with the monospace formatting applies here too.
>
> > +
> > +The Other Mode WMI interface uses the fw_attributes class to expose
> > +various WMI attributes provided by the interface in the sysfs. This enables
> > +CPU and GPU power limit tuning as well as various other attributes for
> > +devices that fall under the "Gaming Series" of Lenovo devices. Each
> > +attribute exposed by the Other Mode interface has corresponding
> > +capability data blocks which allow the driver to probe details about the
> > +attribute. Each attibute has multiple pages, one for each of the platform
> > +profiles managed by the Gamezone interface. Attributes are exposed in sysfs
> > +under the following path:
> > +
> > +::
> > +/sys/class/firmware-attributes/lenovo-wmi-other/attributes/<attribute>/
> > +
> > +LENOVO_CAPABILITY_DATA_01
> > +~~~~~~~~~~~~~~~~~~~~~~~~~
> > +WMI GUID "7A8F5407-CB67-4D6E-B547-39B3BE018154"
>
> Same as above.
>
> > +
> > +The LENOVO_CAPABILITY_DATA_01 interface provides information on various
> > +power limits of integrated CPU and GPU components.
> > +
> > +The following attributes are supported:
> > + - ppt_pl1_spl: Platform Profile Tracking Sustained Power Limit
> > + - ppt_pl2_sppt: Platform Profile Tracking Slow Package Power Tracking
> > + - ppt_pl3_fppt: Platform Profile Tracking Fast Package Power Tracking
> > +
> > +Each attribute has the following properties:
> > + - current_value
> > + - default_value
> > + - display_name
> > + - max_value
> > + - min_value
> > + - scalar_increment
> > + - type
> > +
> > +
> > +WMI interface description
> > +=========================
> > +
> > +The WMI interface description can be decoded from the embedded binary MOF (bmof)
> > +data using the `bmfdec <https://github.com/pali/bmfdec>`_ utility:
> > +
> > +::
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("LENOVO_OTHER_METHOD class"),
> > + guid("{dc2a8805-3a8c-41ba-a6f7-092e0089cd3b}")]
> > + class LENOVO_OTHER_METHOD {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiMethodId(17), Implemented, Description("Get Feature Value ")]
> > + void GetFeatureValue([in] uint32 IDs, [out] uint32 value);
> > +
> > + [WmiMethodId(18), Implemented, Description("Set Feature Value ")]
> > + void SetFeatureValue([in] uint32 IDs, [in] uint32 value);
> > +
> > + [WmiMethodId(19), Implemented, Description("Get Data By Command ")]
> > + void GetDataByCommand([in] uint32 IDs, [in] uint32 Command, [out] uint32 DataSize, [out, WmiSizeIs("DataSize")] uint32 Data[]);
> > +
> > + [WmiMethodId(99), Implemented, Description("Get Data By Package for TAC")]
> > + void GetDataByPackage([in, Max(40)] uint8 Input[], [out] uint32 DataSize, [out, WmiSizeIs("DataSize")] uint8 Data[]);
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("LENOVO CAPABILITY DATA 00"),
> > + guid("{362a3afe-3d96-4665-8530-96dad5bb300e}")]
> > + class LENOVO_CAPABILITY_DATA_00 {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description(" IDs.")]
> > + uint32 IDs;
> > +
> > + [WmiDataId(2), read, Description("Capability.")]
> > + uint32 Capability;
> > +
> > + [WmiDataId(3), read, Description("Capability Default Value.")]
> > + uint32 DefaultValue;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("LENOVO CAPABILITY DATA 01"),
> > + guid("{7a8f5407-cb67-4d6e-b547-39b3be018154}")]
> > + class LENOVO_CAPABILITY_DATA_01 {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description(" IDs.")]
> > + uint32 IDs;
> > +
> > + [WmiDataId(2), read, Description("Capability.")]
> > + uint32 Capability;
> > +
> > + [WmiDataId(3), read, Description("Default Value.")]
> > + uint32 DefaultValue;
> > +
> > + [WmiDataId(4), read, Description("Step.")]
> > + uint32 Step;
> > +
> > + [WmiDataId(5), read, Description("Minimum Value.")]
> > + uint32 MinValue;
> > +
> > + [WmiDataId(6), read, Description("Maximum Value.")]
> > + uint32 MaxValue;
> > + };
> > +
> > + [WMI, Dynamic, Provider("WmiProv"), Locale("MS\\0x409"),
> > + Description("LENOVO CAPABILITY DATA 02"),
> > + guid("{bbf1f790-6c2f-422b-bc8c-4e7369c7f6ab}")]
> > + class LENOVO_CAPABILITY_DATA_02 {
> > + [key, read] string InstanceName;
> > + [read] boolean Active;
> > +
> > + [WmiDataId(1), read, Description(" IDs.")]
> > + uint32 IDs;
> > +
> > + [WmiDataId(2), read, Description("Capability.")]
> > + uint32 Capability;
> > +
> > + [WmiDataId(3), read, Description("Data Size.")]
> > + uint32 DataSize;
> > +
> > + [WmiDataId(4), read, Description("Default Value"), WmiSizeIs("DataSize")]
> > + uint8 DefaultValue[];
> > + };
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 6f78d6bcbc7b..e20c32b3c480 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -13151,6 +13151,13 @@ S: Maintained
> > W: http://legousb.sourceforge.net/
> > F: drivers/usb/misc/legousbtower.c
> >
> > +LENOVO WMI drivers
> > +M: Derek J. Clark <derekjohn.clark@xxxxxxxxx>
> > +L: platform-driver-x86@xxxxxxxxxxxxxxx
> > +S: Maintained
> > +F: Documentation/wmi/devices/lenovo-wmi-gamezone.rst
> > +F: Documentation/wmi/devices/lenovo-wmi-other.rst
>
> Just a side note: when rebasing your work on the current for-next branch you will have to resolve a minor merge
> conflict here due to the new Lenovo hotkey utilities driver.

Okay, thanks for the heads up.

Cheers,
- Derek

> Apart from the formatting issues the remaining patch looks good to me.
>
> Thanks,
> Armin Wolf
>
> > +
> > LETSKETCH HID TABLET DRIVER
> > M: Hans de Goede <hdegoede@xxxxxxxxxx>
> > L: linux-input@xxxxxxxxxxxxxxx

Next message: Paul Moore: "Re: [PATCH] MAINTAINERS: add Serge Hallyn as maintainer for creds"
Previous message: Vladimir Zapolskiy: "Re: [PATCH v4 1/4] dt-bindings: media: Add qcom,x1e80100-camss"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]