From: Mingyou Chen <qby140326@gmail.com>
To: "Miloš Vlku" <vlku.milos.fun@gmail.com>,
"Anton Karasev" <uselessfire@gmail.com>
Cc: ilpo.jarvinen@linux.intel.com, hansg@kernel.org,
platform-driver-x86@vger.kernel.org,
linux-kernel@vger.kernel.org, W_Armin@gmx.de,
ilya.gladyshev@linux.dev, foxido@foxido.dev, kento@kekto.ru,
ericted8810@gmail.com, chris@miget.com,
matias.civadda2342001@gmail.com, btx342@gmail.com,
wleizc7319@gmail.com, wolf109909@outlook.com, i@rsplwe.com,
bozhenpeng93@gmail.com, martiya.ar@gmail.com,
aleksejirlik@gmail.com
Subject: Re: [PATCH v6 5/5] platform/x86: bitland-mifs-wmi: Add per-machine ops table
Date: Fri, 2 Oct 2026 09:34:23 +0800 [thread overview]
Message-ID: <3a035210-a620-465c-9f87-b5c5b45fdc17@gmail.com> (raw)
In-Reply-To: <CAObHBTxHQDfVH8Su1DLABtC5uUsAGBBk=KvcCOvtTs5gMGamnA@mail.gmail.com>
On 10/2/26 08:51, Miloš Vlku wrote:
> Follow-up to my own mail: I have now run those values rather than only
> reading them out of the tables.
>
> Built current pdx86 for-next with Chris's GET-only function-id change
> and balanced 1 / quiet 2 / performance 3 / full speed 4 on top. All four
> profiles apply and read back correctly, checked against the EC register
> each time:
>
> low-power -> QFAN 2
> balanced -> QFAN 1
> balanced-performance -> QFAN 3
> performance -> QFAN 4
>
> All three power-profiles-daemon profiles work as well, so the desktop
> slider works end to end on TM2307.
>
> Two things fell out of that:
>
> - Without Chris's change every one of those writes fails with -ENOMSG
> here, exactly as you described, and with it all of them succeed. So
> that is a Tested-by from a second board if it is useful.
>
> - No machine-specific capability check is needed on this one.
> WMI_FN_SYSTEM_AC_TYPE is unimplemented, Armin's work reports that as
> -EOPNOTSUPP, and bitland_pp_check_capability() already treats that as
> "skip the check". The stock one is fine as-is.
>
> So the TM2307 entry comes out as just a mode map plus a DMI match.
> Attached, against the v9 ops table and checkpatch-clean, in case it is
> useful to have in hand while you write the TM2309 one. I will resend it
> properly once that series settles -- posting it now mainly so the values
> sit somewhere concrete, and Mingyou can fold it straight into v10 if
> that is easier than carrying a separate patch.
>
> Thanks,
> Milos
Hello,
`git am` failed to apply it with a "corrupt patch" error.
Could you please regenerate it cleanly with `git format-patch` and send
a v2 via `git send-email`? (don't edit the patch content manually)
Since the code logic itself doesn't need to change, feel free to include
my Reviewed-by tag below in your v2 commit message:
Reviewed-by: Mingyou Chen <qby140326@gmail.com>
next prev parent reply other threads:[~2026-10-02 1:34 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-29 13:44 [PATCH v6 0/5] Merge redmi-wmi into bitland-mifs-wmi Mingyou Chen
2026-09-29 13:44 ` [PATCH v6 1/5] MAINTAINERS: Add maintainer entry of bitland-mifs-wmi driver Mingyou Chen
2026-09-29 13:45 ` [PATCH v6 2/5] platform/x86: bitland-mifs-wmi: Merge the function of redmi-wmi into the bitland driver Mingyou Chen
2026-09-30 1:58 ` Anton Karasev
2026-09-30 20:53 ` Ilya Gladyshev
2026-09-30 22:01 ` Ilya Gladyshev
2026-09-29 13:45 ` [PATCH v6 3/5] platform/x86: bitland-mifs-wmi: Add Redmi mic-mute key entries Mingyou Chen
2026-09-29 13:45 ` [PATCH v6 4/5] platform/x86: redmi-wmi: Drop redmi-wmi driver Mingyou Chen
2026-09-29 13:45 ` [PATCH v6 5/5] platform/x86: bitland-mifs-wmi: Add per-machine ops table Mingyou Chen
2026-09-30 1:50 ` Anton Karasev
2026-10-02 0:18 ` Miloš Vlku
2026-10-02 0:51 ` Miloš Vlku
2026-10-02 1:15 ` Mingyou Chen
2026-10-02 1:34 ` Mingyou Chen [this message]
2026-10-02 9:21 ` Ilpo Järvinen
2026-10-08 15:28 ` Anton Karasev
2026-10-08 22:06 ` Miloš Vlku
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=3a035210-a620-465c-9f87-b5c5b45fdc17@gmail.com \
--to=qby140326@gmail.com \
--cc=W_Armin@gmx.de \
--cc=aleksejirlik@gmail.com \
--cc=bozhenpeng93@gmail.com \
--cc=btx342@gmail.com \
--cc=chris@miget.com \
--cc=ericted8810@gmail.com \
--cc=foxido@foxido.dev \
--cc=hansg@kernel.org \
--cc=i@rsplwe.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=ilya.gladyshev@linux.dev \
--cc=kento@kekto.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=martiya.ar@gmail.com \
--cc=matias.civadda2342001@gmail.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=uselessfire@gmail.com \
--cc=vlku.milos.fun@gmail.com \
--cc=wleizc7319@gmail.com \
--cc=wolf109909@outlook.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 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.