google/*: Clean up Kconfig board selection for non-MTK Google Arm boards
This patch unifies and simplifies the Kconfig selection model for the Gru, Herobrine, Trogdor and Veyron boards according to the model discussed in CB:78972. Also add missing license headers to two Kconfig files while I'm here. Change-Id: If679a05afd10869afba9c2a33b54862e102b5f40 Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/79022 Reviewed-by: Yu-Ping Wu <yupingso@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
		| @@ -1,25 +1,12 @@ | |||||||
| ## SPDX-License-Identifier: GPL-2.0-only | ## SPDX-License-Identifier: GPL-2.0-only | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_GRU_COMMON  # Umbrella option to be selected by variant boards. | config BOARD_GOOGLE_GRU_COMMON | ||||||
| 	def_bool n | 	def_bool BOARD_GOOGLE_BOB || \ | ||||||
|  | 		 BOARD_GOOGLE_GRU || \ | ||||||
| config BOARD_GOOGLE_KEVIN | 		 BOARD_GOOGLE_KEVIN || \ | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON | 		 BOARD_GOOGLE_NEFARIO || \ | ||||||
|  | 		 BOARD_GOOGLE_RAINIER || \ | ||||||
| config BOARD_GOOGLE_GRU | 		 BOARD_GOOGLE_SCARLET | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_BOB |  | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_SCARLET |  | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_NEFARIO |  | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_RAINIER |  | ||||||
| 	select BOARD_GOOGLE_GRU_COMMON |  | ||||||
|  |  | ||||||
| if BOARD_GOOGLE_GRU_COMMON | if BOARD_GOOGLE_GRU_COMMON | ||||||
|  |  | ||||||
| @@ -27,23 +14,21 @@ if BOARD_GOOGLE_GRU_COMMON | |||||||
| # convenience. Explicitly override it when building for earlier Kevins. | # convenience. Explicitly override it when building for earlier Kevins. | ||||||
| config GRU_HAS_TPM2 | config GRU_HAS_TPM2 | ||||||
| 	bool "Build for a board revision with a SPI TPM 2.0" | 	bool "Build for a board revision with a SPI TPM 2.0" | ||||||
| 	default n if BOARD_GOOGLE_KEVIN | 	default !BOARD_GOOGLE_KEVIN | ||||||
| 	default y |  | ||||||
|  |  | ||||||
| config GRU_HAS_CENTERLOG_PWM | config GRU_HAS_CENTERLOG_PWM | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_GRU || \ | ||||||
| 	default y if BOARD_GOOGLE_GRU || BOARD_GOOGLE_KEVIN || BOARD_GOOGLE_BOB | 		 BOARD_GOOGLE_KEVIN || \ | ||||||
| 	default n | 		 BOARD_GOOGLE_BOB | ||||||
|  |  | ||||||
| config GRU_HAS_WLAN_RESET | config GRU_HAS_WLAN_RESET | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_GRU || \ | ||||||
| 	default y if BOARD_GOOGLE_GRU || BOARD_GOOGLE_KEVIN || BOARD_GOOGLE_BOB | 		 BOARD_GOOGLE_KEVIN || \ | ||||||
| 	default n | 		 BOARD_GOOGLE_BOB | ||||||
|  |  | ||||||
| config GRU_BASEBOARD_SCARLET | config GRU_BASEBOARD_SCARLET | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_SCARLET || \ | ||||||
| 	default y if BOARD_GOOGLE_SCARLET || BOARD_GOOGLE_RAINIER | 		 BOARD_GOOGLE_RAINIER | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config BOARD_SPECIFIC_OPTIONS | config BOARD_SPECIFIC_OPTIONS | ||||||
| 	def_bool y | 	def_bool y | ||||||
|   | |||||||
| @@ -1,39 +1,21 @@ | |||||||
| config BOARD_GOOGLE_HEROBRINE_COMMON  # Umbrella option to be selected by variants | ## SPDX-License-Identifier: GPL-2.0-only | ||||||
| 	def_bool n |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_HEROBRINE | config BOARD_GOOGLE_HEROBRINE_COMMON | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON | 	def_bool BOARD_GOOGLE_EVOKER || \ | ||||||
|  | 		 BOARD_GOOGLE_HEROBRINE || \ | ||||||
| config BOARD_GOOGLE_HEROBRINE_REV0 | 		 BOARD_GOOGLE_HEROBRINE_REV0 || \ | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON | 		 BOARD_GOOGLE_HOGLIN || \ | ||||||
|  | 		 BOARD_GOOGLE_PIGLIN || \ | ||||||
| config BOARD_GOOGLE_SENOR | 		 BOARD_GOOGLE_SENOR || \ | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON | 		 BOARD_GOOGLE_VILLAGER || \ | ||||||
|  | 		 BOARD_GOOGLE_ZOGLIN || \ | ||||||
| config BOARD_GOOGLE_PIGLIN | 		 BOARD_GOOGLE_ZOMBIE | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_HOGLIN |  | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VILLAGER |  | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_EVOKER |  | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_ZOGLIN |  | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_ZOMBIE |  | ||||||
| 	select BOARD_GOOGLE_HEROBRINE_COMMON |  | ||||||
|  |  | ||||||
| if BOARD_GOOGLE_HEROBRINE_COMMON | if BOARD_GOOGLE_HEROBRINE_COMMON | ||||||
|  |  | ||||||
| config HEROBRINE_HAS_FINGERPRINT | config HEROBRINE_HAS_FINGERPRINT | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_HEROBRINE || \ | ||||||
| 	default y if BOARD_GOOGLE_HEROBRINE || BOARD_GOOGLE_HEROBRINE_REV0 | 		 BOARD_GOOGLE_HEROBRINE_REV0 | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config BOARD_SPECIFIC_OPTIONS | config BOARD_SPECIFIC_OPTIONS | ||||||
| 	def_bool y | 	def_bool y | ||||||
|   | |||||||
| @@ -1,45 +1,19 @@ | |||||||
|  | ## SPDX-License-Identifier: GPL-2.0-only | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_TROGDOR_COMMON  # Umbrella option to be selected by variants | config BOARD_GOOGLE_TROGDOR_COMMON | ||||||
| 	def_bool n | 	def_bool BOARD_GOOGLE_BUBS || \ | ||||||
|  | 		 BOARD_GOOGLE_COACHZ || \ | ||||||
| config BOARD_GOOGLE_BUBS | 		 BOARD_GOOGLE_GELARSHIE || \ | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON | 		 BOARD_GOOGLE_HOMESTAR || \ | ||||||
|  | 		 BOARD_GOOGLE_KINGOFTOWN || \ | ||||||
| config BOARD_GOOGLE_COACHZ | 		 BOARD_GOOGLE_LAZOR || \ | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON | 		 BOARD_GOOGLE_MARZIPAN || \ | ||||||
|  | 		 BOARD_GOOGLE_MRBLAND || \ | ||||||
| config BOARD_GOOGLE_GELARSHIE | 		 BOARD_GOOGLE_PAZQUEL || \ | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON | 		 BOARD_GOOGLE_POMPOM || \ | ||||||
|  | 		 BOARD_GOOGLE_QUACKINGSTICK || \ | ||||||
| config BOARD_GOOGLE_HOMESTAR | 		 BOARD_GOOGLE_TROGDOR || \ | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON | 		 BOARD_GOOGLE_WORMDINGLER | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_KINGOFTOWN |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_LAZOR |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_MARZIPAN |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_MRBLAND |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_PAZQUEL |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_POMPOM |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_QUACKINGSTICK |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_WORMDINGLER |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_TROGDOR |  | ||||||
| 	select BOARD_GOOGLE_TROGDOR_COMMON |  | ||||||
|  |  | ||||||
| if BOARD_GOOGLE_TROGDOR_COMMON | if BOARD_GOOGLE_TROGDOR_COMMON | ||||||
|  |  | ||||||
| @@ -47,24 +21,20 @@ config TROGDOR_REV0 | |||||||
| 	def_bool n | 	def_bool n | ||||||
|  |  | ||||||
| config TROGDOR_HAS_BRIDGE_BACKLIGHT | config TROGDOR_HAS_BRIDGE_BACKLIGHT | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_HOMESTAR | ||||||
| 	default y if BOARD_GOOGLE_HOMESTAR |  | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config TROGDOR_HAS_MIPI_PANEL | config TROGDOR_HAS_MIPI_PANEL | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_MRBLAND || \ | ||||||
| 	default y if BOARD_GOOGLE_MRBLAND || BOARD_GOOGLE_QUACKINGSTICK || BOARD_GOOGLE_WORMDINGLER | 		 BOARD_GOOGLE_QUACKINGSTICK || \ | ||||||
|  | 		 BOARD_GOOGLE_WORMDINGLER | ||||||
| 	select MIPI_PANEL_AUO_B101UAN08_3 if BOARD_GOOGLE_MRBLAND || BOARD_GOOGLE_QUACKINGSTICK | 	select MIPI_PANEL_AUO_B101UAN08_3 if BOARD_GOOGLE_MRBLAND || BOARD_GOOGLE_QUACKINGSTICK | ||||||
| 	select MIPI_PANEL_BOE_TV101WUM_N53 if BOARD_GOOGLE_MRBLAND | 	select MIPI_PANEL_BOE_TV101WUM_N53 if BOARD_GOOGLE_MRBLAND | ||||||
| 	select MIPI_PANEL_INX_HJ110IZ_01A_B2 if BOARD_GOOGLE_WORMDINGLER | 	select MIPI_PANEL_INX_HJ110IZ_01A_B2 if BOARD_GOOGLE_WORMDINGLER | ||||||
| 	select MIPI_PANEL_BOE_TV110C9M_LL0 if BOARD_GOOGLE_WORMDINGLER | 	select MIPI_PANEL_BOE_TV110C9M_LL0 if BOARD_GOOGLE_WORMDINGLER | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config TROGDOR_HAS_FINGERPRINT | config TROGDOR_HAS_FINGERPRINT | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_COACHZ || \ | ||||||
| 	default y if BOARD_GOOGLE_COACHZ | 		 BOARD_GOOGLE_MARZIPAN | ||||||
| 	default y if BOARD_GOOGLE_MARZIPAN |  | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config BOARD_SPECIFIC_OPTIONS | config BOARD_SPECIFIC_OPTIONS | ||||||
| 	def_bool y | 	def_bool y | ||||||
|   | |||||||
| @@ -1,27 +1,11 @@ | |||||||
| ## SPDX-License-Identifier: GPL-2.0-only | ## SPDX-License-Identifier: GPL-2.0-only | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VEYRON | config BOARD_GOOGLE_VEYRON | ||||||
| 	def_bool n | 	def_bool BOARD_GOOGLE_VEYRON_JAQ || \ | ||||||
|  | 		 BOARD_GOOGLE_VEYRON_JERRY || \ | ||||||
| config BOARD_GOOGLE_VEYRON_JAQ | 		 BOARD_GOOGLE_VEYRON_MIGHTY || \ | ||||||
| 	select BOARD_GOOGLE_VEYRON | 		 BOARD_GOOGLE_VEYRON_MINNIE || \ | ||||||
| 	select SYSTEM_TYPE_LAPTOP | 		 BOARD_GOOGLE_VEYRON_SPEEDY | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VEYRON_JERRY |  | ||||||
| 	select BOARD_GOOGLE_VEYRON |  | ||||||
| 	select SYSTEM_TYPE_LAPTOP |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VEYRON_MIGHTY |  | ||||||
| 	select BOARD_GOOGLE_VEYRON |  | ||||||
| 	select SYSTEM_TYPE_LAPTOP |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VEYRON_MINNIE |  | ||||||
| 	select BOARD_GOOGLE_VEYRON |  | ||||||
| 	select SYSTEM_TYPE_LAPTOP |  | ||||||
|  |  | ||||||
| config BOARD_GOOGLE_VEYRON_SPEEDY |  | ||||||
| 	select BOARD_GOOGLE_VEYRON |  | ||||||
| 	select SYSTEM_TYPE_LAPTOP |  | ||||||
|  |  | ||||||
| if BOARD_GOOGLE_VEYRON | if BOARD_GOOGLE_VEYRON | ||||||
|  |  | ||||||
| @@ -29,11 +13,9 @@ if BOARD_GOOGLE_VEYRON | |||||||
| # resistors. These get overpowered by the SoC's internal pull-downs, so we | # resistors. These get overpowered by the SoC's internal pull-downs, so we | ||||||
| # cannot read those pins as tri-state. They're restricted to binary RAM codes. | # cannot read those pins as tri-state. They're restricted to binary RAM codes. | ||||||
| config VEYRON_FORCE_BINARY_RAM_CODE | config VEYRON_FORCE_BINARY_RAM_CODE | ||||||
| 	bool | 	def_bool BOARD_GOOGLE_VEYRON_JAQ || \ | ||||||
| 	default y if BOARD_GOOGLE_VEYRON_JAQ | 		 BOARD_GOOGLE_VEYRON_JERRY || \ | ||||||
| 	default y if BOARD_GOOGLE_VEYRON_JERRY | 		 BOARD_GOOGLE_VEYRON_MIGHTY | ||||||
| 	default y if BOARD_GOOGLE_VEYRON_MIGHTY |  | ||||||
| 	default n |  | ||||||
|  |  | ||||||
| config BOARD_SPECIFIC_OPTIONS | config BOARD_SPECIFIC_OPTIONS | ||||||
| 	def_bool y | 	def_bool y | ||||||
| @@ -48,6 +30,7 @@ config BOARD_SPECIFIC_OPTIONS | |||||||
| 	select SPI_FLASH_WINBOND | 	select SPI_FLASH_WINBOND | ||||||
| 	select I2C_TPM | 	select I2C_TPM | ||||||
| 	select MAINBOARD_HAS_TPM1 | 	select MAINBOARD_HAS_TPM1 | ||||||
|  | 	select SYSTEM_TYPE_LAPTOP | ||||||
|  |  | ||||||
| config VBOOT | config VBOOT | ||||||
| 	select VBOOT_VBNV_FLASH | 	select VBOOT_VBNV_FLASH | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user