platform/x86: amd-pmc: Move to later in the suspend process
authorMario Limonciello <mario.limonciello@amd.com>
Thu, 17 Mar 2022 14:14:43 +0000 (09:14 -0500)
committerHans de Goede <hdegoede@redhat.com>
Thu, 17 Mar 2022 18:47:24 +0000 (19:47 +0100)
commitb1f66033cd4e9ce8cbe2c74c98d4e04c0b2d7b40
tree26ae5b16ded6e26544e1a70392bc697fe5311999
parent20e1d6402a71dba7ad2b81f332a3c14c7d3b939b
platform/x86: amd-pmc: Move to later in the suspend process

The `OS_HINT` message is supposed to indicate that everything else
that is supposed to go into the deepest state has done so.

This assumption is invalid as:
1) The CPUs will still go in and out of the deepest state
2) Other devices may still run their `noirq` suspend routines
3) The LPS0 ACPI device will still run

To more closely mirror how this works on other operating systems,
move the `amd-pmc` suspend to the very last thing before the s2idle
loop via an lps0 callback.

Fixes: 8d89835b0467 ("PM: suspend: Do not pause cpuidle in the suspend-to-idle path")
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/r/20220317141445.6498-2-mario.limonciello@amd.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/amd-pmc.c