From c42e28f077cbf108085dff8d847a74d23782d703 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Kope=C4=87?= Date: Thu, 19 Oct 2023 16:23:16 +0200 Subject: [PATCH] mb/protectli/vault_cml: use combo v1/v2 FSP MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Also switch configs to use combo v1/v2 FSP The reason for this change is to simplify configuration - instead of multiple targets for VP4630 and VP4650 or VP4670, it's now possible to have one target covering all VP46x0. Change-Id: I1a6f6e873e4ec35b9777dc17c0495151348d1d88 Signed-off-by: Michał Kopeć Reviewed-on: https://review.coreboot.org/c/coreboot/+/81963 Reviewed-by: Maciej Pijanowski Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- ...protectli_vp4670 => config.protectli_cml_vp46xx} | 2 +- configs/config.protectli_vp4630_vp4650 | 13 ------------- src/mainboard/protectli/vault_cml/Kconfig | 8 +++----- src/mainboard/protectli/vault_cml/Kconfig.name | 7 ++----- 4 files changed, 6 insertions(+), 24 deletions(-) rename configs/{config.protectli_vp4670 => config.protectli_cml_vp46xx} (91%) delete mode 100644 configs/config.protectli_vp4630_vp4650 diff --git a/configs/config.protectli_vp4670 b/configs/config.protectli_cml_vp46xx similarity index 91% rename from configs/config.protectli_vp4670 rename to configs/config.protectli_cml_vp46xx index a9decdaf6f..11642325db 100644 --- a/configs/config.protectli_vp4670 +++ b/configs/config.protectli_cml_vp46xx @@ -1,5 +1,5 @@ CONFIG_VENDOR_PROTECTLI=y -CONFIG_BOARD_PROTECTLI_VP4670=y +CONFIG_BOARD_PROTECTLI_VP46XX=y CONFIG_TPM_MEASURED_BOOT=y CONFIG_SMMSTORE_SIZE=0x40000 CONFIG_TPM2=y diff --git a/configs/config.protectli_vp4630_vp4650 b/configs/config.protectli_vp4630_vp4650 deleted file mode 100644 index af50a7e12e..0000000000 --- a/configs/config.protectli_vp4630_vp4650 +++ /dev/null @@ -1,13 +0,0 @@ -CONFIG_VENDOR_PROTECTLI=y -CONFIG_BOARD_PROTECTLI_VP4630_VP4650=y -CONFIG_TPM_MEASURED_BOOT=y -CONFIG_SMMSTORE_SIZE=0x40000 -CONFIG_TPM2=y -CONFIG_POST_IO_PORT=0x80 -CONFIG_PAYLOAD_EDK2=y -CONFIG_EDK2_BOOT_MANAGER_ESCAPE=y -CONFIG_EDK2_BOOT_TIMEOUT=6 -CONFIG_EDK2_FOLLOW_BGRT_SPEC=y -# CONFIG_EDK2_FULL_SCREEN_SETUP is not set -CONFIG_EDK2_SD_MMC_TIMEOUT=10 -CONFIG_EDK2_SERIAL_SUPPORT=y diff --git a/src/mainboard/protectli/vault_cml/Kconfig b/src/mainboard/protectli/vault_cml/Kconfig index 2389520ef8..87de08829a 100644 --- a/src/mainboard/protectli/vault_cml/Kconfig +++ b/src/mainboard/protectli/vault_cml/Kconfig @@ -1,11 +1,12 @@ ## SPDX-License-Identifier: GPL-2.0-only -if BOARD_PROTECTLI_VP4630_VP4650 || BOARD_PROTECTLI_VP4670 +if BOARD_PROTECTLI_VP46XX config BOARD_SPECIFIC_OPTIONS def_bool y select BOARD_ROMSIZE_KB_16384 select GFX_GMA_IGNORE_PRESENCE_STRAPS + select SOC_INTEL_COMETLAKE_1_2 select HAVE_ACPI_RESUME select HAVE_ACPI_TABLES select HAVE_CMOS_DEFAULT @@ -15,8 +16,6 @@ config BOARD_SPECIFIC_OPTIONS select MAINBOARD_HAS_LIBGFXINIT select MEMORY_MAPPED_TPM select SEABIOS_ADD_SERCON_PORT_FILE if PAYLOAD_SEABIOS - select SOC_INTEL_COMETLAKE_1 if BOARD_PROTECTLI_VP4670 - select SOC_INTEL_COMETLAKE_2 if BOARD_PROTECTLI_VP4630_VP4650 select SOC_INTEL_COMMON_BLOCK_HDA_VERB select SPI_FLASH_MACRONIX select SUPERIO_ITE_IT8784E @@ -25,8 +24,7 @@ config MAINBOARD_DIR default "protectli/vault_cml" config MAINBOARD_PART_NUMBER - default "VP4630/VP4650" if BOARD_PROTECTLI_VP4630_VP4650 - default "VP4670" if BOARD_PROTECTLI_VP4670 + default "VP46XX" config MAINBOARD_FAMILY default "Vault Pro" diff --git a/src/mainboard/protectli/vault_cml/Kconfig.name b/src/mainboard/protectli/vault_cml/Kconfig.name index f0f3257f05..1aaac81544 100644 --- a/src/mainboard/protectli/vault_cml/Kconfig.name +++ b/src/mainboard/protectli/vault_cml/Kconfig.name @@ -1,7 +1,4 @@ ## SPDX-License-Identifier: GPL-2.0-only -config BOARD_PROTECTLI_VP4630_VP4650 - bool "VP4630/VP4650" - -config BOARD_PROTECTLI_VP4670 - bool "VP4670" +config BOARD_PROTECTLI_VP46XX + bool "VP4630/VP4650/VP4670"