From d48d72c9ceeccb56d89f13fa2f78d7c6b8666a0b Mon Sep 17 00:00:00 2001 From: Evgeny Sorokin Date: Sat, 10 Feb 2024 03:32:41 +0300 Subject: [PATCH] ec/lenovo/h8/acpi: Support pulsing LEDLOGO on Haswell ThinkPads The name LEDLOGO comes from schematics. It's the red indicator, embedded in the dot of the 'i' of the ThinkPad logo on laptop's lid. In vendor firmware, this led starts fading in-and-out, or, in other words, pulsing, when laptop is put to S3. It helps to determine whether the laptop is in S3 just by taking a look at the logo. As of now, coreboot doesn't do anything with this particular indicator, it's always in enabled (on) state, which is not very convenient. This patch fixes it. Tested on T440p. Change-Id: I85fb69c8c1bed8635a1b31e9b8385c7036bb46dd Signed-off-by: Evgeny Zinoviev Reviewed-on: https://review.coreboot.org/c/coreboot/+/80437 Reviewed-by: Felix Singer Reviewed-by: Nicholas Chin Tested-by: build bot (Jenkins) --- src/ec/lenovo/h8/Kconfig | 4 ++++ src/ec/lenovo/h8/acpi/systemstatus.asl | 16 ++++++++++++++++ src/mainboard/lenovo/haswell/Kconfig | 1 + 3 files changed, 21 insertions(+) diff --git a/src/ec/lenovo/h8/Kconfig b/src/ec/lenovo/h8/Kconfig index 57d7da6c69..6311d54163 100644 --- a/src/ec/lenovo/h8/Kconfig +++ b/src/ec/lenovo/h8/Kconfig @@ -52,6 +52,10 @@ config H8_HAS_PRIMARY_FN_KEYS bool default n +config H8_HAS_LEDLOGO + bool + default n + config THINKPADEC_HKEY_EISAID string default "IBM0068" diff --git a/src/ec/lenovo/h8/acpi/systemstatus.asl b/src/ec/lenovo/h8/acpi/systemstatus.asl index 7598154282..80544499ee 100644 --- a/src/ec/lenovo/h8/acpi/systemstatus.asl +++ b/src/ec/lenovo/h8/acpi/systemstatus.asl @@ -12,6 +12,10 @@ Scope (\_SI) \_SB.PCI0.LPCB.EC.TLED(0x00) /* suspend TLED off */ \_SB.PCI0.LPCB.EC.TLED(0x07) +#if CONFIG(H8_HAS_LEDLOGO) + /* logo TLED off */ + \_SB.PCI0.LPCB.EC.TLED(0x0a) +#endif } If (Arg0 == 1) { @@ -21,6 +25,10 @@ Scope (\_SI) \_SB.PCI0.LPCB.EC.TLED(0x80) /* suspend TLED off */ \_SB.PCI0.LPCB.EC.TLED(0x07) +#if CONFIG(H8_HAS_LEDLOGO) + /* logo TLED on */ + \_SB.PCI0.LPCB.EC.TLED(0x8a) +#endif } If (Arg0 == 2) { @@ -30,6 +38,10 @@ Scope (\_SI) \_SB.PCI0.LPCB.EC.TLED(0x80) /* suspend LED blinking */ \_SB.PCI0.LPCB.EC.TLED(0xc7) +#if CONFIG(H8_HAS_LEDLOGO) + /* logo TLED on */ + \_SB.PCI0.LPCB.EC.TLED(0x8a) +#endif } If (Arg0 == 3) { @@ -39,6 +51,10 @@ Scope (\_SI) \_SB.PCI0.LPCB.EC.TLED(0xa0) /* suspend TLED on */ \_SB.PCI0.LPCB.EC.TLED(0x87) +#if CONFIG(H8_HAS_LEDLOGO) + /* logo TLED pulsing */ + \_SB.PCI0.LPCB.EC.TLED(0xaa) +#endif } } } diff --git a/src/mainboard/lenovo/haswell/Kconfig b/src/mainboard/lenovo/haswell/Kconfig index 486baf05ca..6e3955646e 100644 --- a/src/mainboard/lenovo/haswell/Kconfig +++ b/src/mainboard/lenovo/haswell/Kconfig @@ -4,6 +4,7 @@ config BOARD_LENOVO_HASWELL_COMMON select EC_LENOVO_H8 select EC_LENOVO_PMH7 select H8_HAS_BAT_THRESHOLDS_IMPL + select H8_HAS_LEDLOGO select H8_HAS_PRIMARY_FN_KEYS select HAVE_ACPI_RESUME select HAVE_ACPI_TABLES