* [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
@ 2024-02-28 15:01 Vishnu Sankar
2024-02-28 16:09 ` Hans de Goede
2024-03-04 13:17 ` Ilpo Järvinen
0 siblings, 2 replies; 5+ messages in thread
From: Vishnu Sankar @ 2024-02-28 15:01 UTC (permalink / raw
To: hdegoede, ilpo.jarvinen
Cc: platform-driver-x86, linux-kernel, mpearson-lenovo, vsankar,
Vishnu Sankar
Add more ThinkPads with non-standard register addresses to read fan values.
ThinkPads added are L13 Yoga Gen1, X13 Yoga Gen1, L380, L390, 11e Gen5 GL,
11e Gen5 GL-R, 11e Gen5 KL-Y.
Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
---
drivers/platform/x86/thinkpad_acpi.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
index 5ecd9d33250d..23b2abe44a5a 100644
--- a/drivers/platform/x86/thinkpad_acpi.c
+++ b/drivers/platform/x86/thinkpad_acpi.c
@@ -8767,6 +8767,13 @@ static const struct tpacpi_quirk fan_quirk_table[] __initconst = {
TPACPI_Q_LNV3('N', '3', '7', TPACPI_FAN_2CTL), /* T15g (2nd gen) */
TPACPI_Q_LNV3('R', '1', 'F', TPACPI_FAN_NS), /* L13 Yoga Gen 2 */
TPACPI_Q_LNV3('N', '2', 'U', TPACPI_FAN_NS), /* X13 Yoga Gen 2*/
+ TPACPI_Q_LNV3('R', '0', 'R', TPACPI_FAN_NS), /* L380 */
+ TPACPI_Q_LNV3('R', '1', '5', TPACPI_FAN_NS), /* L13 Yoga Gen 1 */
+ TPACPI_Q_LNV3('R', '1', '0', TPACPI_FAN_NS), /* L390 */
+ TPACPI_Q_LNV3('N', '2', 'L', TPACPI_FAN_NS), /* X13 Yoga Gen 1 */
+ TPACPI_Q_LNV3('R', '0', 'T', TPACPI_FAN_NS), /* 11e Gen5 GL */
+ TPACPI_Q_LNV3('R', '1', 'D', TPACPI_FAN_NS), /* 11e Gen5 GL-R */
+ TPACPI_Q_LNV3('R', '0', 'V', TPACPI_FAN_NS), /* 11e Gen5 KL-Y */
TPACPI_Q_LNV3('N', '1', 'O', TPACPI_FAN_NOFAN), /* X1 Tablet (2nd gen) */
};
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
2024-02-28 15:01 [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan Vishnu Sankar
@ 2024-02-28 16:09 ` Hans de Goede
2024-02-28 20:19 ` Mark Pearson
2024-03-04 13:17 ` Ilpo Järvinen
1 sibling, 1 reply; 5+ messages in thread
From: Hans de Goede @ 2024-02-28 16:09 UTC (permalink / raw
To: Vishnu Sankar, ilpo.jarvinen
Cc: platform-driver-x86, linux-kernel, mpearson-lenovo, vsankar
Hi,
On 2/28/24 16:01, Vishnu Sankar wrote:
> Add more ThinkPads with non-standard register addresses to read fan values.
>
> ThinkPads added are L13 Yoga Gen1, X13 Yoga Gen1, L380, L390, 11e Gen5 GL,
> 11e Gen5 GL-R, 11e Gen5 KL-Y.
>
> Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
Thanks, I have no objection against this patch:
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
But this feels like it will become a game of whack-a-mole.
Mark can you reach out to the ThinkPad firmware team and
figure out if there is some supported way to automatically detect
this ?
Regards,
Hans
> ---
> drivers/platform/x86/thinkpad_acpi.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c
> index 5ecd9d33250d..23b2abe44a5a 100644
> --- a/drivers/platform/x86/thinkpad_acpi.c
> +++ b/drivers/platform/x86/thinkpad_acpi.c
> @@ -8767,6 +8767,13 @@ static const struct tpacpi_quirk fan_quirk_table[] __initconst = {
> TPACPI_Q_LNV3('N', '3', '7', TPACPI_FAN_2CTL), /* T15g (2nd gen) */
> TPACPI_Q_LNV3('R', '1', 'F', TPACPI_FAN_NS), /* L13 Yoga Gen 2 */
> TPACPI_Q_LNV3('N', '2', 'U', TPACPI_FAN_NS), /* X13 Yoga Gen 2*/
> + TPACPI_Q_LNV3('R', '0', 'R', TPACPI_FAN_NS), /* L380 */
> + TPACPI_Q_LNV3('R', '1', '5', TPACPI_FAN_NS), /* L13 Yoga Gen 1 */
> + TPACPI_Q_LNV3('R', '1', '0', TPACPI_FAN_NS), /* L390 */
> + TPACPI_Q_LNV3('N', '2', 'L', TPACPI_FAN_NS), /* X13 Yoga Gen 1 */
> + TPACPI_Q_LNV3('R', '0', 'T', TPACPI_FAN_NS), /* 11e Gen5 GL */
> + TPACPI_Q_LNV3('R', '1', 'D', TPACPI_FAN_NS), /* 11e Gen5 GL-R */
> + TPACPI_Q_LNV3('R', '0', 'V', TPACPI_FAN_NS), /* 11e Gen5 KL-Y */
> TPACPI_Q_LNV3('N', '1', 'O', TPACPI_FAN_NOFAN), /* X1 Tablet (2nd gen) */
> };
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
2024-02-28 16:09 ` Hans de Goede
@ 2024-02-28 20:19 ` Mark Pearson
2024-02-28 21:43 ` Vishnu Sankar
0 siblings, 1 reply; 5+ messages in thread
From: Mark Pearson @ 2024-02-28 20:19 UTC (permalink / raw
To: Hans de Goede, Vishnu Sankar, Ilpo Järvinen
Cc: platform-driver-x86@vger.kernel.org, linux-kernel, Vishnu Sankar
Hi Hans,
On Wed, Feb 28, 2024, at 11:09 AM, Hans de Goede wrote:
> Hi,
>
> On 2/28/24 16:01, Vishnu Sankar wrote:
>> Add more ThinkPads with non-standard register addresses to read fan values.
>>
>> ThinkPads added are L13 Yoga Gen1, X13 Yoga Gen1, L380, L390, 11e Gen5 GL,
>> 11e Gen5 GL-R, 11e Gen5 KL-Y.
>>
>> Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
>
> Thanks, I have no objection against this patch:
>
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
>
> But this feels like it will become a game of whack-a-mole.
>
> Mark can you reach out to the ThinkPad firmware team and
> figure out if there is some supported way to automatically detect
> this ?
>
These are all older platforms and we're not expecting to see this on any newer platforms...if it shows up it's because they messed up (there is a spec and they're supposed to follow it).
Vishnu did review with the FW team which platforms had used this implementation, and we believe the list below to be complete.
Vishnu, please correct me if you've heard otherwise.
As a note, I did review this during an internal review, before Vishnu pushed it, so I'll add:
Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
Mark
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
2024-02-28 20:19 ` Mark Pearson
@ 2024-02-28 21:43 ` Vishnu Sankar
0 siblings, 0 replies; 5+ messages in thread
From: Vishnu Sankar @ 2024-02-28 21:43 UTC (permalink / raw
To: Mark Pearson
Cc: Hans de Goede, Ilpo Järvinen,
platform-driver-x86@vger.kernel.org, linux-kernel, Vishnu Sankar
Hi Hans and Mark,
Thanks for the review.
On Thu, Feb 29, 2024 at 5:19 AM Mark Pearson <mpearson-lenovo@squebb.ca> wrote:
>
> Hi Hans,
>
> On Wed, Feb 28, 2024, at 11:09 AM, Hans de Goede wrote:
> > Hi,
> >
> > On 2/28/24 16:01, Vishnu Sankar wrote:
> >> Add more ThinkPads with non-standard register addresses to read fan values.
> >>
> >> ThinkPads added are L13 Yoga Gen1, X13 Yoga Gen1, L380, L390, 11e Gen5 GL,
> >> 11e Gen5 GL-R, 11e Gen5 KL-Y.
> >>
> >> Signed-off-by: Vishnu Sankar <vishnuocv@gmail.com>
> >
> > Thanks, I have no objection against this patch:
> >
> > Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> >
> > But this feels like it will become a game of whack-a-mole.
> >
> > Mark can you reach out to the ThinkPad firmware team and
> > figure out if there is some supported way to automatically detect
> > this ?
> >
> These are all older platforms and we're not expecting to see this on any newer platforms...if it shows up it's because they messed up (there is a spec and they're supposed to follow it).
> Vishnu did review with the FW team which platforms had used this implementation, and we believe the list below to be complete.
>
> Vishnu, please correct me if you've heard otherwise.
This is confirmed by the FW team.
The list of systems is complete and no more systems are expected (as of now).
>
> As a note, I did review this during an internal review, before Vishnu pushed it, so I'll add:
>
> Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
>
> Mark
--
Regards,
Vishnu Sankar
+817015150407 (Japan)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
2024-02-28 15:01 [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan Vishnu Sankar
2024-02-28 16:09 ` Hans de Goede
@ 2024-03-04 13:17 ` Ilpo Järvinen
1 sibling, 0 replies; 5+ messages in thread
From: Ilpo Järvinen @ 2024-03-04 13:17 UTC (permalink / raw
To: hdegoede, Vishnu Sankar
Cc: platform-driver-x86, linux-kernel, mpearson-lenovo, vsankar
On Thu, 29 Feb 2024 00:01:49 +0900, Vishnu Sankar wrote:
> Add more ThinkPads with non-standard register addresses to read fan values.
>
> ThinkPads added are L13 Yoga Gen1, X13 Yoga Gen1, L380, L390, 11e Gen5 GL,
> 11e Gen5 GL-R, 11e Gen5 KL-Y.
>
>
Thank you for your contribution, it has been applied to my local
review-ilpo branch. Note it will show up in the public
platform-drivers-x86/review-ilpo branch only once I've pushed my
local branch there, which might take a while.
The list of commits applied:
[1/1] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan
commit: 0dfb4617af0d4cfbaf75590a83e2ce367b67134a
--
i.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-03-04 13:17 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-28 15:01 [PATCH] platform/x86: thinkpad_acpi: Add more ThinkPads with non-standard reg address for fan Vishnu Sankar
2024-02-28 16:09 ` Hans de Goede
2024-02-28 20:19 ` Mark Pearson
2024-02-28 21:43 ` Vishnu Sankar
2024-03-04 13:17 ` Ilpo Järvinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).