Hi Armin, On 2/9/23 22:15, Armin Wolf wrote: > Thanks to bugreport 216655 on bugzilla, the contents of the > fan/thermal sensor buffers could be almost completely decoded. > > The first patch adds support for exposing the sensor values > over a standard hwmon interface, and the second patch enables > asynchronous probing for the driver since the sensor interface > can be quiet slow. > > The patch series was tested on a Dell Inspiron 3505, with the hwmon > patch being tested by multiple people over bugzilla and email. Those > who tested the final version of the hwmon patch are credited with > Tested-by tags. Thanks this looks nice and clean, much appreciate the attention to detail. I've applied the series to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > Changes in v3: > - remove empty lines before break/return > - invalidate buffer cache on resume > Changes in v2: > - remove already merged patches > - add patch enabling asynchronous probing > - cache sensor buffers > - fix compilation on 32 bit > - add missing break statements > - remove C++ style comments > - rework battery hook and hwmon probing > - rework Kconfig dependencies > > Armin Wolf (2): > platform/x86: dell-ddv: Add hwmon support > platform/x86: dell-ddv: Prefer asynchronous probing > > drivers/platform/x86/dell/Kconfig | 8 +- > drivers/platform/x86/dell/dell-wmi-ddv.c | 492 ++++++++++++++++++++++- > 2 files changed, 495 insertions(+), 5 deletions(-) > > -- > 2.30.2 >
