[test] enable flash scrambling/ECC in power virus test

This partially addresses lowRISC#14814. Signed-off-by: Timothy Trippel <[email protected]>
timothytrippel · Mar 16, 2023 · 6b944e7 · 6b944e7
1 parent 0b4fd8f
commit 6b944e7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/hw/top_earlgrey/data/chip_testplan.hjson b/hw/top_earlgrey/data/chip_testplan.hjson
@@ -3518,7 +3518,7 @@
  in #14095 for more details on how to approach the implementation.
  '''
  stage: V3
- tests: []
+ tests: ["chip_sw_power_virus"]
  }
  {
  name: chip_sw_power_idle_load

diff --git a/sw/device/tests/BUILD b/sw/device/tests/BUILD
@@ -2100,10 +2100,8 @@ otp_json(
  otp_partition(
  name = "CREATOR_SW_CFG",
  items = {
- "CREATOR_SW_CFG_FLASH_DATA_DEFAULT_CFG": "0000090909",
- # TODO(#14814): enable flash scrambling / ECC by replacing above
- # with below.
- #"CREATOR_SW_CFG_FLASH_DATA_DEFAULT_CFG": "0000090606",
+ # Enable flash scrambling and ECC.
+ "CREATOR_SW_CFG_FLASH_DATA_DEFAULT_CFG": "0000090606",
  },
  ),
  ],

diff --git a/sw/device/tests/power_virus_systemtest.c b/sw/device/tests/power_virus_systemtest.c
@@ -1157,8 +1157,8 @@ static void check_otp_csr_configs(void) {
  dif_flash_ctrl_region_properties_t default_properties;
  CHECK_DIF_OK(dif_flash_ctrl_get_default_region_properties(
  &flash_ctrl, &default_properties));
- CHECK(default_properties.scramble_en == kMultiBitBool4False);
- CHECK(default_properties.ecc_en == kMultiBitBool4False);
+ CHECK(default_properties.scramble_en == kMultiBitBool4True);
+ CHECK(default_properties.ecc_en == kMultiBitBool4True);
  CHECK(default_properties.high_endurance_en == kMultiBitBool4False);
 }