summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@silverbackltd.com>2018-02-02 10:42:22 -0700
committerMartin Roth <martinroth@google.com>2018-02-05 21:21:26 +0000
commite8d8b064e833f12a5a65c57a750bd8356fbe08a9 (patch)
treebaca5c2569661f6b1d5a621df9a2aa9c88b72082 /src
parent3e9694ef48cc2ad6b5c5e8c59a244d862fa037ae (diff)
downloadcoreboot-e8d8b064e833f12a5a65c57a750bd8356fbe08a9.tar.xz
soc/amd/stoneyridge/acpi/sb_pci0_fch.asl: Fix instability
A file that has several methods cannot be included inside a method. It has to be included inside a scope, but not inside a method or it'll cause problems (instability). There is an ugly construction in method _INI. It's needed because if AmdImc is not included then the call to ITZE would break the build. BUG=b:62200858 TEST=Build kahlee. Change-Id: If6c877df5a87df1b348de92868b91eed4a76de55 Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/23573 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Justin TerAvest <teravest@chromium.org> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl
index 216b57c7ce..e767fddbc9 100644
--- a/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl
+++ b/src/soc/amd/stoneyridge/acpi/sb_pci0_fch.asl
@@ -130,6 +130,10 @@ Method(_CRS, 0) {
Return(CRES) /* note to change the Name buffer */
} /* end of Method(_SB.PCI0._CRS) */
+#if IS_ENABLED(CONFIG_STONEYRIDGE_IMC_FWM)
+ #include "acpi/AmdImc.asl"
+#endif
+
/*
*
* FIRST METHOD CALLED UPON BOOT
@@ -156,8 +160,6 @@ Method(_INI, 0) {
OSFL()
#if IS_ENABLED(CONFIG_STONEYRIDGE_IMC_FWM)
- /* TODO: It is unstable. */
- #include "acpi/AmdImc.asl"
#if IS_ENABLED(CONFIG_ACPI_ENABLE_THERMAL_ZONE)
ITZE() /* enable IMC Fan Control*/
#endif