mbox series

[v2,0/2] asus-wmi: Modify fn+f5 and support GPU fan

Message ID 20220826004210.356534-1-luke@ljones.dev (mailing list archive)
Headers show
Series asus-wmi: Modify fn+f5 and support GPU fan | expand

Message

Luke Jones Aug. 26, 2022, 12:42 a.m. UTC
Some TUF laptops have both fan_boost and thermal_throttle, and some in that
group may not actually have an effect on one or the other - this patch series
adjusts behaviour to try both. There doesn't appear to be any consistency
to check for so far.

Some laptops can also control the GPU fan between auto and boost.

Changelog:
- V2
+ Rebase on review-hans

Luke D. Jones (2):
  asus-wmi: Modify behaviour of Fn+F5 fan key
  asus-wmi: Support the GPU fan on TUF laptops

 drivers/platform/x86/asus-nb-wmi.c         |  1 +
 drivers/platform/x86/asus-wmi.c            | 82 ++++++++++++++++++++--
 include/linux/platform_data/x86/asus-wmi.h |  1 +
 3 files changed, 78 insertions(+), 6 deletions(-)

Comments

Hans de Goede Aug. 26, 2022, 9:36 a.m. UTC | #1
Hi,

On 8/26/22 02:42, Luke D. Jones wrote:
> Some TUF laptops have both fan_boost and thermal_throttle, and some in that
> group may not actually have an effect on one or the other - this patch series
> adjusts behaviour to try both. There doesn't appear to be any consistency
> to check for so far.
> 
> Some laptops can also control the GPU fan between auto and boost.
> 
> Changelog:
> - V2
> + Rebase on review-hans
> 
> Luke D. Jones (2):
>   asus-wmi: Modify behaviour of Fn+F5 fan key
>   asus-wmi: Support the GPU fan on TUF laptops

Thank you for your patch-series, 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

Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.

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