From 59a5645fe64c4cfc663c7fdb3496fafb7ae02edb Mon Sep 17 00:00:00 2001 From: Alex Jones Date: Fri, 29 Nov 2024 11:10:12 +0000 Subject: [PATCH] [sival,spi_host] Fix tag on Macronix flash SiVal test This will properly tie the tag to the test (and not just to the execution environment, which does nothing, due to the implementation of `_hacky_tags` used in `opentitan_test`), so that the test will run in the nightly SiVal job. Signed-off-by: Alex Jones --- sw/device/tests/pmod/BUILD | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/sw/device/tests/pmod/BUILD b/sw/device/tests/pmod/BUILD index 277d2ac5afa74..9ca0d0adc3a42 100644 --- a/sw/device/tests/pmod/BUILD +++ b/sw/device/tests/pmod/BUILD @@ -41,7 +41,6 @@ fpga_cw310( "assemble": "{rom_ext}@0 {firmware}@0x10000", }, rom_mmi = "//hw/bitstream:rom_mmi", - tags = ["cw310_sival_rom_ext"], visibility = ["//visibility:private"], ) @@ -60,7 +59,7 @@ opentitan_test( "//hw/top_earlgrey:fpga_cw310_test_rom": None, # Custom execution environment must be used for SiVal ROM EXT # to ensure that all the PMOD pins are available. - ":fpga_cw310_sival_rom_ext_no_hyper": None, + ":fpga_cw310_sival_rom_ext_no_hyper": "sival_rom_ext_no_hyper", }, ), fpga = fpga_params( @@ -74,6 +73,13 @@ opentitan_test( "pmod", ], # Requires the PMOD::BoB. ), + sival_rom_ext_no_hyper = fpga_params( + tags = [ + "cw310_sival_rom_ext", + "manual", + "pmod", + ], # Requires the PMOD::BoB. + ), deps = [ "//hw/top_earlgrey/sw/autogen:top_earlgrey", "//sw/device/lib/arch:device",