Naresh Solanki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/79524?usp=email )
Change subject: acpi/acpi_pm.c: Default PM profile ......................................................................
acpi/acpi_pm.c: Default PM profile
ACPI default PM profile depends on SMBIOS function smbios_mainboard_enclosure_type.
Add condition check to make sure GENERATE_SMBIOS_TABLES is selected before calling any smbios specific function call. And if not selected then default to PM_UNSPECIFIED.
Change-Id: Ie01af5adbc781bb6e0f37a25ec73b1cab4039781 Signed-off-by: Naresh Solanki naresh.solanki@9elements.com --- M src/acpi/acpi_pm.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/79524/1
diff --git a/src/acpi/acpi_pm.c b/src/acpi/acpi_pm.c index 722d8f6..afa9a31 100644 --- a/src/acpi/acpi_pm.c +++ b/src/acpi/acpi_pm.c @@ -21,6 +21,9 @@ /* Default mapping to ACPI FADT preferred_pm_profile field. */ uint8_t acpi_get_preferred_pm_profile(void) { + if (!CONFIG(GENERATE_SMBIOS_TABLES)) + return PM_UNSPECIFIED; + switch (smbios_mainboard_enclosure_type()) { case SMBIOS_ENCLOSURE_LAPTOP: case SMBIOS_ENCLOSURE_CONVERTIBLE: