[riot-notifications] [RIOT-OS/RIOT] cpu/esp32/freertos: fix semaphore take (#11239)

Julian Holzwarth notifications at github.com
Fri Mar 22 18:41:41 CET 2019


JulianHolzwarth commented on this pull request.



> +};
+
+/**
+ * @brief   shell command to test freertos mutex semaphore
+ *
+ * @param[in] argc  Number of arguments
+ * @param[in] argv  Array of arguments
+ *
+ * @return 0 on success
+ */
+static int cmd_test_mutex(int argc, char **argv)
+{
+    (void) argc;
+    (void) argv;
+    puts("starting test: mutex semaphore");
+    if (semaphore_test_mutex_take() == pdPASS) {

The test was only to test `xSemaphoreTake()` and` xSemaphoreTakeRecursive()`, with `xTicksToWait 0`. I wanted to show the bug and have a starting point for the future `freertos` library test. I hope i fixed the things you found correctly. I am sorry for the inconveniences. This is my first pr to riot.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11239#discussion_r268273741
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190322/a63e38a2/attachment-0001.html>


More information about the notifications mailing list