[sw] Avoid using word "sanity" in SW Tree
This follows to match changes suggested by the DV team, and implemented
in lowRISC/opentitan#4248.
Signed-off-by: Sam Elliott <selliott@lowrisc.org>
diff --git a/doc/project/checklist.md b/doc/project/checklist.md
index 1f99c24..aac16a1 100644
--- a/doc/project/checklist.md
+++ b/doc/project/checklist.md
@@ -455,14 +455,14 @@
Software unit tests exist for the DIF in `sw/device/tests/dif` named `dif_<ip>_unittest.cc`.
-### DIF_TEST_SANITY
+### DIF_TEST_SMOKE
-Sanity tests exist for the DIF in `sw/device/tests/dif` named `dif_<ip>_sanitytest.c`.
+Smoke tests exist for the DIF in `sw/device/tests/dif` named `dif_<ip>_smoketest.c`.
This should perform a basic test of the main datapath of the hardware module by the embedded core, via the DIF, and should be able to be run on all OpenTitan platforms (including FPGA, simulation, and DV).
This test will be shared with DV.
-Sanity tests are for diagnosing major issues in both software and hardware, and with this in mind, they should execute quickly.
+Smoke tests are for diagnosing major issues in both software and hardware, and with this in mind, they should execute quickly.
Initially we expect this kind of test to be written by hardware designers for debugging issues during module development.
This happens long before a DIF is implemented, so there are no requirements on how these should work, though we suggest they are placed in `sw/device/tests/<ip>/<ip>.c` as this has been the convention until now.
Later, when a DIF is written, the DIF author is responsible for updating this test to use the DIF, and for moving this test into the aforementioned location.
diff --git a/doc/project/development_stages.md b/doc/project/development_stages.md
index db22673..6f59b96 100644
--- a/doc/project/development_stages.md
+++ b/doc/project/development_stages.md
@@ -149,7 +149,7 @@
| **Stage** | **Name** | **Definition** |
| --- | --- | --- |
| S0 | Initial Work | Work has started on a DIF for the given IP block. |
-| S1 | Functional | <ul> <li> DIF has been reviewed and merged <li> DIF is used by all in-tree device code <li> DIF has (mocked) unit tests </ul> |
+| S1 | Functional | <ul> <li> DIF has been reviewed and merged <li> DIF is used by all in-tree device code <li> DIF has (mocked) unit tests <li> DIF has smoke test </ul> |
| S2 | Complete | <ul> <li> DIF API is now Complete <li> The respective IP block is feature complete (at least D2) <li> DIF matches HW designer's agreed IP block usage <li> DIF covers all specified functionality of the IP block <li> DIF is used for chip-level DV <li> DIF documented in IP documentation <li> DIF has initial Tock integration </ul> |
| S3 | Stable | <ul> <li> DIF API Reviewed and Stable <li> The respective IP block is at D3/V3 <li> DIF tested fully (DV + Unit tests, full functional coverage) <li> Complete and Stable Tock interface to DIF </ul> |
diff --git a/doc/project/sw_checklist.md.tpl b/doc/project/sw_checklist.md.tpl
index 1e54490..7d10acb 100644
--- a/doc/project/sw_checklist.md.tpl
+++ b/doc/project/sw_checklist.md.tpl
@@ -20,12 +20,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_alert_handler.md b/sw/device/lib/dif/dif_alert_handler.md
index a668fef..3360f1d 100644
--- a/sw/device/lib/dif/dif_alert_handler.md
+++ b/sw/device/lib/dif/dif_alert_handler.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_gpio.md b/sw/device/lib/dif/dif_gpio.md
index 04130c4..7e5eb6a 100644
--- a/sw/device/lib/dif/dif_gpio.md
+++ b/sw/device/lib/dif/dif_gpio.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
@@ -64,4 +64,5 @@
[DIF_REVIEW_C_STABLE]: {{< relref "/doc/project/checklist.md#dif_review_c_stable" >}}
[DIF_TEST_UNIT_COMPLETE]: {{< relref "/doc/project/checklist.md#dif_test_unit_complete" >}}
[DIF_TODO_COMPLETE]: {{< relref "/doc/project/checklist.md#dif_todo_complete" >}}
-[DIF_REVIEW_TOCK_STABLE]: {{< relref "/doc/project/checklist.md#dif_review_tock_stable" >}}
\ No newline at end of file
+[DIF_REVIEW_TOCK_STABLE]: {{< relref
+"/doc/project/checklist.md#dif_review_tock_stable" >}}
diff --git a/sw/device/lib/dif/dif_hmac.c b/sw/device/lib/dif/dif_hmac.c
index 6098b7f..b6724a4 100644
--- a/sw/device/lib/dif/dif_hmac.c
+++ b/sw/device/lib/dif/dif_hmac.c
@@ -67,8 +67,7 @@
dif_hmac_result_t dif_hmac_init(const dif_hmac_config_t *config,
dif_hmac_t *hmac) {
- // Basic sanity checks on parameters. In `kDifHmacModeHmac` mode a key is
- // required.
+ // Basic checks on parameters. In `kDifHmacModeHmac` mode a key is required.
if (config == NULL || hmac == NULL) {
return kDifHmacBadArg;
}
diff --git a/sw/device/lib/dif/dif_hmac.md b/sw/device/lib/dif/dif_hmac.md
index 2529ae7..d18298a 100644
--- a/sw/device/lib/dif/dif_hmac.md
+++ b/sw/device/lib/dif/dif_hmac.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_smoke][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_smoke]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_i2c.md b/sw/device/lib/dif/dif_i2c.md
index a5560da..7e14523 100644
--- a/sw/device/lib/dif/dif_i2c.md
+++ b/sw/device/lib/dif/dif_i2c.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_smoke][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_smoke]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_otbn.md b/sw/device/lib/dif/dif_otbn.md
index 8ce3d32..21787c8 100644
--- a/sw/device/lib/dif/dif_otbn.md
+++ b/sw/device/lib/dif/dif_otbn.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Done |
Implementation | [DIF_USED_IN_TREE][] | Done |
Tests | [DIF_TEST_UNIT][] | Done |
-Tests | [DIF_TEST_SANITY][] | Done |
+Tests | [DIF_TEST_SMOKE][] | Done |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_otp_ctrl.md b/sw/device/lib/dif/dif_otp_ctrl.md
index 7a16e50..d50408a 100644
--- a/sw/device/lib/dif/dif_otp_ctrl.md
+++ b/sw/device/lib/dif/dif_otp_ctrl.md
@@ -12,12 +12,12 @@
Implementation | [DIF_EXISTS][] | Done |
Implementation | [DIF_USED_IN_TREE][] | Done |
Tests | [DIF_TEST_UNIT][] | Done |
-Tests | [DIF_TEST_SANITY][] | Done |
+Tests | [DIF_TEST_SMOKE][] | Done |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
Type | Item | Resolution | Note/Collaterals
diff --git a/sw/device/lib/dif/dif_plic.md b/sw/device/lib/dif/dif_plic.md
index 06b45a0..696b25f 100644
--- a/sw/device/lib/dif/dif_plic.md
+++ b/sw/device/lib/dif/dif_plic.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Done |
Implementation | [DIF_USED_IN_TREE][] | Done |
Tests | [DIF_TEST_UNIT][] | Done |
-Tests | [DIF_TEST_SANITY][] | Done |
+Tests | [DIF_TEST_SMOKE][] | Done |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_pwrmgr.md b/sw/device/lib/dif/dif_pwrmgr.md
index 4678f1e..a5593f8 100644
--- a/sw/device/lib/dif/dif_pwrmgr.md
+++ b/sw/device/lib/dif/dif_pwrmgr.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_rstmgr.md b/sw/device/lib/dif/dif_rstmgr.md
index 4bf7eb9..4c006e9 100644
--- a/sw/device/lib/dif/dif_rstmgr.md
+++ b/sw/device/lib/dif/dif_rstmgr.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Done |
Implementation | [DIF_USED_IN_TREE][] | Done |
Tests | [DIF_TEST_UNIT][] | Done |
-Tests | [DIF_TEST_SANITY][] | Done |
+Tests | [DIF_TEST_SMOKE][] | Done |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_rv_timer.md b/sw/device/lib/dif/dif_rv_timer.md
index cb5b67b..c77f361 100644
--- a/sw/device/lib/dif/dif_rv_timer.md
+++ b/sw/device/lib/dif/dif_rv_timer.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_spi_device.md b/sw/device/lib/dif/dif_spi_device.md
index 6f6b1d3..d2a9810 100644
--- a/sw/device/lib/dif/dif_spi_device.md
+++ b/sw/device/lib/dif/dif_spi_device.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_uart.md b/sw/device/lib/dif/dif_uart.md
index 2be120a..7fffaa4 100644
--- a/sw/device/lib/dif/dif_uart.md
+++ b/sw/device/lib/dif/dif_uart.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Done |
Implementation | [DIF_USED_IN_TREE][] | Done |
Tests | [DIF_TEST_UNIT][] | Done |
-Tests | [DIF_TEST_SANITY][] | Done |
+Tests | [DIF_TEST_SMOKE][] | Done |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/lib/dif/dif_usbdev.md b/sw/device/lib/dif/dif_usbdev.md
index 8162a9f..8ac296c 100644
--- a/sw/device/lib/dif/dif_usbdev.md
+++ b/sw/device/lib/dif/dif_usbdev.md
@@ -14,12 +14,12 @@
Implementation | [DIF_EXISTS][] | Not Started |
Implementation | [DIF_USED_IN_TREE][] | Not Started |
Tests | [DIF_TEST_UNIT][] | Not Started |
-Tests | [DIF_TEST_SANITY][] | Not Started |
+Tests | [DIF_TEST_SMOKE][] | Not Started |
[DIF_EXISTS]: {{< relref "/doc/project/checklist.md#dif_exists" >}}
[DIF_USED_IN_TREE]: {{< relref "/doc/project/checklist.md#dif_used_in_tree" >}}
[DIF_TEST_UNIT]: {{< relref "/doc/project/checklist.md#dif_test_unit" >}}
-[DIF_TEST_SANITY]: {{< relref "/doc/project/checklist.md#dif_test_sanity" >}}
+[DIF_TEST_SMOKE]: {{< relref "/doc/project/checklist.md#dif_test_smoke" >}}
### S2
diff --git a/sw/device/mask_rom/boot.md b/sw/device/mask_rom/boot.md
index 243269d..3481956 100644
--- a/sw/device/mask_rom/boot.md
+++ b/sw/device/mask_rom/boot.md
@@ -77,7 +77,7 @@
// Determine which ROM_EXT slot is prioritised (2.b, 2.c.i)
for ( current_rom_ext_manifest in rom_ext_manifests_to_try(boot_policy) ) { // Boot Policy Module
- // Sanity Check ROM_EXT Manifest (2.c.ii)
+ // Check ROM_EXT Manifest (2.c.ii)
// **Open Q:** Integration with Secure Boot Hardware
// - Header Format (ROM_EXT Manifest Module)
// - Plausible Key (??)
diff --git a/sw/device/tests/dif/dif_hmac_sanitytest.c b/sw/device/tests/dif/dif_hmac_smoketest.c
similarity index 99%
rename from sw/device/tests/dif/dif_hmac_sanitytest.c
rename to sw/device/tests/dif/dif_hmac_smoketest.c
index 0e874bb..dfae771 100644
--- a/sw/device/tests/dif/dif_hmac_sanitytest.c
+++ b/sw/device/tests/dif/dif_hmac_smoketest.c
@@ -2,9 +2,9 @@
// Licensed under the Apache License, Version 2.0, see LICENSE for details.
// SPDX-License-Identifier: Apache-2.0
-#include "sw/device/lib/dif/dif_hmac.h"
#include "sw/device/lib/arch/device.h"
#include "sw/device/lib/base/mmio.h"
+#include "sw/device/lib/dif/dif_hmac.h"
#include "sw/device/lib/flash_ctrl.h"
#include "sw/device/lib/runtime/log.h"
#include "sw/device/lib/testing/check.h"
diff --git a/sw/device/tests/dif/dif_plic_sanitytest.c b/sw/device/tests/dif/dif_plic_smoketest.c
similarity index 100%
rename from sw/device/tests/dif/dif_plic_sanitytest.c
rename to sw/device/tests/dif/dif_plic_smoketest.c
diff --git a/sw/device/tests/dif/dif_rstmgr_sanitytest.c b/sw/device/tests/dif/dif_rstmgr_smoketest.c
similarity index 100%
rename from sw/device/tests/dif/dif_rstmgr_sanitytest.c
rename to sw/device/tests/dif/dif_rstmgr_smoketest.c
diff --git a/sw/device/tests/dif/dif_rv_timer_sanitytest.c b/sw/device/tests/dif/dif_rv_timer_smoketest.c
similarity index 100%
rename from sw/device/tests/dif/dif_rv_timer_sanitytest.c
rename to sw/device/tests/dif/dif_rv_timer_smoketest.c
diff --git a/sw/device/tests/dif/dif_uart_sanitytest.c b/sw/device/tests/dif/dif_uart_smoketest.c
similarity index 96%
rename from sw/device/tests/dif/dif_uart_sanitytest.c
rename to sw/device/tests/dif/dif_uart_smoketest.c
index d7feedd..db9391a 100644
--- a/sw/device/tests/dif/dif_uart_sanitytest.c
+++ b/sw/device/tests/dif/dif_uart_smoketest.c
@@ -12,7 +12,7 @@
#include "hw/top_earlgrey/sw/autogen/top_earlgrey.h" // Generated.
-static const uint8_t kSendData[] = "Sanity test!";
+static const uint8_t kSendData[] = "Smoke test!";
const test_config_t kTestConfig = {
.can_clobber_uart = true,
diff --git a/sw/device/tests/dif/meson.build b/sw/device/tests/dif/meson.build
index 4d6eb76..9d512a9 100644
--- a/sw/device/tests/dif/meson.build
+++ b/sw/device/tests/dif/meson.build
@@ -178,10 +178,10 @@
cpp_args: ['-DMOCK_MMIO'],
))
-dif_plic_sanitytest_lib = declare_dependency(
+dif_plic_smoketest_lib = declare_dependency(
link_with: static_library(
- 'dif_plic_sanitytest_lib',
- sources: ['dif_plic_sanitytest.c'],
+ 'dif_plic_smoketest_lib',
+ sources: ['dif_plic_smoketest.c'],
dependencies: [
sw_lib_dif_uart,
sw_lib_dif_plic,
@@ -194,15 +194,15 @@
),
)
sw_tests += {
- 'dif_plic_sanitytest': {
- 'library': dif_plic_sanitytest_lib,
+ 'dif_plic_smoketest': {
+ 'library': dif_plic_smoketest_lib,
}
}
-dif_uart_sanitytest_lib = declare_dependency(
+dif_uart_smoketest_lib = declare_dependency(
link_with: static_library(
- 'dif_uart_sanitytest_lib',
- sources: ['dif_uart_sanitytest.c'],
+ 'dif_uart_smoketest_lib',
+ sources: ['dif_uart_smoketest.c'],
dependencies: [
sw_lib_dif_uart,
sw_lib_mmio,
@@ -211,15 +211,15 @@
),
)
sw_tests += {
- 'dif_uart_sanitytest': {
- 'library': dif_uart_sanitytest_lib,
+ 'dif_uart_smoketest': {
+ 'library': dif_uart_smoketest_lib,
}
}
-dif_rv_timer_sanitytest_lib = declare_dependency(
+dif_rv_timer_smoketest_lib = declare_dependency(
link_with: static_library(
- 'dif_rv_timer_sanitytest_lib',
- sources: ['dif_rv_timer_sanitytest.c'],
+ 'dif_rv_timer_smoketest_lib',
+ sources: ['dif_rv_timer_smoketest.c'],
dependencies: [
sw_lib_dif_rv_timer,
sw_lib_runtime_log,
@@ -230,15 +230,15 @@
),
)
sw_tests += {
- 'dif_rv_timer_sanitytest': {
- 'library': dif_rv_timer_sanitytest_lib,
+ 'dif_rv_timer_smoketest': {
+ 'library': dif_rv_timer_smoketest_lib,
}
}
-dif_hmac_sanitytest_lib = declare_dependency(
+dif_hmac_smoketest_lib = declare_dependency(
link_with: static_library(
- 'dif_hmac_sanitytest_lib',
- sources: ['dif_hmac_sanitytest.c'],
+ 'dif_hmac_smoketest_lib',
+ sources: ['dif_hmac_smoketest.c'],
dependencies: [
sw_lib_dif_hmac,
sw_lib_runtime_log,
@@ -248,15 +248,15 @@
),
)
sw_tests += {
- 'dif_hmac_sanitytest': {
- 'library': dif_hmac_sanitytest_lib,
+ 'dif_hmac_smoketest': {
+ 'library': dif_hmac_smoketest_lib,
}
}
-dif_rstmgr_sanitytest_lib = declare_dependency(
+dif_rstmgr_smoketest_lib = declare_dependency(
link_with: static_library(
- 'dif_rstmgr_sanitytest_lib',
- sources: ['dif_rstmgr_sanitytest.c'],
+ 'dif_rstmgr_smoketest_lib',
+ sources: ['dif_rstmgr_smoketest.c'],
dependencies: [
sw_lib_dif_rstmgr,
sw_lib_mmio,
@@ -265,8 +265,8 @@
),
)
sw_tests += {
- 'dif_rstmgr_sanitytest': {
- 'library': dif_rstmgr_sanitytest_lib,
+ 'dif_rstmgr_smoketest': {
+ 'library': dif_rstmgr_smoketest_lib,
}
}
diff --git a/sw/device/tests/meson.build b/sw/device/tests/meson.build
index 12d1b9f..6d5b57f 100644
--- a/sw/device/tests/meson.build
+++ b/sw/device/tests/meson.build
@@ -100,10 +100,10 @@
}
}
-pmp_sanitytest_napot_lib = declare_dependency(
+pmp_smoketest_napot_lib = declare_dependency(
link_with: static_library(
- 'pmp_sanitytest_napot_lib',
- sources: ['pmp_sanitytest_napot.c'],
+ 'pmp_smoketest_napot_lib',
+ sources: ['pmp_smoketest_napot.c'],
dependencies: [
sw_lib_irq,
sw_lib_runtime_log,
@@ -114,15 +114,15 @@
),
)
sw_tests += {
- 'pmp_sanitytest_napot': {
- 'library': pmp_sanitytest_napot_lib,
+ 'pmp_smoketest_napot': {
+ 'library': pmp_smoketest_napot_lib,
}
}
-pmp_sanitytest_tor_lib = declare_dependency(
+pmp_smoketest_tor_lib = declare_dependency(
link_with: static_library(
- 'pmp_sanitytest_tor_lib',
- sources: ['pmp_sanitytest_tor.c'],
+ 'pmp_smoketest_tor_lib',
+ sources: ['pmp_smoketest_tor.c'],
dependencies: [
sw_lib_irq,
sw_lib_runtime_log,
@@ -133,8 +133,8 @@
),
)
sw_tests += {
- 'pmp_sanitytest_tor': {
- 'library': pmp_sanitytest_tor_lib,
+ 'pmp_smoketest_tor': {
+ 'library': pmp_smoketest_tor_lib,
}
}
diff --git a/sw/device/tests/pmp_sanitytest_napot.c b/sw/device/tests/pmp_smoketest_napot.c
similarity index 97%
rename from sw/device/tests/pmp_sanitytest_napot.c
rename to sw/device/tests/pmp_smoketest_napot.c
index d2509a1..60497c7 100644
--- a/sw/device/tests/pmp_sanitytest_napot.c
+++ b/sw/device/tests/pmp_smoketest_napot.c
@@ -68,7 +68,8 @@
(uintptr_t)&pmp_load_store_test_data[PMP_LOAD_RANGE_BOTTOM_OFFSET];
pmp_region_config_t config = {
- .lock = kPmpRegionLockLocked, .permissions = kPmpRegionPermissionsNone,
+ .lock = kPmpRegionLockLocked,
+ .permissions = kPmpRegionPermissionsNone,
};
pmp_region_configure_napot_result_t result = pmp_region_configure_napot(
diff --git a/sw/device/tests/pmp_sanitytest_tor.c b/sw/device/tests/pmp_smoketest_tor.c
similarity index 97%
rename from sw/device/tests/pmp_sanitytest_tor.c
rename to sw/device/tests/pmp_smoketest_tor.c
index 39d2530..7c8f880 100644
--- a/sw/device/tests/pmp_sanitytest_tor.c
+++ b/sw/device/tests/pmp_smoketest_tor.c
@@ -87,7 +87,8 @@
(uintptr_t)&pmp_load_test_data[PMP_LOAD_RANGE_SIZE];
pmp_region_config_t config = {
- .lock = kPmpRegionLockLocked, .permissions = kPmpRegionPermissionsNone,
+ .lock = kPmpRegionLockLocked,
+ .permissions = kPmpRegionPermissionsNone,
};
pmp_region_configure_result_t result = pmp_region_configure_tor(
diff --git a/test/systemtest/config.py b/test/systemtest/config.py
index c953da4..c12d38f 100644
--- a/test/systemtest/config.py
+++ b/test/systemtest/config.py
@@ -40,25 +40,25 @@
"name": "dif_otp_ctrl_smoketest",
},
{
- "name": "dif_plic_sanitytest",
+ "name": "dif_plic_smoketest",
},
{
- "name": "dif_rstmgr_sanitytest",
+ "name": "dif_rstmgr_smoketest",
},
{
- "name": "dif_rv_timer_sanitytest",
+ "name": "dif_rv_timer_smoketest",
},
{
- "name": "dif_uart_sanitytest",
+ "name": "dif_uart_smoketest",
},
{
"name": "flash_ctrl_test",
},
{
- "name": "pmp_sanitytest_napot",
+ "name": "pmp_smoketest_napot",
},
{
- "name": "pmp_sanitytest_tor",
+ "name": "pmp_smoketest_tor",
},
{
"name": "sha256_test",
diff --git a/test/systemtest/earlgrey/test_sim_verilator.py b/test/systemtest/earlgrey/test_sim_verilator.py
index e21d745..828a2d4 100644
--- a/test/systemtest/earlgrey/test_sim_verilator.py
+++ b/test/systemtest/earlgrey/test_sim_verilator.py
@@ -248,7 +248,7 @@
assert sim.find_in_uart0(spiwait_msg, timeout=120)
log.debug("SPI is ready, continuing with spiload")
- app_bin = bin_dir / 'sw/device/tests/dif_uart_sanitytest_sim_verilator.bin'
+ app_bin = bin_dir / 'sw/device/tests/dif_uart_smoketest_sim_verilator.bin'
spiflash = bin_dir / 'sw/host/spiflash/spiflash'
utils.load_sw_over_spi(tmp_path, spiflash, app_bin,
['--verilator', sim.spi0_device_path])
