* [PATCH 0/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops @ 2026-06-19 22:09 Marco Scardovi 2026-06-19 22:09 ` [PATCH 1/1] " Marco Scardovi 0 siblings, 1 reply; 6+ messages in thread From: Marco Scardovi @ 2026-06-19 22:09 UTC (permalink / raw) To: moravec Cc: hansg, ilpo.jarvinen, krishna.chomal108, eliadevito, emreleno, rafael, linux-kernel, platform-driver-x86, regressions, regressions, Marco Scardovi Hi everyone, I was looking for [1] and come with a quick fix for it. Please review it and let me know if you have any questions/suggestions. TL;DR: The driver was blindly registering all 4 profiles (quiet, cool, balanced, performance) without checking which ones were actually supported by the BIOS. This patch fixes a platform_profile regression on generic (non-Omen/non-Victus) HP laptops, where switching or querying power profiles fails with the error: "platform_profile: Failed to get profile for handler hp-wmi" We resolve this by: 1. Dynamically probing supported thermal profiles at boot time. 2. Initializing and updating a cached active_platform_profile. 3. Falling back to the cached profile if the BIOS query fails or returns an unmapped value. This has been verified to build cleanly and resolves the reported Bugzilla issues 221569 and 220008. [1] https://lore.kernel.org/platform-driver-x86/a3b137df-1b21-4460-b003-58c5ca2d59d4@ukf.sk/ Marco Scardovi (1): platform/x86: hp-wmi: fix platform profile issues on generic HP laptops drivers/platform/x86/hp/hp-wmi.c | 53 +++++++++++++++++++++++++++++--- 1 file changed, 48 insertions(+), 5 deletions(-) -- 2.54.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops 2026-06-19 22:09 [PATCH 0/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops Marco Scardovi @ 2026-06-19 22:09 ` Marco Scardovi 2026-06-20 9:36 ` Milan Oravec 2026-06-23 15:52 ` Krishna Chomal 0 siblings, 2 replies; 6+ messages in thread From: Marco Scardovi @ 2026-06-19 22:09 UTC (permalink / raw) To: moravec Cc: hansg, ilpo.jarvinen, krishna.chomal108, eliadevito, emreleno, rafael, linux-kernel, platform-driver-x86, regressions, regressions, Marco Scardovi On generic (non-Omen/non-Victus) HP laptops supported by the hp-wmi driver, the platform_profile sysfs operations could fail, leading to 'platform_profile: Failed to get profile for handler hp-wmi' errors and preventing userspace power profile management from working correctly. The driver was blindly registering all 4 profiles (quiet, cool, balanced, performance) without checking which ones were actually supported by the BIOS. Furthermore, when userspace switched profiles, hp_wmi_platform_profile_get() was called, which queried the BIOS using thermal_profile_get(). If the BIOS query returned an error or returned a value not in the recognized cases, the get call failed, throwing an error. Fix this by: 1. Dynamically probing which thermal profiles are supported by the BIOS during driver registration by temporarily setting each profile and checking the return code, then restoring the original profile. 2. Initializing and updating active_platform_profile to cache the last set/boot profile. 3. Falling back to the cached active_platform_profile in hp_wmi_platform_profile_get() if the BIOS query fails or returns an invalid/unmapped value, rather than returning an error. Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220008 Closes: https://bugzilla.kernel.org/show_bug.cgi?id=221569 Closes: https://lore.kernel.org/platform-driver-x86/a3b137df-1b21-4460-b003-58c5ca2d59d4@ukf.sk/ Fixes: 4296f679ca50 ("platform/x86: hp-wmi: add platform profile support") Assisted-by: Antigravity:gemini-3.5-flash Signed-off-by: Marco Scardovi <scardracs@disroot.org> --- drivers/platform/x86/hp/hp-wmi.c | 53 +++++++++++++++++++++++++++++--- 1 file changed, 48 insertions(+), 5 deletions(-) diff --git a/drivers/platform/x86/hp/hp-wmi.c b/drivers/platform/x86/hp/hp-wmi.c index 8ba286ed8721..93df269445eb 100644 --- a/drivers/platform/x86/hp/hp-wmi.c +++ b/drivers/platform/x86/hp/hp-wmi.c @@ -1662,8 +1662,11 @@ static int hp_wmi_platform_profile_get(struct device *dev, int tp; tp = thermal_profile_get(); - if (tp < 0) - return tp; + if (tp < 0) { + guard(mutex)(&active_platform_profile_lock); + *profile = active_platform_profile; + return 0; + } switch (tp) { case HP_THERMAL_PROFILE_PERFORMANCE: @@ -1679,7 +1682,9 @@ static int hp_wmi_platform_profile_get(struct device *dev, *profile = PLATFORM_PROFILE_QUIET; break; default: - return -EINVAL; + guard(mutex)(&active_platform_profile_lock); + *profile = active_platform_profile; + break; } return 0; @@ -1707,10 +1712,14 @@ static int hp_wmi_platform_profile_set(struct device *dev, return -EOPNOTSUPP; } + guard(mutex)(&active_platform_profile_lock); + err = thermal_profile_set(tp); if (err) return err; + active_platform_profile = profile; + return 0; } @@ -2017,8 +2026,23 @@ static int hp_wmi_platform_profile_probe(void *drvdata, unsigned long *choices) /* Adding an equivalent to HP Omen software ECO mode: */ set_bit(PLATFORM_PROFILE_LOW_POWER, choices); } else { - set_bit(PLATFORM_PROFILE_QUIET, choices); - set_bit(PLATFORM_PROFILE_COOL, choices); + int current_tp = thermal_profile_get(); + + if (current_tp < 0) + return current_tp; + + if (thermal_profile_set(HP_THERMAL_PROFILE_QUIET) == 0) + set_bit(PLATFORM_PROFILE_QUIET, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_COOL) == 0) + set_bit(PLATFORM_PROFILE_COOL, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_DEFAULT) == 0) + set_bit(PLATFORM_PROFILE_BALANCED, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_PERFORMANCE) == 0) + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + + /* Restore the original thermal profile */ + thermal_profile_set(current_tp); + return 0; } set_bit(PLATFORM_PROFILE_BALANCED, choices); @@ -2263,6 +2287,25 @@ static int thermal_profile_setup(struct platform_device *device) if (err) return err; + /* Initialize active_platform_profile */ + switch (tp) { + case HP_THERMAL_PROFILE_PERFORMANCE: + active_platform_profile = PLATFORM_PROFILE_PERFORMANCE; + break; + case HP_THERMAL_PROFILE_DEFAULT: + active_platform_profile = PLATFORM_PROFILE_BALANCED; + break; + case HP_THERMAL_PROFILE_COOL: + active_platform_profile = PLATFORM_PROFILE_COOL; + break; + case HP_THERMAL_PROFILE_QUIET: + active_platform_profile = PLATFORM_PROFILE_QUIET; + break; + default: + active_platform_profile = PLATFORM_PROFILE_BALANCED; + break; + } + ops = &hp_wmi_platform_profile_ops; } -- 2.54.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops 2026-06-19 22:09 ` [PATCH 1/1] " Marco Scardovi @ 2026-06-20 9:36 ` Milan Oravec 2026-06-23 15:52 ` Krishna Chomal 1 sibling, 0 replies; 6+ messages in thread From: Milan Oravec @ 2026-06-20 9:36 UTC (permalink / raw) To: Marco Scardovi Cc: hansg, ilpo.jarvinen, krishna.chomal108, eliadevito, emreleno, rafael, linux-kernel, platform-driver-x86, regressions, regressions [-- Attachment #1: Type: text/plain, Size: 5427 bytes --] Hello, thank you for work! This patch resolves kernel errors (platform_profile: Failed to get profile for handler hp-wmi) in the dmesg log on my system (HP OmniBook X Flip) during power profile switching. Best regards, Milan Oravec CIKT UKF Nitra Mobil/Signal: +421907670270 email: moravec@ukf.sk -------------------------------------------------------------------- Tuto spravu nepreveril ziadny antivirus - nebolo treba... On 6/20/26 00:09, Marco Scardovi wrote: > On generic (non-Omen/non-Victus) HP laptops supported by the > hp-wmi driver, the platform_profile sysfs operations could fail, > leading to 'platform_profile: Failed to get profile for handler hp-wmi' > errors and preventing userspace power profile management from working > correctly. > > The driver was blindly registering all 4 profiles (quiet, cool, > balanced, performance) without checking which ones were actually > supported by the BIOS. Furthermore, when userspace switched profiles, > hp_wmi_platform_profile_get() was called, which queried the BIOS > using thermal_profile_get(). If the BIOS query returned an error or > returned a value not in the recognized cases, the get call failed, > throwing an error. > > Fix this by: > 1. Dynamically probing which thermal profiles are supported by the BIOS > during driver registration by temporarily setting each profile and > checking the return code, then restoring the original profile. > 2. Initializing and updating active_platform_profile to cache the > last set/boot profile. > 3. Falling back to the cached active_platform_profile in > hp_wmi_platform_profile_get() if the BIOS query fails or returns > an invalid/unmapped value, rather than returning an error. > > Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220008 > Closes: https://bugzilla.kernel.org/show_bug.cgi?id=221569 > Closes: https://lore.kernel.org/platform-driver-x86/a3b137df-1b21-4460-b003-58c5ca2d59d4@ukf.sk/ > Fixes: 4296f679ca50 ("platform/x86: hp-wmi: add platform profile support") > Assisted-by: Antigravity:gemini-3.5-flash > Signed-off-by: Marco Scardovi <scardracs@disroot.org> > --- > drivers/platform/x86/hp/hp-wmi.c | 53 +++++++++++++++++++++++++++++--- > 1 file changed, 48 insertions(+), 5 deletions(-) > > diff --git a/drivers/platform/x86/hp/hp-wmi.c b/drivers/platform/x86/hp/hp-wmi.c > index 8ba286ed8721..93df269445eb 100644 > --- a/drivers/platform/x86/hp/hp-wmi.c > +++ b/drivers/platform/x86/hp/hp-wmi.c > @@ -1662,8 +1662,11 @@ static int hp_wmi_platform_profile_get(struct device *dev, > int tp; > > tp = thermal_profile_get(); > - if (tp < 0) > - return tp; > + if (tp < 0) { > + guard(mutex)(&active_platform_profile_lock); > + *profile = active_platform_profile; > + return 0; > + } > > switch (tp) { > case HP_THERMAL_PROFILE_PERFORMANCE: > @@ -1679,7 +1682,9 @@ static int hp_wmi_platform_profile_get(struct device *dev, > *profile = PLATFORM_PROFILE_QUIET; > break; > default: > - return -EINVAL; > + guard(mutex)(&active_platform_profile_lock); > + *profile = active_platform_profile; > + break; > } > > return 0; > @@ -1707,10 +1712,14 @@ static int hp_wmi_platform_profile_set(struct device *dev, > return -EOPNOTSUPP; > } > > + guard(mutex)(&active_platform_profile_lock); > + > err = thermal_profile_set(tp); > if (err) > return err; > > + active_platform_profile = profile; > + > return 0; > } > > @@ -2017,8 +2026,23 @@ static int hp_wmi_platform_profile_probe(void *drvdata, unsigned long *choices) > /* Adding an equivalent to HP Omen software ECO mode: */ > set_bit(PLATFORM_PROFILE_LOW_POWER, choices); > } else { > - set_bit(PLATFORM_PROFILE_QUIET, choices); > - set_bit(PLATFORM_PROFILE_COOL, choices); > + int current_tp = thermal_profile_get(); > + > + if (current_tp < 0) > + return current_tp; > + > + if (thermal_profile_set(HP_THERMAL_PROFILE_QUIET) == 0) > + set_bit(PLATFORM_PROFILE_QUIET, choices); > + if (thermal_profile_set(HP_THERMAL_PROFILE_COOL) == 0) > + set_bit(PLATFORM_PROFILE_COOL, choices); > + if (thermal_profile_set(HP_THERMAL_PROFILE_DEFAULT) == 0) > + set_bit(PLATFORM_PROFILE_BALANCED, choices); > + if (thermal_profile_set(HP_THERMAL_PROFILE_PERFORMANCE) == 0) > + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); > + > + /* Restore the original thermal profile */ > + thermal_profile_set(current_tp); > + return 0; > } > > set_bit(PLATFORM_PROFILE_BALANCED, choices); > @@ -2263,6 +2287,25 @@ static int thermal_profile_setup(struct platform_device *device) > if (err) > return err; > > + /* Initialize active_platform_profile */ > + switch (tp) { > + case HP_THERMAL_PROFILE_PERFORMANCE: > + active_platform_profile = PLATFORM_PROFILE_PERFORMANCE; > + break; > + case HP_THERMAL_PROFILE_DEFAULT: > + active_platform_profile = PLATFORM_PROFILE_BALANCED; > + break; > + case HP_THERMAL_PROFILE_COOL: > + active_platform_profile = PLATFORM_PROFILE_COOL; > + break; > + case HP_THERMAL_PROFILE_QUIET: > + active_platform_profile = PLATFORM_PROFILE_QUIET; > + break; > + default: > + active_platform_profile = PLATFORM_PROFILE_BALANCED; > + break; > + } > + > ops = &hp_wmi_platform_profile_ops; > } > [-- Attachment #2: S/MIME Cryptographic Signature --] [-- Type: application/pkcs7-signature, Size: 4983 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops 2026-06-19 22:09 ` [PATCH 1/1] " Marco Scardovi 2026-06-20 9:36 ` Milan Oravec @ 2026-06-23 15:52 ` Krishna Chomal 2026-06-23 16:45 ` [PATCH v2 0/1] " Marco Scardovi 1 sibling, 1 reply; 6+ messages in thread From: Krishna Chomal @ 2026-06-23 15:52 UTC (permalink / raw) To: Marco Scardovi Cc: moravec, hansg, ilpo.jarvinen, eliadevito, emreleno, rafael, linux-kernel, platform-driver-x86, regressions, regressions On Sat, Jun 20, 2026 at 12:09:11AM +0200, Marco Scardovi wrote: >On generic (non-Omen/non-Victus) HP laptops supported by the >hp-wmi driver, the platform_profile sysfs operations could fail, >leading to 'platform_profile: Failed to get profile for handler hp-wmi' >errors and preventing userspace power profile management from working >correctly. > >The driver was blindly registering all 4 profiles (quiet, cool, >balanced, performance) without checking which ones were actually >supported by the BIOS. Furthermore, when userspace switched profiles, >hp_wmi_platform_profile_get() was called, which queried the BIOS >using thermal_profile_get(). If the BIOS query returned an error or >returned a value not in the recognized cases, the get call failed, >throwing an error. > >Fix this by: >1. Dynamically probing which thermal profiles are supported by the BIOS > during driver registration by temporarily setting each profile and > checking the return code, then restoring the original profile. >2. Initializing and updating active_platform_profile to cache the > last set/boot profile. >3. Falling back to the cached active_platform_profile in > hp_wmi_platform_profile_get() if the BIOS query fails or returns > an invalid/unmapped value, rather than returning an error. > >Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220008 Is the patch tested by them? >Closes: https://bugzilla.kernel.org/show_bug.cgi?id=221569 >Closes: https://lore.kernel.org/platform-driver-x86/a3b137df-1b21-4460-b003-58c5ca2d59d4@ukf.sk/ >Fixes: 4296f679ca50 ("platform/x86: hp-wmi: add platform profile support") >Assisted-by: Antigravity:gemini-3.5-flash >Signed-off-by: Marco Scardovi <scardracs@disroot.org> >--- > drivers/platform/x86/hp/hp-wmi.c | 53 +++++++++++++++++++++++++++++--- > 1 file changed, 48 insertions(+), 5 deletions(-) > >diff --git a/drivers/platform/x86/hp/hp-wmi.c b/drivers/platform/x86/hp/hp-wmi.c >index 8ba286ed8721..93df269445eb 100644 >--- a/drivers/platform/x86/hp/hp-wmi.c >+++ b/drivers/platform/x86/hp/hp-wmi.c >@@ -1662,8 +1662,11 @@ static int hp_wmi_platform_profile_get(struct device *dev, > int tp; > > tp = thermal_profile_get(); >- if (tp < 0) >- return tp; >+ if (tp < 0) { >+ guard(mutex)(&active_platform_profile_lock); >+ *profile = active_platform_profile; >+ return 0; >+ } This does silence the error but I don't think that's a wise decision since it effectively makes userspace blind to any WMI failure. Ideally, if thermal_profile_get() fails then the board isn't supposed to be using HPWMI_THERMAL_PROFILE_QUERY. > > switch (tp) { > case HP_THERMAL_PROFILE_PERFORMANCE: >@@ -1679,7 +1682,9 @@ static int hp_wmi_platform_profile_get(struct device *dev, > *profile = PLATFORM_PROFILE_QUIET; > break; > default: >- return -EINVAL; >+ guard(mutex)(&active_platform_profile_lock); >+ *profile = active_platform_profile; >+ break; > } > > return 0; >@@ -1707,10 +1712,14 @@ static int hp_wmi_platform_profile_set(struct device *dev, > return -EOPNOTSUPP; > } > >+ guard(mutex)(&active_platform_profile_lock); >+ > err = thermal_profile_set(tp); > if (err) > return err; > >+ active_platform_profile = profile; >+ > return 0; > } > >@@ -2017,8 +2026,23 @@ static int hp_wmi_platform_profile_probe(void *drvdata, unsigned long *choices) > /* Adding an equivalent to HP Omen software ECO mode: */ > set_bit(PLATFORM_PROFILE_LOW_POWER, choices); > } else { >- set_bit(PLATFORM_PROFILE_QUIET, choices); >- set_bit(PLATFORM_PROFILE_COOL, choices); >+ int current_tp = thermal_profile_get(); >+ >+ if (current_tp < 0) >+ return current_tp; >+ >+ if (thermal_profile_set(HP_THERMAL_PROFILE_QUIET) == 0) >+ set_bit(PLATFORM_PROFILE_QUIET, choices); >+ if (thermal_profile_set(HP_THERMAL_PROFILE_COOL) == 0) >+ set_bit(PLATFORM_PROFILE_COOL, choices); >+ if (thermal_profile_set(HP_THERMAL_PROFILE_DEFAULT) == 0) >+ set_bit(PLATFORM_PROFILE_BALANCED, choices); >+ if (thermal_profile_set(HP_THERMAL_PROFILE_PERFORMANCE) == 0) >+ set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); Probing for HP_THERMAL_PROFILE_DEFAULT and HP_THERMAL_PROFILE_PERFORMANCE seem redundant since those bits are unconditionally set in this function after the if-else blocks. >+ >+ /* Restore the original thermal profile */ >+ thermal_profile_set(current_tp); >+ return 0; > } > > set_bit(PLATFORM_PROFILE_BALANCED, choices); >@@ -2263,6 +2287,25 @@ static int thermal_profile_setup(struct platform_device *device) > if (err) > return err; > >+ /* Initialize active_platform_profile */ >+ switch (tp) { >+ case HP_THERMAL_PROFILE_PERFORMANCE: >+ active_platform_profile = PLATFORM_PROFILE_PERFORMANCE; >+ break; >+ case HP_THERMAL_PROFILE_DEFAULT: >+ active_platform_profile = PLATFORM_PROFILE_BALANCED; >+ break; >+ case HP_THERMAL_PROFILE_COOL: >+ active_platform_profile = PLATFORM_PROFILE_COOL; >+ break; >+ case HP_THERMAL_PROFILE_QUIET: >+ active_platform_profile = PLATFORM_PROFILE_QUIET; >+ break; >+ default: >+ active_platform_profile = PLATFORM_PROFILE_BALANCED; >+ break; >+ } >+ > ops = &hp_wmi_platform_profile_ops; > } > >-- >2.54.0 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 0/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops 2026-06-23 15:52 ` Krishna Chomal @ 2026-06-23 16:45 ` Marco Scardovi 2026-06-23 16:45 ` [PATCH v2 1/1] " Marco Scardovi 0 siblings, 1 reply; 6+ messages in thread From: Marco Scardovi @ 2026-06-23 16:45 UTC (permalink / raw) To: krishna.chomal108 Cc: eliadevito, emreleno, hansg, ilpo.jarvinen, linux-kernel, moravec, platform-driver-x86, rafael, regressions, regressions, Marco Scardovi Hi Krishna, Thank you for your review :) Your points were indeed correct so I came up with another way to fix it. Here is v2 of the patch to fix platform_profile support on generic HP laptops. To answer your question the patch has been tested by Milan, who reported bug 221569. For 220008 I had not any response from TuralTo unfortunately but the underlying problem was the same AFAICS. @milan please test this out and let me know like you did before. Changes in v2: - Drop active_platform_profile fallback/caching and mutex logic for generic HP laptops. Propagate WMI query errors directly to userspace instead of silencing them, as suggested. - Restructure the probe function to explicitly set platform profile choices in each path rather than relying on a confusing combination of early returns and fall-throughs. The driver was blindly registering all 4 profiles (quiet, cool, balanced, performance) without checking which ones were actually supported by the BIOS. This patch fixes a platform_profile regression on generic (non-Omen/non-Victus) HP laptops, where switching or querying power profiles fails with the error: "platform_profile: Failed to get profile for handler hp-wmi" We resolve this by dynamically probing supported thermal profiles at boot time by temporarily setting each profile and checking the return code, then restoring the original profile. Marco Scardovi (1): platform/x86: hp-wmi: fix platform profile issues on generic HP laptops drivers/platform/x86/hp/hp-wmi.c | 37 ++++++++++++++++++++++++++------ 1 file changed, 30 insertions(+), 7 deletions(-) -- 2.54.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops 2026-06-23 16:45 ` [PATCH v2 0/1] " Marco Scardovi @ 2026-06-23 16:45 ` Marco Scardovi 0 siblings, 0 replies; 6+ messages in thread From: Marco Scardovi @ 2026-06-23 16:45 UTC (permalink / raw) To: krishna.chomal108 Cc: eliadevito, emreleno, hansg, ilpo.jarvinen, linux-kernel, moravec, platform-driver-x86, rafael, regressions, regressions, Marco Scardovi On generic (non-Omen/non-Victus) HP laptops supported by the hp-wmi driver, the platform_profile sysfs operations could fail, leading to 'platform_profile: Failed to get profile for handler hp-wmi' errors and preventing userspace power profile management from working correctly. The driver was blindly registering all 4 profiles (quiet, cool, balanced, performance) without checking which ones were actually supported by the BIOS. Furthermore, when userspace switched profiles, hp_wmi_platform_profile_get() was called, which queried the BIOS using thermal_profile_get(). If the BIOS query returned an error or returned a value not in the recognized cases, the get call failed, throwing an error. Fix this by dynamically probing which thermal profiles are supported by the BIOS during driver registration. This is done by temporarily setting each profile and checking the return code, then restoring the original profile. Restructure the probe function to explicitly populate choices for each laptop type, avoiding confusing fall-through behavior. There are no expected negative side effects, as the active profile is restored to its original state during probe time, and only validated profiles are exposed. Closes: https://bugzilla.kernel.org/show_bug.cgi?id=220008 Closes: https://bugzilla.kernel.org/show_bug.cgi?id=221569 Closes: https://lore.kernel.org/platform-driver-x86/a3b137df-1b21-4460-b003-58c5ca2d59d4@ukf.sk/ Fixes: 4296f679ca50 ("platform/x86: hp-wmi: add platform profile support") Assisted-by: Antigravity:gemini-3.5-flash Signed-off-by: Marco Scardovi <scardracs@disroot.org> --- drivers/platform/x86/hp/hp-wmi.c | 37 ++++++++++++++++++++++++++------ 1 file changed, 30 insertions(+), 7 deletions(-) diff --git a/drivers/platform/x86/hp/hp-wmi.c b/drivers/platform/x86/hp/hp-wmi.c index 8ba286ed8721..82fdfaa169ec 100644 --- a/drivers/platform/x86/hp/hp-wmi.c +++ b/drivers/platform/x86/hp/hp-wmi.c @@ -2011,19 +2011,42 @@ static int hp_wmi_platform_profile_probe(void *drvdata, unsigned long *choices) { if (is_omen_thermal_profile()) { set_bit(PLATFORM_PROFILE_COOL, choices); - } else if (is_victus_thermal_profile()) { + set_bit(PLATFORM_PROFILE_BALANCED, choices); + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + return 0; + } + + if (is_victus_thermal_profile()) { set_bit(PLATFORM_PROFILE_QUIET, choices); - } else if (is_victus_s_thermal_profile()) { + set_bit(PLATFORM_PROFILE_BALANCED, choices); + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + return 0; + } + + if (is_victus_s_thermal_profile()) { /* Adding an equivalent to HP Omen software ECO mode: */ set_bit(PLATFORM_PROFILE_LOW_POWER, choices); - } else { - set_bit(PLATFORM_PROFILE_QUIET, choices); - set_bit(PLATFORM_PROFILE_COOL, choices); + set_bit(PLATFORM_PROFILE_BALANCED, choices); + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + return 0; } - set_bit(PLATFORM_PROFILE_BALANCED, choices); - set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + /* Generic HP laptops supported by hp-wmi: probe dynamically */ + int current_tp = thermal_profile_get(); + if (current_tp < 0) + return current_tp; + + if (thermal_profile_set(HP_THERMAL_PROFILE_QUIET) == 0) + set_bit(PLATFORM_PROFILE_QUIET, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_COOL) == 0) + set_bit(PLATFORM_PROFILE_COOL, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_DEFAULT) == 0) + set_bit(PLATFORM_PROFILE_BALANCED, choices); + if (thermal_profile_set(HP_THERMAL_PROFILE_PERFORMANCE) == 0) + set_bit(PLATFORM_PROFILE_PERFORMANCE, choices); + /* Restore original thermal profile */ + thermal_profile_set(current_tp); return 0; } -- 2.54.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-06-23 16:46 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-06-19 22:09 [PATCH 0/1] platform/x86: hp-wmi: fix platform profile issues on generic HP laptops Marco Scardovi 2026-06-19 22:09 ` [PATCH 1/1] " Marco Scardovi 2026-06-20 9:36 ` Milan Oravec 2026-06-23 15:52 ` Krishna Chomal 2026-06-23 16:45 ` [PATCH v2 0/1] " Marco Scardovi 2026-06-23 16:45 ` [PATCH v2 1/1] " Marco Scardovi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.