Skip to content

RFC asus notebook fan control

Kast Bernd linux-acpi lkml platform-driver-x86 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 RFC Latest2 patches 8 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2ACPI: activate&export acpi_os_get_physical_addressdrivers/acpi/osl.c+1 -25 replies
2/2asus-wmi: add fan controldrivers/platform/x86/asus-wmi.c+277 -202 replies

Reviews trailers per patch

None of the 2 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchCorentin Chary
SeriesA-b A-b
1/2
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [RFC 0/2] asus notebook fan control

Hello,

This patchset implements a fan control for asus notebooks. It's based on Felipe Contreras' patch
(https://lkml.org/lkml/2013/10/8/800), that was proposed one and a half year ago, but never made it to the kernel, because of the use of "virt_to_phys".
This problem is solved by using the acpi internal function that is activated by the first patch (ACPI_FUTURE_USAGE is removed). The second patch implements the actual fan control functions, with a similar logic than Felipe, but using hwmon instead of the thermal interface and some bugs removed.

This patchset contains the following 2 patches:
0001-ACPI-activate-export-acpi_os_get_physical_address.patch 0002-asus-wmi-add-fan-control.patch

This is my first kernel patch. I tried to respect all style rules and implement it properly. I couldn't find any try to fix the old patch in recent time, thus I wrote this new patch.

I would appreciate any kind of feedback to get this patch upstream.

Kast Bernd (2):
ACPI: activate&export acpi_os_get_physical_address
asus-wmi: add fan control

 drivers/acpi/osl.c              |   3 +-
 drivers/platform/x86/asus-wmi.c | 297 +++++++++++++++++++++++++++++++++++++---
 2 files changed, 278 insertions(+), 22 deletions(-)
Signature · 1 line
2.3.5