Thread (9 messages) 9 messages, 4 authors, 2021-10-11

Re: [PATCH v4 1/2] hwmon: (asus_wmi_ec_sensors) Support B550 Asus WMI.

From: Guenter Roeck <linux@roeck-us.net>
Date: 2021-10-10 23:52:13
Also in: linux-hwmon, lkml, platform-driver-x86

On 10/10/21 3:09 PM, Eugene Shalygin wrote:
quoted
Changes in v4:
  - implement wmi driver instead platform driver.
There are many ASUS boards where the required function is present but
does nothing. With the WMI module alias the driver will be attempted
to load for those boards (and rejected by the _probe() function). Is
it a good thing?
Drivers failing probe with -ENODEV (presumably here because dmi_check_system()
does not find a match) is pretty normal. Board detection with the WMI driver
seems much more straightforward/simple compared the previous version with
the platform driver. Maybe I am missing something, but I don't see a
problem with the WMI approach.

Guenter
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help