summaryrefslogtreecommitdiff
path: root/src/mainboard
diff options
context:
space:
mode:
authorFelix Singer <felixsinger@posteo.net>2021-07-12 19:01:23 +0200
committerFelix Singer <felixsinger@posteo.net>2021-10-20 09:55:21 +0000
commita61e781e4b7a9c2b918401d0e61a7c86e888ae01 (patch)
treeddb1ed6eb3540dc498f32e5f1825e42950028556 /src/mainboard
parentd23981998c81438a2eed6a1c2ba2d4ae54b090f0 (diff)
mb/intel/jasperlake_rvp: Rework Kconfig
Rework Kconfig file that each variant has its own config option with their specific selects / configuration and move common selects to `BOARD_INTEL_JASPERLAKE_RVP_COMMON`, which is used as base for each variant. Also, move selects from Kconfig.name to Kconfig that the configuration is at one place and not distributed over two files. Built each variant with `BUILD_TIMELESS=1` and all generated coreboot.rom files remain identical. Excluded the .config file by disabling `INCLUDE_CONFIG_FILE` to make this reproducible. Change-Id: Ic7552195ed5a3ae6ab8e456d7d38d5539a052009 Signed-off-by: Felix Singer <felixsinger@posteo.net> Reviewed-on: https://review.coreboot.org/c/coreboot/+/56222 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Diffstat (limited to 'src/mainboard')
-rw-r--r--src/mainboard/intel/jasperlake_rvp/Kconfig30
-rw-r--r--src/mainboard/intel/jasperlake_rvp/Kconfig.name6
2 files changed, 21 insertions, 15 deletions
diff --git a/src/mainboard/intel/jasperlake_rvp/Kconfig b/src/mainboard/intel/jasperlake_rvp/Kconfig
index 9409432eb5..a5b5c39fd4 100644
--- a/src/mainboard/intel/jasperlake_rvp/Kconfig
+++ b/src/mainboard/intel/jasperlake_rvp/Kconfig
@@ -1,25 +1,37 @@
-if BOARD_INTEL_JASPERLAKE_RVP || BOARD_INTEL_JASPERLAKE_RVP_EXT_EC
-
-config BOARD_SPECIFIC_OPTIONS
- def_bool y
+config BOARD_INTEL_JASPERLAKE_RVP_COMMON
+ def_bool n
select BOARD_ROMSIZE_KB_16384
select DRIVERS_I2C_DA7219
- select DRIVERS_I2C_HID
- select DRIVERS_INTEL_DPTF
select DRIVERS_I2C_GENERIC
+ select DRIVERS_I2C_HID
select DRIVERS_I2C_MAX98373
+ select DRIVERS_INTEL_DPTF
select DRIVERS_INTEL_MIPI_CAMERA
select DRIVERS_SPI_ACPI
select DRIVERS_USB_ACPI
select EC_ACPI
- select HAVE_SPD_IN_CBFS
select HAVE_ACPI_RESUME
select HAVE_ACPI_TABLES
+ select HAVE_SPD_IN_CBFS
select MAINBOARD_HAS_CHROMEOS
- select SOC_INTEL_COMMON_BLOCK_IPU
- select SOC_INTEL_JASPERLAKE
select SOC_INTEL_COMMON_BLOCK_DTT
+ select SOC_INTEL_COMMON_BLOCK_IPU
select SOC_INTEL_CSE_LITE_SKU
+ select SOC_INTEL_JASPERLAKE
+
+config BOARD_INTEL_JASPERLAKE_RVP
+ select BOARD_INTEL_JASPERLAKE_RVP_COMMON
+ select DRIVERS_UART_8250IO
+ select MAINBOARD_USES_IFD_EC_REGION
+
+config BOARD_INTEL_JASPERLAKE_RVP_EXT_EC
+ select BOARD_INTEL_JASPERLAKE_RVP_COMMON
+ select EC_GOOGLE_CHROMEEC
+ select EC_GOOGLE_CHROMEEC_LPC
+ select EC_GOOGLE_CHROMEEC_SWITCHES if VBOOT
+ select INTEL_LPSS_UART_FOR_CONSOLE
+
+if BOARD_INTEL_JASPERLAKE_RVP_COMMON
config MAINBOARD_DIR
default "intel/jasperlake_rvp"
diff --git a/src/mainboard/intel/jasperlake_rvp/Kconfig.name b/src/mainboard/intel/jasperlake_rvp/Kconfig.name
index 68419ac1f5..1a56f05485 100644
--- a/src/mainboard/intel/jasperlake_rvp/Kconfig.name
+++ b/src/mainboard/intel/jasperlake_rvp/Kconfig.name
@@ -1,11 +1,5 @@
config BOARD_INTEL_JASPERLAKE_RVP
bool "Jasperlake DDR4/LPDDR4 RVP"
- select DRIVERS_UART_8250IO
- select MAINBOARD_USES_IFD_EC_REGION
config BOARD_INTEL_JASPERLAKE_RVP_EXT_EC
bool "Jasperlake DDR4/LPDDR4 RVP with Chrome EC"
- select EC_GOOGLE_CHROMEEC
- select EC_GOOGLE_CHROMEEC_LPC
- select EC_GOOGLE_CHROMEEC_SWITCHES if VBOOT
- select INTEL_LPSS_UART_FOR_CONSOLE