[coreboot-gerrit] Change in coreboot[master]: sb,soc/amd, ACPI: Do not override FADT preferred_pm_profile

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/42032 )

Change subject: sb,soc/amd, ACPI: Do not override FADT preferred_pm_profile
......................................................................

sb,soc/amd, ACPI: Do not override FADT preferred_pm_profile

Setting preferred_pm_profile under sb/ or soc/ overrides the
default determined from SYSTEM_TYPE_xx (or possibly
SMBIOS_ENCLOSURE_TYPE with followup work). This is not desireable.

With the overrides removed, AMD platforms will switch from
PM_UNSPECIFIED to PM_DESKTOP as their preferred profile.

Boards need to either select a pre-defined SYSTEM_TYPE_xx or provide
board-specific mainboard_fill_fadt() should they need to change this.

As they already select SYSTEM_TYPE_LAPTOP, following boards
will change to PM_MOBILE:

google/kahlee
  hp/pavilion_m6_1035dx
  lenovo/g505s

Change-Id: I45c4a495a4bf3422adae9e22a6e436adef252e77
Signed-off-by: Kyösti Mälkki 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42032
Tested-by: build bot (Jenkins) 
Reviewed-by: HAOUAS Elyes 
---
M src/soc/amd/picasso/acpi.c
M src/soc/amd/stoneyridge/acpi.c
M src/southbridge/amd/agesa/hudson/fadt.c
M src/southbridge/amd/cimx/sb800/fadt.c
M src/southbridge/amd/pi/hudson/fadt.c
5 files changed, 0 insertions(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  HAOUAS Elyes: Looks good to me, approved

diff --git a/src/soc/amd/picasso/acpi.c b/src/soc/amd/picasso/acpi.c
index 018e57e..fe5879c 100644
--- a/src/soc/amd/picasso/acpi.c
+++ b/src/soc/amd/picasso/acpi.c
@@ -89,7 +89,6 @@
 {
    printk(BIOS_DEBUG, "pm_base: 0x%04x\n", PICASSO_ACPI_IO_BASE);

-	fadt->preferred_pm_profile = PM_UNSPECIFIED;
    fadt->sci_int = 9;		/* IRQ 09 - ACPI SCI */

if (permanent_smi_handler()) {
diff --git a/src/soc/amd/stoneyridge/acpi.c b/src/soc/amd/stoneyridge/acpi.c
index b21f034..ea67aa3 100644
--- a/src/soc/amd/stoneyridge/acpi.c
+++ b/src/soc/amd/stoneyridge/acpi.c
@@ -61,7 +61,6 @@
 {
    printk(BIOS_DEBUG, "pm_base: 0x%04x\n", STONEYRIDGE_ACPI_IO_BASE);

-	fadt->preferred_pm_profile = PM_UNSPECIFIED;
    fadt->sci_int = 9;		/* IRQ 09 - ACPI SCI */

if (permanent_smi_handler()) {
diff --git a/src/southbridge/amd/agesa/hudson/fadt.c b/src/southbridge/amd/agesa/hudson/fadt.c
index cbdcbab..719eb54 100644
--- a/src/southbridge/amd/agesa/hudson/fadt.c
+++ b/src/southbridge/amd/agesa/hudson/fadt.c
@@ -27,7 +27,6 @@
 {
    printk(BIOS_DEBUG, "pm_base: 0x%04x\n", HUDSON_ACPI_IO_BASE);

-	fadt->preferred_pm_profile = PM_UNSPECIFIED;
    fadt->sci_int = 9;		/* HUDSON - IRQ 09 - ACPI SCI */

if (permanent_smi_handler()) {
diff --git a/src/southbridge/amd/cimx/sb800/fadt.c b/src/southbridge/amd/cimx/sb800/fadt.c
index 5ddb2f4..ba277f9 100644
--- a/src/southbridge/amd/cimx/sb800/fadt.c
+++ b/src/southbridge/amd/cimx/sb800/fadt.c
@@ -33,7 +33,6 @@

printk(BIOS_DEBUG, "ACPI_BLK_BASE: 0x%04x\n", ACPI_BLK_BASE);

-	fadt->preferred_pm_profile = PM_DESKTOP;
    fadt->sci_int = 9;		/* HUDSON 1 - IRQ 09 - ACPI SCI */

val = PM1_EVT_BLK_ADDRESS;
diff --git a/src/southbridge/amd/pi/hudson/fadt.c b/src/southbridge/amd/pi/hudson/fadt.c
index 5bc6fbd..8ed0330 100644
--- a/src/southbridge/amd/pi/hudson/fadt.c
+++ b/src/southbridge/amd/pi/hudson/fadt.c
@@ -27,7 +27,6 @@
 {
    printk(BIOS_DEBUG, "pm_base: 0x%04x\n", HUDSON_ACPI_IO_BASE);

-	fadt->preferred_pm_profile = PM_UNSPECIFIED;
    fadt->sci_int = 9;		/* HUDSON - IRQ 09 - ACPI SCI */

if (permanent_smi_handler()) {