Change in coreboot[master]: mb/prodrive/hermes: Disable ACPI S3 and S4 with SPS

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/54058 )

Change subject: mb/prodrive/hermes: Disable ACPI S3 and S4 with SPS
......................................................................

mb/prodrive/hermes: Disable ACPI S3 and S4 with SPS

Hermes can be used with either CSME or SPS firmware. However, the SPS
(Server Platform Services) firmware does not support ACPI S3 and S4
sleep states, and coreboot should not report S3 and S4 as supported.

Add a Kconfig option to be selected when building coreboot to use with
SPS firmware, which allows disabling ACPI S3 and S4 sleep state support.

Change-Id: I9d0fa8530e198e86415f92da6719d2fb0d2401ec
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/54058
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
Reviewed-by: Michael Niewöhner 
---
M src/mainboard/prodrive/hermes/Kconfig
1 file changed, 9 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved
  Michael Niewöhner: Looks good to me, approved

diff --git a/src/mainboard/prodrive/hermes/Kconfig b/src/mainboard/prodrive/hermes/Kconfig
index 178acce..ac497e6 100644
--- a/src/mainboard/prodrive/hermes/Kconfig
+++ b/src/mainboard/prodrive/hermes/Kconfig
@@ -14,10 +14,18 @@
    select INTEL_GMA_HAVE_VBT
    select SOC_INTEL_COMMON_BLOCK_HDA_VERB
    select ONBOARD_VGA_IS_PRIMARY
-	select HAVE_ACPI_RESUME
+	select HAVE_ACPI_RESUME if !HERMES_USES_SPS_FIRMWARE
+	select DISABLE_ACPI_HIBERNATE if HERMES_USES_SPS_FIRMWARE

if BOARD_PRODRIVE_HERMES_BASEBOARD

+config HERMES_USES_SPS_FIRMWARE
+	bool "Build for use with SPS (Server Platform Services) firmware"
+	help
+	  Depending on the intended use case, the Hermes mainboard can use either
+	  CSME or SPS firmware SKUs. Choose this option if using SPS firmware, as
+	  SPS doesn't support ACPI S3 and S4 sleep states.
+
 config MAINBOARD_FAMILY
    string
    default "PRODRIVE_HERMES_SERIES"