Skip to content

Commit f59a905

Browse files
alexdeucherrafaeljw
authored andcommitted
ACPI: PM: s2idle: Add missing LPS0 functions for AMD
These are supposedly not required for AMD platforms, but at least some HP laptops seem to require it to properly turn off the keyboard backlight. Based on a patch from Marcin Bachry <hegel666@gmail.com>. Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1230 Reviewed-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent d07f6ca commit f59a905

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

drivers/acpi/x86/s2idle.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ static const struct acpi_device_id lps0_device_ids[] = {
4242

4343
/* AMD */
4444
#define ACPI_LPS0_DSM_UUID_AMD "e3f32452-febc-43ce-9039-932122d37721"
45+
#define ACPI_LPS0_ENTRY_AMD 2
46+
#define ACPI_LPS0_EXIT_AMD 3
4547
#define ACPI_LPS0_SCREEN_OFF_AMD 4
4648
#define ACPI_LPS0_SCREEN_ON_AMD 5
4749

@@ -408,6 +410,7 @@ int acpi_s2idle_prepare_late(void)
408410

409411
if (acpi_s2idle_vendor_amd()) {
410412
acpi_sleep_run_lps0_dsm(ACPI_LPS0_SCREEN_OFF_AMD);
413+
acpi_sleep_run_lps0_dsm(ACPI_LPS0_ENTRY_AMD);
411414
} else {
412415
acpi_sleep_run_lps0_dsm(ACPI_LPS0_SCREEN_OFF);
413416
acpi_sleep_run_lps0_dsm(ACPI_LPS0_ENTRY);
@@ -422,6 +425,7 @@ void acpi_s2idle_restore_early(void)
422425
return;
423426

424427
if (acpi_s2idle_vendor_amd()) {
428+
acpi_sleep_run_lps0_dsm(ACPI_LPS0_EXIT_AMD);
425429
acpi_sleep_run_lps0_dsm(ACPI_LPS0_SCREEN_ON_AMD);
426430
} else {
427431
acpi_sleep_run_lps0_dsm(ACPI_LPS0_EXIT);

0 commit comments

Comments
 (0)