Skip to content

Commit be892e9

Browse files
committed
platform/x86: thinkpad_acpi: Add lid_logo_dot to the list of safe LEDs
There have been various bugs / forum threads about allowing control of the LED in the ThinkPad logo on the lid of various models. This seems to be something which users want to control and there really is no reason to require setting CONFIG_THINKPAD_ACPI_UNSAFE_LEDS for this. The lid-logo-dot is LED number 10, so change the name of the 10th led from unknown_led2 to lid_logo_dot and add it to the TPACPI_SAFE_LEDS mask. Link: https://www.reddit.com/r/thinkpad/comments/7n8eyu/thinkpad_led_control_under_gnulinux/ BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1943318 Signed-off-by: Hans de Goede <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent b68f8a1 commit be892e9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/platform/x86/thinkpad_acpi.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5728,11 +5728,11 @@ static const char * const tpacpi_led_names[TPACPI_LED_NUMLEDS] = {
57285728
"tpacpi::standby",
57295729
"tpacpi::dock_status1",
57305730
"tpacpi::dock_status2",
5731-
"tpacpi::unknown_led2",
5731+
"tpacpi::lid_logo_dot",
57325732
"tpacpi::unknown_led3",
57335733
"tpacpi::thinkvantage",
57345734
};
5735-
#define TPACPI_SAFE_LEDS 0x1081U
5735+
#define TPACPI_SAFE_LEDS 0x1481U
57365736

57375737
static inline bool tpacpi_is_led_restricted(const unsigned int led)
57385738
{

0 commit comments

Comments
 (0)