alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

fix(lp_core_test): fixed race-condition in lp core tests

Browse Source
This commit is contained in:
Marius Vikhammer 2024-05-15 10:50:23 +08:00
parent 70fc0e3037
commit d885b47eaf
2 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
components/ulp/test_apps/lp_core/main/lp_core/test_main.c
View File

@ -73,11 +73,9 @@ void handle_commands(lp_core_test_commands_t cmd)
break; break;
case LP_CORE_NO_COMMAND: case LP_CORE_NO_COMMAND:
main_cpu_reply = LP_CORE_COMMAND_NOK;
break; break;
default: default:
main_cpu_reply = LP_CORE_COMMAND_NOK;
break; break;
} }
} }

16
components/ulp/test_apps/lp_core/main/test_lp_core.c
View File

@ -41,6 +41,12 @@ static void load_and_start_lp_core_firmware(ulp_lp_core_cfg_t* cfg, const uint8_
} }
static void clear_test_cmds(void)
{
ulp_main_cpu_command = LP_CORE_NO_COMMAND;
ulp_command_resp = LP_CORE_NO_COMMAND;
}
TEST_CASE("LP core and main CPU are able to exchange data", "[lp_core]") TEST_CASE("LP core and main CPU are able to exchange data", "[lp_core]")
{ {
const uint32_t test_data = 0x12345678; const uint32_t test_data = 0x12345678;
@ -70,9 +76,7 @@ TEST_CASE("LP core and main CPU are able to exchange data", "[lp_core]")
printf("data out: 0x%" PRIx32 ", expected: 0x%" PRIx32 " \n", ulp_test_data_out, test_data); printf("data out: 0x%" PRIx32 ", expected: 0x%" PRIx32 " \n", ulp_test_data_out, test_data);
TEST_ASSERT(test_data == ulp_test_data_out); TEST_ASSERT(test_data == ulp_test_data_out);
/* Clear test data */ clear_test_cmds();
ulp_main_cpu_command = LP_CORE_NO_COMMAND;
ulp_command_resp = LP_CORE_NO_COMMAND;
} }
TEST_CASE("Test LP core delay", "[lp_core]") TEST_CASE("Test LP core delay", "[lp_core]")
@ -107,9 +111,7 @@ TEST_CASE("Test LP core delay", "[lp_core]")
printf("Waited for %" PRIi64 "us, expected: %" PRIi32 "us\n", diff, delay_period_us); printf("Waited for %" PRIi64 "us, expected: %" PRIi32 "us\n", diff, delay_period_us);
TEST_ASSERT_INT_WITHIN(delta_us, delay_period_us, diff); TEST_ASSERT_INT_WITHIN(delta_us, delay_period_us, diff);
/* Clear test data */ clear_test_cmds();
ulp_main_cpu_command = LP_CORE_NO_COMMAND;
ulp_command_resp = LP_CORE_NO_COMMAND;
} }
@ -223,6 +225,8 @@ static void check_reset_reason_and_sleep_duration(void)
printf("CPU slept for %"PRIi64" ms, expected it to sleep approx %"PRIi64" ms\n", sleep_duration, expected_sleep_duration_ms); printf("CPU slept for %"PRIi64" ms, expected it to sleep approx %"PRIi64" ms\n", sleep_duration, expected_sleep_duration_ms);
/* Rough estimate, as CPU spends quite some time waking up, but will test if lp core is waking up way too often etc */ /* Rough estimate, as CPU spends quite some time waking up, but will test if lp core is waking up way too often etc */
TEST_ASSERT_INT_WITHIN_MESSAGE(1000, expected_sleep_duration_ms, sleep_duration, "LP Core did not wake up the expected number of times"); TEST_ASSERT_INT_WITHIN_MESSAGE(1000, expected_sleep_duration_ms, sleep_duration, "LP Core did not wake up the expected number of times");
clear_test_cmds();
} }
TEST_CASE_MULTIPLE_STAGES("LP Timer can wakeup lp core periodically during deep sleep", "[ulp]", TEST_CASE_MULTIPLE_STAGES("LP Timer can wakeup lp core periodically during deep sleep", "[ulp]",