HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/31760
Change subject: soc/intel/apollolake: Fix wrong IS_ENABLED test ......................................................................
soc/intel/apollolake: Fix wrong IS_ENABLED test
Add missing "CONFIG_" to SOC_INTEL_COMMON_BLOCK_SGX.
Change-Id: Id9b70e9552af282f4f378929987150e936a22487 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr --- M src/soc/intel/apollolake/cpu.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/31760/1
diff --git a/src/soc/intel/apollolake/cpu.c b/src/soc/intel/apollolake/cpu.c index d1c5f6f..a08f1f0 100644 --- a/src/soc/intel/apollolake/cpu.c +++ b/src/soc/intel/apollolake/cpu.c @@ -73,7 +73,7 @@ /* Clear out pending MCEs */ /* TODO(adurbin): Some of these banks are core vs package scope. For now every CPU clears every bank. */ - if (IS_ENABLED(SOC_INTEL_COMMON_BLOCK_SGX) || + if (IS_ENABLED(CONFIG_SOC_INTEL_COMMON_BLOCK_SGX) || acpi_get_sleep_type() == ACPI_S5) mca_configure(NULL);
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31760 )
Change subject: soc/intel/apollolake: Fix wrong IS_ENABLED test ......................................................................
Patch Set 1: Code-Review+1
Julius does the same in I79a0fca65853798ee45c3779b437864ba3cf2b1e [1].
[1]: https://review.coreboot.org/c/31745/
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31760 )
Change subject: soc/intel/apollolake: Fix wrong IS_ENABLED test ......................................................................
Patch Set 1:
Thank you.
HAOUAS Elyes has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/31760 )
Change subject: soc/intel/apollolake: Fix wrong IS_ENABLED test ......................................................................
Abandoned
see https://review.coreboot.org/#/c/coreboot/+/31745/