diff --git a/sw/device/tests/BUILD b/sw/device/tests/BUILD index 81bec9d5b28bed..662dd32fd74e39 100644 --- a/sw/device/tests/BUILD +++ b/sw/device/tests/BUILD @@ -6603,6 +6603,7 @@ opentitan_test( name = "sram_ctrl_scrambled_access_test", srcs = ["sram_ctrl_scrambled_access_test.c"], exec_env = dicts.add( + EARLGREY_CW340_TEST_ENVS, EARLGREY_TEST_ENVS, EARLGREY_SILICON_OWNER_ROM_EXT_ENVS, ), diff --git a/sw/device/tests/sram_ctrl_scrambled_access_test.c b/sw/device/tests/sram_ctrl_scrambled_access_test.c index d23c870ee798a8..00f8667c3f9da6 100644 --- a/sw/device/tests/sram_ctrl_scrambled_access_test.c +++ b/sw/device/tests/sram_ctrl_scrambled_access_test.c @@ -267,16 +267,17 @@ static void check_sram_data(scramble_test_frame *mem_frame) { // Decide whether to perform ECC error count checks after memory is scrambled. // - // This is not done on FPGAs because the interrupt handler that counts them - // does not currently trigger on our FPGA platforms. - // See #20119 for more details. + // This is not done on CW305/CW310 FPGAs because interrupts for ECC errors are + // only triggered when the SecureIbex parameter is enabled. This parameter is + // disabled for these boards due to resource constraints. On CW340 and the + // other targets, this parameter is enabled. bool check_ecc_errors = false; switch (kDeviceType) { case kDeviceFpgaCw305: case kDeviceFpgaCw310: - case kDeviceFpgaCw340: check_ecc_errors = false; break; + case kDeviceFpgaCw340: case kDeviceSilicon: case kDeviceSimDV: case kDeviceSimVerilator: