| Message ID | 20220701023328.2783-2-mario.limonciello@amd.com |
|---|---|
| State | New |
| Headers | show |
| Series | None | expand |
On Fri, Jul 1, 2022 at 4:33 AM Mario Limonciello <mario.limonciello@amd.com> wrote: > > Drop the direct check from the FADT and use the helper instead. > > Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> > --- > drivers/acpi/x86/s2idle.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/acpi/x86/s2idle.c b/drivers/acpi/x86/s2idle.c > index 2963229062f8..b1483d5092c1 100644 > --- a/drivers/acpi/x86/s2idle.c > +++ b/drivers/acpi/x86/s2idle.c > @@ -369,7 +369,7 @@ static int lps0_device_attach(struct acpi_device *adev, > if (lps0_device_handle) > return 0; > > - if (!(acpi_gbl_FADT.flags & ACPI_FADT_LOW_POWER_S0)) > + if (!pm_suspend_preferred_s2idle()) This needs to be checked in a different place in this function. Let me cut a patch for that. > return 0; > > if (acpi_s2idle_vendor_amd()) { > --
diff --git a/drivers/acpi/x86/s2idle.c b/drivers/acpi/x86/s2idle.c index 2963229062f8..b1483d5092c1 100644 --- a/drivers/acpi/x86/s2idle.c +++ b/drivers/acpi/x86/s2idle.c @@ -369,7 +369,7 @@ static int lps0_device_attach(struct acpi_device *adev, if (lps0_device_handle) return 0; - if (!(acpi_gbl_FADT.flags & ACPI_FADT_LOW_POWER_S0)) + if (!pm_suspend_preferred_s2idle()) return 0; if (acpi_s2idle_vendor_amd()) {
Drop the direct check from the FADT and use the helper instead. Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> --- drivers/acpi/x86/s2idle.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)