mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: Matias Civadda <matias.civadda2342001@gmail.com>,
	 Chris Taraszka <chris@miget.com>
Cc: Hans de Goede <hansg@kernel.org>,
	platform-driver-x86@vger.kernel.org,
	 LKML <linux-kernel@vger.kernel.org>, Armin Wolf <W_Armin@gmx.de>,
	 Mingyou Chen <qby140326@gmail.com>
Subject: Re: [PATCH] platform/x86: bitland-mifs-wmi: Add Xiaomi Book Pro 14 platform profile values
Date: Tue, 29 Sep 2026 12:14:25 +0300 (EEST)	[thread overview]
Message-ID: <622041b8-6ace-4fdf-43b5-5427ba0d88d6@linux.intel.com> (raw)
In-Reply-To: <595cf557-e386-4ca3-b8ea-2ce7fe160018@gmail.com>

FYI,

There are a few more of these in the queue:

https://patchwork.kernel.org/project/platform-driver-x86/patch/20260826132924.179614-1-kento@kekto.ru/
https://patchwork.kernel.org/project/platform-driver-x86/patch/fb4e0678-3028-463e-8a8c-1b680e40c4ee@gmail.com/

The first one is a bit messy, as it adds some keys as well (which should 
have been in a separate patch).

And some more in Mingyou's redmi-wmi merge series's last patch:

https://patchwork.kernel.org/project/platform-driver-x86/patch/20260816100813.300450-7-qby140326@gmail.com/

The profile mode abstraction in that series seems not general enough 
though so the generalization work we've discussed is still necessary and 
should be used as a base for that series.

--
 i.

On Mon, 28 Sep 2026, Matias Civadda wrote:

> Hi Chris,
> 
> No problem, your proposal works for me. Please CC me when you send
> the series so I can test the shared changes and base my TM2107 v3
> on your generic patch.
> 
> Thanks,
> Matias
> 
> > Thanks, makes sense.
> > 
> > Matias, one thing I ran into when trying your generic part on the
> > Xiaomi Book Pro 14: its values are sparse (0x02, 0x03, 0x04, 0x09,
> > 0x0a) and 0x02 and 0x0a both mean low-power. With a profile array
> > indexed by the firmware value, the unused slots read back as
> > PLATFORM_PROFILE_LOW_POWER (0), and the reverse lookup for low-power
> > finds index 0, which this firmware ignores.
> > 
> > So I'd suggest {value, profile} pairs for the shared part. I have that
> > as a standalone patch on pdx86/for-next, with only the profile table
> > in the quirk struct, so your response layout and AC check handling
> > would go on top in your TM2107 patch. If that works for you I'll send
> > it as v2 together with the Xiaomi entry, and you can base your v3 on
> > it. If you'd rather own the generic patch, that's fine too, I'll
> > rebase the Xiaomi part on yours.
> > 
> 

-- 
 i.


      reply	other threads:[~2026-09-29  9:14 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-28 15:39 Chris Taraszka
2026-09-28 15:49 ` Ilpo Järvinen
2026-09-28 16:41   ` Chris Taraszka
2026-09-28 23:26     ` Matias Civadda
2026-09-29  9:14       ` Ilpo Järvinen [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=622041b8-6ace-4fdf-43b5-5427ba0d88d6@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=W_Armin@gmx.de \
    --cc=chris@miget.com \
    --cc=hansg@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matias.civadda2342001@gmail.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=qby140326@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®