Re: [PATCH] platform/x86: thinkpad_acpi: Fix fan speed reporting on Edge E330

[Mark Pearson]

On Thu, Aug 6, 2026, at 11:44 AM, Andrew Onyshchuk wrote:
> The ThinkPad Edge E330 with H3 firmware uses the non-standard EC fan
> register block. Without a matching quirk, thinkpad_acpi reads the legacy
> tachometer registers and reports 0 RPM.
>
> Add the H3 BIOS family to TPACPI_FAN_NS so the driver reads the fan
> period from EC register 0x95 using the existing non-standard reporting
> path.
>
> Tested on a ThinkPad Edge E330 with BIOS H3ET77WW and EC H3EC35WW.
>
> Signed-off-by: Andrew Onyshchuk <andryk.rv@xxxxxxxxx>
> ---
>  drivers/platform/x86/lenovo/thinkpad_acpi.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/platform/x86/lenovo/thinkpad_acpi.c 
> b/drivers/platform/x86/lenovo/thinkpad_acpi.c
> index 445e1403308e..cca76711c547 100644
> --- a/drivers/platform/x86/lenovo/thinkpad_acpi.c
> +++ b/drivers/platform/x86/lenovo/thinkpad_acpi.c
> @@ -8840,6 +8840,7 @@ static const struct tpacpi_quirk 
> fan_quirk_table[] __initconst = {
>  	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_LNV('H', '3', TPACPI_FAN_NS),		/* Edge E330 */
>  	TPACPI_Q_LNV3('N', '1', 'O', TPACPI_FAN_NOFAN),	/* X1 Tablet (2nd 
> gen) */
>  	TPACPI_Q_LNV3('R', '0', 'Q', TPACPI_FAN_DECRPM),/* L480 */
>  	TPACPI_Q_LNV('8', 'F', TPACPI_FAN_TPR),		/* ThinkPad x120e */
>
> base-commit: 0d839570765118029aa8bf4a95444c6a11aacf85
> -- 
> 2.34.1

Afraid this platform is old enough (~14 years I believe?) that I have no easy way to confirm if it's correct or not - so glad you figured it out!

Reviewed-by: Mark Pearson <mpearson-lenovo@xxxxxxxxx>

Mark