Change in coreboot[master]: configs: Add builder config to create a working Cedar Island CRB

[Andrey Petrov (Code Review)]

Andrey Petrov has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

configs: Add builder config to create a working Cedar Island CRB

Change-Id: I2a2de7ccb96996211c45da3f9ec9bf6f71cc0c89
Signed-off-by: Andrey Petrov 
---
A configs/builder/config.intel.cpx.crb
1 file changed, 16 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/39783/1

diff --git a/configs/builder/config.intel.cpx.crb b/configs/builder/config.intel.cpx.crb
new file mode 100644
index 0000000..2a2d4a0
--- /dev/null
+++ b/configs/builder/config.intel.cpx.crb
@@ -0,0 +1,16 @@
+# type this to get working .config:
+#   make defconfig KBUILD_DEFCONFIG=configs/builder/config.intel.cpx.crb
+
+CONFIG_VENDOR_INTEL=y
+CONFIG_BOARD_INTEL_CEDARISLAND_CRB=y
+CONFIG_HAVE_IFD_BIN=y
+CONFIG_HAVE_ME_BIN=y
+CONFIG_DO_NOT_TOUCH_DESCRIPTOR_REGION=y
+CONFIG_PAYLOAD_NONE=y
+CONFIG_USE_CPU_MICROCODE_CBFS_BINS=y
+CONFIG_CPU_MICROCODE_CBFS_EXTERNAL_BINS=y
+CONFIG_CPU_UCODE_BINARIES="3rdparty/blobs/mainboard/intel/cedarisland_crb/ucode-05-06-5a"
+CONFIG_ADD_FSP_BINARIES=y
+CONFIG_FSP_T_FILE="3rdparty/blobs/mainboard/intel/cedarisland_crb/Server_T.fd"
+CONFIG_FSP_M_FILE="3rdparty/blobs/mainboard/intel/cedarisland_crb/Server_M.fd"
+CONFIG_FSP_S_FILE="3rdparty/blobs/mainboard/intel/cedarisland_crb/Server_S.fd"

[Andrey Petrov (Code Review)]

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/39783

to look at the new patch set (#3).

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

configs: Add builder config to create a working Cedar Island CRB

Change-Id: I2a2de7ccb96996211c45da3f9ec9bf6f71cc0c89
Signed-off-by: Andrey Petrov 
---
A configs/builder/config.intel.cpx.crb
1 file changed, 15 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/39783/3

[Andrey Petrov (Code Review)]

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/39783

to look at the new patch set (#4).

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

configs: Add builder config to create a working Cedar Island CRB

Change-Id: I2a2de7ccb96996211c45da3f9ec9bf6f71cc0c89
Signed-off-by: Andrey Petrov 
---
A configs/builder/config.intel.cpx.crb
1 file changed, 17 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/39783/4

[David Hendricks (Code Review)]

David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

Patch Set 4: Code-Review+2

[Maxim Polyakov (Code Review)]

Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

Patch Set 9: Code-Review+2

[Andrey Petrov (Code Review)]

Andrey Petrov has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

configs: Add builder config to create a working Cedar Island CRB

Change-Id: I2a2de7ccb96996211c45da3f9ec9bf6f71cc0c89
Signed-off-by: Andrey Petrov 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39783
Tested-by: build bot (Jenkins) 
Reviewed-by: Maxim Polyakov 
Reviewed-by: David Hendricks 
---
A configs/builder/config.intel.cpx.crb
1 file changed, 17 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  David Hendricks: Looks good to me, approved
  Maxim Polyakov: Looks good to me, approved

diff --git a/configs/builder/config.intel.cpx.crb b/configs/builder/config.intel.cpx.crb
new file mode 100644
index 0000000..b825a92
--- /dev/null
+++ b/configs/builder/config.intel.cpx.crb
@@ -0,0 +1,17 @@
+# type this to get working .config:
+#   make defconfig KBUILD_DEFCONFIG=configs/builder/config.intel.cpx.crb
+
+CONFIG_VENDOR_INTEL=y
+CONFIG_BOARD_INTEL_CEDARISLAND_CRB=y
+CONFIG_HAVE_IFD_BIN=y
+CONFIG_HAVE_ME_BIN=y
+CONFIG_DO_NOT_TOUCH_DESCRIPTOR_REGION=y
+CONFIG_USE_CPU_MICROCODE_CBFS_BINS=y
+CONFIG_CPU_MICROCODE_CBFS_EXTERNAL_BINS=y
+CONFIG_CPU_UCODE_BINARIES="site-local/cedarisland_crb/ucode-05-06-5a"
+CONFIG_ADD_FSP_BINARIES=y
+CONFIG_FSP_T_FILE="site-local/cedarisland_crb/Server_T.fd"
+CONFIG_FSP_M_FILE="site-local/cedarisland_crb/Server_M.fd"
+CONFIG_FSP_S_FILE="site-local/cedarisland_crb/Server_S.fd"
+CONFIG_ME_BIN_PATH="site-local/cedarisland_crb/me.bin"
+CONFIG_IFD_BIN_PATH="site-local/cedarisland_crb/descriptor.bin"

[9elements QA (Code Review)]

9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

Patch Set 12:

Automatic boot test returned (PASS/FAIL/TOTAL): 5/0/5
Emulation targets:
EMULATION_QEMU_X86_Q35 using payload TianoCore : SUCCESS : https://lava.9esec.io/r/1820
EMULATION_QEMU_X86_Q35 using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1819
EMULATION_QEMU_X86_I440FX using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1818
Non-emulation targets:
HP_COMPAQ_8200_ELITE_SFF_PC using payload TianoCore : SUCCESS : https://lava.9esec.io/r/1822
HP_COMPAQ_8200_ELITE_SFF_PC using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1821

Please note: This test is under development and might not be accurate at all!

[9elements QA (Code Review)]

9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39783 )

Change subject: configs: Add builder config to create a working Cedar Island CRB
......................................................................

Patch Set 12:

Automatic boot test returned (PASS/FAIL/TOTAL): 5/0/5
Emulation targets:
EMULATION_QEMU_X86_Q35 using payload TianoCore : SUCCESS : https://lava.9esec.io/r/1825
EMULATION_QEMU_X86_Q35 using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1824
EMULATION_QEMU_X86_I440FX using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1823
Non-emulation targets:
HP_COMPAQ_8200_ELITE_SFF_PC using payload TianoCore : SUCCESS : https://lava.9esec.io/r/1827
HP_COMPAQ_8200_ELITE_SFF_PC using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/1826

Please note: This test is under development and might not be accurate at all!