Patrick Georgi submitted this change.

View Change

Approvals: build bot (Jenkins): Verified Patrick Rudolph: Looks good to me, approved
mb/supermicro/x11-lga1151-series: fix cmos layout and add default config

This fixes the warning that power_on_after_fail could not be found,
adds a default config and adds the parameter hyper_threading.

Change-Id: I10b0aa71fa7916b01e93e16cbd81e427fd14f6a4
Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35526
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M src/mainboard/supermicro/x11-lga1151-series/Kconfig
A src/mainboard/supermicro/x11-lga1151-series/cmos.default
M src/mainboard/supermicro/x11-lga1151-series/cmos.layout
3 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/mainboard/supermicro/x11-lga1151-series/Kconfig b/src/mainboard/supermicro/x11-lga1151-series/Kconfig
index 6fc9aed..541b23d 100644
--- a/src/mainboard/supermicro/x11-lga1151-series/Kconfig
+++ b/src/mainboard/supermicro/x11-lga1151-series/Kconfig
@@ -4,6 +4,7 @@
select HAVE_ACPI_RESUME
select HAVE_ACPI_TABLES
select HAVE_OPTION_TABLE
+ select HAVE_CMOS_DEFAULT
select SOC_INTEL_KABYLAKE
select SKYLAKE_SOC_PCH_H
select MAINBOARD_HAS_LPC_TPM
diff --git a/src/mainboard/supermicro/x11-lga1151-series/cmos.default b/src/mainboard/supermicro/x11-lga1151-series/cmos.default
new file mode 100644
index 0000000..d564953
--- /dev/null
+++ b/src/mainboard/supermicro/x11-lga1151-series/cmos.default
@@ -0,0 +1,3 @@
+boot_option=Fallback
+debug_level=Debug
+power_on_after_fail=Enable
diff --git a/src/mainboard/supermicro/x11-lga1151-series/cmos.layout b/src/mainboard/supermicro/x11-lga1151-series/cmos.layout
index 201ca33..03aea17 100644
--- a/src/mainboard/supermicro/x11-lga1151-series/cmos.layout
+++ b/src/mainboard/supermicro/x11-lga1151-series/cmos.layout
@@ -17,7 +17,6 @@
entries

#start-bit length config config-ID name
-
0 120 r 0 reserved_memory

# -----------------------------------------------------------------
@@ -28,8 +27,20 @@
# -----------------------------------------------------------------
# coreboot config options: console
395 4 e 6 debug_level
+
+# -----------------------------------------------------------------
+# coreboot config options: cpu
+400 1 e 2 hyper_threading
+
+# -----------------------------------------------------------------
+# coreboot config options: southbridge
+409 2 e 7 power_on_after_fail
+
+# -----------------------------------------------------------------
+# coreboot config options: bootloader
448 128 r 0 vbnv

+# -----------------------------------------------------------------
# coreboot config options: check sums
984 16 h 0 check_sum


To view, visit change 35526. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I10b0aa71fa7916b01e93e16cbd81e427fd14f6a4
Gerrit-Change-Number: 35526
Gerrit-PatchSet: 18
Gerrit-Owner: Michael Niewöhner
Gerrit-Reviewer: Michael Niewöhner
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged