X86 platform drivers
 help / color / mirror / Atom feed
From: kemal <kmal@cock.li>
To: Armin Wolf <W_Armin@gmx.de>
Cc: platform-driver-x86@vger.kernel.org
Subject: Re: toshiba_acpi: fn key started working inverse in satellite L50-B
Date: Mon, 01 Jul 2024 22:41:50 +0300	[thread overview]
Message-ID: <QPMYFS.N62TVTKUUR0K@cock.li> (raw)
In-Reply-To: <a58e68e5-286c-4f16-bd4b-37d44dc021d8@gmx.de>

im very sorry, i think i failed to mail my reply to the previous 
message.
also huge thanks for the quick response and fix.

yes, the prototype driver does indeed fix the issue.
On Pzt, Tem 1 2024 at 21:09:40 +02:00:00, Armin Wolf <W_Armin@gmx.de> 
wrote:
> Am 01.07.24 um 19:14 schrieb Armin Wolf:
> 
>> Am 01.07.24 um 17:53 schrieb kemal:
>> 
>>> in my TOSHIBA SATELLITE L50-B, the fn key started working
>>> the inverse of how it should after upgrading to kernel 6.9.7
>>> fn+function key used to emit an actual function key press, now 
>>> instead
>>> it emits a hotkey press and the key itself emits an actual function 
>>> key
>>> press. there are some recent commits to toshiba_acpi, but i cant 
>>> see how
>>> it could affect my hardware.
>>> what could be the reason i lost the old behavior?
>> 
>> Hi,
>> 
>> it seems that commit 23f1d8b47d12 ("platform/x86: toshiba_acpi: Add
>> quirk for buttons on Z830")
>> botched up the DMI quirk handling.
>> 
>> The global quick flag is tested in toshiba_acpi_enable_hotkeys(),
>> which is called by
>> toshiba_acpi_setup_keyboard() before the global quirk flag is 
>> properly
>> initialized.
>> 
>> Can you compile kernel modules on your device? If yes, then i can 
>> send
>> you a prototype version
>> of this driver which should fix this issue.
>> 
>> Thanks,
>> Armin Wolf
>> 
> I attached a prototype version of the driver which should fix the 
> issue. You can build the module using
> "make" and load the resulting kernel module using "sudo insmod 
> toshiba_acpi.ko" (remember to unload the
> old driver first using "sudo modprobe -r toshiba_acpi").
> 
> Can you verify that the prototype driver indeed fixes the issue?
> 
> Thanks,
> Armin Wolf



      reply	other threads:[~2024-07-01 19:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-01 15:53 toshiba_acpi: fn key started working inverse in satellite L50-B kemal
2024-07-01 17:14 ` Armin Wolf
2024-07-01 19:09   ` Armin Wolf
2024-07-01 19:41     ` kemal [this message]

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=QPMYFS.N62TVTKUUR0K@cock.li \
    --to=kmal@cock.li \
    --cc=W_Armin@gmx.de \
    --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