diff options
author | Nicolas Boichat <drinkcat@chromium.org> | 2018-05-28 09:44:18 +0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-05-28 07:30:36 -0700 |
commit | fe70db89251961428ab2b83335b152f3311022fa (patch) | |
tree | 72c2b8231ab2ec23a3e2a730984dc27183be0bf5 | |
parent | cadc0f2513cb40a52ae70f66b34ff9f6abaa2250 (diff) | |
download | chrome-ec-fe70db89251961428ab2b83335b152f3311022fa.tar.gz |
test/build.mk: Allow boards to specify test lists
Some tests cannot be built on some boards (not enough SRAM,
unusual configuration, etc.). Instead of the long list of
exceptions in test/build.mk that we currently use, allow
each board (or chip) build.mk to set test-list-y, and
only use the default list if it is unset.
BRANCH=poppy
BUG=b:80167548
TEST=make buildalltests -j
Change-Id: I803c691f419451aad4396529302a4805cbe3f9b5
Signed-off-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1074572
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
-rw-r--r-- | board/coffeecake/build.mk | 3 | ||||
-rw-r--r-- | board/glados_pd/build.mk | 3 | ||||
-rw-r--r-- | board/hadoken/build.mk | 3 | ||||
-rw-r--r-- | board/hammer/build.mk | 3 | ||||
-rw-r--r-- | board/samus/build.mk | 4 | ||||
-rw-r--r-- | board/samus_pd/build.mk | 3 | ||||
-rw-r--r-- | board/servo_v4/build.mk | 3 | ||||
-rw-r--r-- | chip/g/build.mk | 3 | ||||
-rw-r--r-- | test/build.mk | 26 |
9 files changed, 26 insertions, 25 deletions
diff --git a/board/coffeecake/build.mk b/board/coffeecake/build.mk index 277c5c60f6..fb5a6fccdb 100644 --- a/board/coffeecake/build.mk +++ b/board/coffeecake/build.mk @@ -10,5 +10,8 @@ CHIP:=stm32 CHIP_FAMILY:=stm32f0 CHIP_VARIANT:=stm32f07x +# Not enough SRAM: Disable all tests +test-list-y= + board-y=board.o board-$(CONFIG_USB_POWER_DELIVERY)+=usb_pd_policy.o diff --git a/board/glados_pd/build.mk b/board/glados_pd/build.mk index ef1c9f1a91..3508c6c173 100644 --- a/board/glados_pd/build.mk +++ b/board/glados_pd/build.mk @@ -10,6 +10,9 @@ CHIP:=stm32 CHIP_FAMILY:=stm32f0 CHIP_VARIANT:=stm32f05x +# Not enough SRAM: Disable all tests +test-list-y= + board-y=board.o # This target builds RW only. Therefore, remove RO from dependencies. diff --git a/board/hadoken/build.mk b/board/hadoken/build.mk index f8bb859201..866f061888 100644 --- a/board/hadoken/build.mk +++ b/board/hadoken/build.mk @@ -10,4 +10,7 @@ CHIP:=nrf51 CHIP_FAMILY:=nrf51x22 CHIP_VARIANT:=nrf51822 +# Hadoken does not support scratchpad +test-list-y= + board-y=board.o diff --git a/board/hammer/build.mk b/board/hammer/build.mk index 28652fee93..b32a6b768a 100644 --- a/board/hammer/build.mk +++ b/board/hammer/build.mk @@ -10,5 +10,8 @@ CHIP:=stm32 CHIP_FAMILY:=stm32f0 CHIP_VARIANT:=stm32f07x +# Build tests that we care about for hammer +test-list-y=entropy rsa3 sha256 sha256_unrolled x25519 + board-y=board.o board-$(CONFIG_BATTERY_SMART)+=battery.o diff --git a/board/samus/build.mk b/board/samus/build.mk index 2ff7e66d94..c6c50e3885 100644 --- a/board/samus/build.mk +++ b/board/samus/build.mk @@ -9,4 +9,8 @@ # the IC is TI Stellaris LM4 CHIP:=lm4 +# Samus has board-specific chipset code, and the tests don't +# compile with it. Disable them for now. +test-list-y= + board-y=battery.o board.o power_sequence.o panel.o extpower.o diff --git a/board/samus_pd/build.mk b/board/samus_pd/build.mk index fa58b7833e..f161ab10f2 100644 --- a/board/samus_pd/build.mk +++ b/board/samus_pd/build.mk @@ -10,5 +10,8 @@ CHIP:=stm32 CHIP_FAMILY:=stm32f0 CHIP_VARIANT:=stm32f07x +# Not enough SRAM: Disable all tests +test-list-y= + board-y=board.o board-$(CONFIG_USB_POWER_DELIVERY)+=usb_mux.o usb_pd_policy.o diff --git a/board/servo_v4/build.mk b/board/servo_v4/build.mk index be86135868..6336bbfab6 100644 --- a/board/servo_v4/build.mk +++ b/board/servo_v4/build.mk @@ -10,6 +10,9 @@ CHIP:=stm32 CHIP_FAMILY:=stm32f0 CHIP_VARIANT:=stm32f07x +# Not enough SRAM: Disable all tests +test-list-y= + board-y=board.o board-$(CONFIG_USB_POWER_DELIVERY)+=usb_pd_policy.o diff --git a/chip/g/build.mk b/chip/g/build.mk index 7dc0b744df..f54a5de8ac 100644 --- a/chip/g/build.mk +++ b/chip/g/build.mk @@ -113,6 +113,9 @@ dirs-y += chip/g/dcrypto dirs-y += chip/g/loader endif +# Do not build any test on chip/g +test-list-y= + %.hex: %.flat ifneq ($(CONFIG_RW_B),) diff --git a/test/build.mk b/test/build.mk index 8853042de6..d1ca94b7dc 100644 --- a/test/build.mk +++ b/test/build.mk @@ -6,33 +6,9 @@ # on-board test binaries build # -test-list-y=pingpong timer_calib timer_dos timer_jump mutex utils utils_str +test-list-y ?= pingpong timer_calib timer_dos timer_jump mutex utils utils_str #disable: powerdemo -test-list-$(BOARD_BDS)+= - -test-list-$(BOARD_HAMMER)+=entropy rsa3 - -# Samus has board-specific chipset code, and the tests don't -# compile with it. Disable them for now. -test-list-$(BOARD_SAMUS)= - -# So does anything that runs on chip/g -ifneq ($(CHIP_G),) -test-list-y= -endif - -# Hadoken does not support scratchpad -test-list-$(BOARD_HADOKEN)= - -# For some tests, we are running out of RAM. Disable them for now. -test-list-$(BOARD_GLADOS_PD)= -test-list-$(BOARD_CHELL_PD)= -test-list-$(BOARD_OAK_PD)= -test-list-$(BOARD_SAMUS_PD)= -test-list-$(BOARD_COFFEECAKE)= -test-list-$(BOARD_SERVO_V4)= - # Emulator tests ifneq ($(TEST_LIST_HOST),) test-list-host=$(TEST_LIST_HOST) |