diff options
author | Jett Rink <jettrink@chromium.org> | 2020-08-13 09:44:34 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-08-13 22:32:17 +0000 |
commit | 0df97d0e5846493c868eabe4c1647d67a946d7fd (patch) | |
tree | 100e343403bb91aabf8661fbf85e1dcf07b16676 /test/usb_sm_checks.c | |
parent | 39a526023e435a776639d68271eed593598e9106 (diff) | |
download | chrome-ec-0df97d0e5846493c868eabe4c1647d67a946d7fd.tar.gz |
test: remove no_empty_state tests
When we use IS_ENABLED style code we may have empty state machine states
that are unused. The linker ensures that we never try to reference these
states. This does mean that extra states do cost ~20 bytes each. This
cost does seem worth it to keep the value of usb states stable.
BRANCH=none
BUG=none
TEST=buildall
Signed-off-by: Jett Rink <jettrink@chromium.org>
Change-Id: I44c1454bed91c9e28d89ebd6b75e9df684c41844
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2354192
Reviewed-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'test/usb_sm_checks.c')
-rw-r--r-- | test/usb_sm_checks.c | 64 |
1 files changed, 0 insertions, 64 deletions
diff --git a/test/usb_sm_checks.c b/test/usb_sm_checks.c index 498d56e323..03e5aa23c2 100644 --- a/test/usb_sm_checks.c +++ b/test/usb_sm_checks.c @@ -108,70 +108,6 @@ int test_pe_no_parent_cycles(void) return EC_SUCCESS; } -static int test_no_empty_state(const struct test_sm_data * const sm_data) -{ - int i; - const struct usb_state empty = { 0 }; - - for (i = 0; i < sm_data->size; ++i) { - if (memcmp(&sm_data->base[i], &empty, sizeof(empty)) == 0) - break; - } - - /* Ensure all states had data, otherwise the ith state is empty. */ - TEST_EQ(i, sm_data->size, "%d"); - - return EC_SUCCESS; -} - -int test_tc_no_empty_state(void) -{ - int i; - - for (i = 0; i < test_tc_sm_data_size; ++i) { - const int rv = test_no_empty_state(&test_tc_sm_data[i]); - - if (rv) { - ccprintf("TC State machine %d has empty state!\n", i); - TEST_ASSERT(0); - } - } - - return EC_SUCCESS; -} - -int test_prl_no_empty_state(void) -{ - int i; - - for (i = 0; i < test_prl_sm_data_size; ++i) { - const int rv = test_no_empty_state(&test_prl_sm_data[i]); - - if (rv) { - ccprintf("PRL State machine %d has empty state!\n", i); - TEST_ASSERT(0); - } - } - - return EC_SUCCESS; -} - -int test_pe_no_empty_state(void) -{ - int i; - - for (i = 0; i < test_pe_sm_data_size; ++i) { - const int rv = test_no_empty_state(&test_pe_sm_data[i]); - - if (rv) { - ccprintf("PE State machine %d has empty state!\n", i); - TEST_ASSERT(0); - } - } - - return EC_SUCCESS; -} - static volatile int state_printed; /* Override the implement version of print */ |