mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Mark Pearson" <mpearson-lenovo@squebb.ca>
To: "Borys Turko" <turkoborys@gmail.com>
Cc: "Ike Panhc" <ikepanhc@gmail.com>,
	"Hans de Goede" <hansg@kernel.org>,
	"Derek J . Clark" <derekjohn.clark@gmail.com>,
	"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
	"platform-driver-x86@vger.kernel.org"
	<platform-driver-x86@vger.kernel.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] platform/x86: ideapad-laptop: remap "Star with S" key to KEY_DASHBOARD
Date: Fri, 15 May 2026 13:43:55 -0400	[thread overview]
Message-ID: <67269355-486f-496c-837f-9122f93a79ed@app.fastmail.com> (raw)
In-Reply-To: <CAEAo-LjdVzQ7zsiz8NrWKYVQ0V11yRwiX98BOsCL6EVQvpZZGw@mail.gmail.com>

Hi Borys,

On Fri, May 15, 2026, at 1:38 PM, Borys wrote:
> Hi,
>
> Any chance you've had a look at my response below?

Sorry - it got lost in my email swamp :)

I don't feel strongly about it - I'll happily defer to the experienced maintainers perspective. Just wanted to highlight that potentially you might break somebodies carefully crafted key mappings
(but realistically, I think the odds on that are towards the low side)

Mark

>
> Best regards,
> Borys
>
> Op do 23 apr 2026 om 09:59 schreef Borys <turkoborys@gmail.com>:
>>
>> Hi Mark,
>>
>> Thanks for the detailed feedback and for testing this on your
>> ThinkPad. I’ve done some further testing on the IdeaPad Gaming 3
>> (Cinnamon) to address your points and cross-referenced several IdeaPad
>> manuals.
>>
>> On this specific hardware, I found that KEY_FAVORITES currently
>> results in no action at all. It acts as a dead key in the desktop
>> environment, so the risk of a regression for existing users seems
>> minimal compared to the benefit of making the key functional.
>>
>> While the ThinkPad intent might be more flexible, the IdeaPad manuals
>> (for the Gaming and 5 series, for example) specifically state the key
>> is intended to "Open an app" or "Open Lenovo Vantage/PC Manager." I
>> tested KEY_BOOKMARKS as you suggested, but found that browsers like
>> Firefox "hijack" this key to open a sidebar, which prevents the user
>> from mapping it to a system tool as the manual intends. Additionally,
>> in my environment, the system labels this key as "Favorites," which
>> might be confusing for a "PC Manager" launcher.
>>
>> In my testing, KEY_CONFIG seems to avoid these conflicts. The system
>> identifies it as "Tools," which is a perfect semantic match for the
>> Vantage tools mentioned in the documentation. It isn't intercepted by
>> browsers and remains fully mappable by the user. Using a neutral key
>> like KEY_PROG4 would also achieve this same goal of giving the user
>> full control without default interference.
>>
>> I’m curious to hear your thoughts on these findings and how we can
>> best align the driver with the "user-defined" intent described in the
>> manuals.
>>
>> Best regards,
>>
>> Borys

      reply	other threads:[~2026-05-15 17:44 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-22 10:42 Borys Turko
2026-04-23  1:35 ` Mark Pearson
2026-04-23  7:59   ` Borys
2026-05-15 17:38     ` Borys
2026-05-15 17:43       ` Mark Pearson [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=67269355-486f-496c-837f-9122f93a79ed@app.fastmail.com \
    --to=mpearson-lenovo@squebb.ca \
    --cc=derekjohn.clark@gmail.com \
    --cc=hansg@kernel.org \
    --cc=ikepanhc@gmail.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=turkoborys@gmail.com \
    /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

all inboxes | Powered by JetHome®