X86 platform drivers
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Armin Wolf <W_Armin@gmx.de>, Hridesh MG <hridesh699@gmail.com>
Cc: platform-driver-x86@vger.kernel.org
Subject: Re: acer-wmi: Nitro button doesn't produce a WMI event
Date: Mon, 6 Jan 2025 19:33:23 +0100	[thread overview]
Message-ID: <5d2ee19b-eb78-4c67-9a5f-82859d8ae8bc@redhat.com> (raw)
In-Reply-To: <ceb159f1-2900-4946-a9c9-088ba99d2d67@gmx.de>

Hi all,

On 6-Jan-25 5:59 PM, Armin Wolf wrote:
> Am 02.01.25 um 07:18 schrieb Hridesh MG:
> 
>> On Thu, Jan 2, 2025 at 1:51 AM Armin Wolf <W_Armin@gmx.de> wrote:
>>> Am 01.01.25 um 20:53 schrieb Hridesh MG:
>>>
>>>>> This ACPI method should trigger the turbo mode button (found inside the DSDT table):
>>>>>
>>>>>           Method (_Q58, 0, NotSerialized)  // _Qxx: EC Query, xx=0x00-0xFF
>>>>>            {
>>>>>                   Debug = "=====PROJECT_QUERY_58====="
>>>>>                    ^^^WMID.FEBC [Zero] = 0x07
>>>>>                    ^^^WMID.FEBC [One] = 0x04
>>>>>                    ^^^WMID.FEBC [0x02] = One
>>>>>                    Notify (WMID, 0xBC) // Device-Specific
>>>>>           }
>>>>>
>>>> I feel like an idiot right now but I just realized something: So far
>>>> I've been assuming that the button I've been calling the "Turbo
>>>> Button" worked the same way on both Nitro and Predator laptops, but
>>>> that's not the case.
>>>>
>>>> On Predator laptops, the button directly enables Turbo mode but on the
>>>> Nitro it only opens the Nitro Sense app. I had assumed that both
>>>> buttons simply opened the app and that directly enabling Turbo mode
>>>> was a feature provided by the Linux driver.
>>>>
>>>> Given this, the ACPI code that you linked earlier is probably for the
>>>> Predator's "Turbo Button". Could it be that the button on my laptop
>>>> doesn’t use that ACPI code at all, considering its functionality is
>>>> completely different? (Though i doubt Acer would leave it in if it
>>>> wasn't being used)
>>> It is quite common for manufactures to just copy and paste ACPI code snippets, so it is not
>>> unusual to have some unused code inside the ACPI tables.
>>>
>>> Did you receive any input events or dmesg messages when pressing that button?
>>>
>>> If no then it could also be that this button depends on the Intel THC touch controller to work,
>>> take a look at https://lore.kernel.org/all/20241216014127.3722172-1-even.xu@intel.com for details
>>> about the Intel THC controller.
>>>
>>> Thanks,
>>> Armin Wolf
>>>
>> I do receive input events, it shows up with the keycode 433 and symbol
>> XF86Presentation. I think you mentioned something regarding an i8042
>> filter, can we use that to call the function to cycle between the
>> profiles?
> 
> Since this is a valid keycode i advise against filtering it out. Instead it would make more
> sense to allow userspace to control the turbo state for example trough a sysfs attribute.
> 
> Then you can use a userspace program to react to this key press.

Since some laptops handle this completely in the EC, so far drivers
which get an event for this have been reacting to this event
themselves and calling the platform_profile_cycle() helper to cycle
through the various performance profiles.

I think it would be best to do this here too.

Regards,

Hans




  reply	other threads:[~2025-01-06 18:33 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-16  6:15 acer-wmi: Nitro button doesn't produce a WMI event Hridesh MG
2024-12-17  0:47 ` Armin Wolf
2024-12-17  8:33   ` Hridesh MG
2024-12-17 13:28     ` Armin Wolf
2024-12-17 17:41       ` Hridesh MG
2024-12-17 20:12         ` Armin Wolf
2024-12-17 20:52           ` Hridesh MG
2024-12-17 21:44             ` Armin Wolf
     [not found]               ` <CALiyAok0_JKE+BkUQxHHH8Z39FHq5bW0Hb1Gk-MYHgDy0yFoMA@mail.gmail.com>
2024-12-23 23:06                 ` Armin Wolf
2024-12-24 12:55                   ` Armin Wolf
2024-12-24 18:45                     ` Hridesh MG
2024-12-25 20:51                       ` Armin Wolf
2024-12-26 17:41                         ` Hridesh MG
2024-12-26 23:40                           ` Armin Wolf
2025-01-01 10:48           ` Hridesh MG
2025-01-01 12:37             ` Armin Wolf
2025-01-01 19:53               ` Hridesh MG
2025-01-01 20:21                 ` Armin Wolf
2025-01-02  6:18                   ` Hridesh MG
2025-01-06 16:59                     ` Armin Wolf
2025-01-06 18:33                       ` Hans de Goede [this message]
2025-01-06 22:29                         ` Armin Wolf
2025-01-07  9:45                           ` Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5d2ee19b-eb78-4c67-9a5f-82859d8ae8bc@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=W_Armin@gmx.de \
    --cc=hridesh699@gmail.com \
    --cc=platform-driver-x86@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox