Fix timeout for Freertos semaphore.

It doesn't make sense to have portMaxDelay and then recalculate to lower value. Currently other ports are using simple implementation so i align with that.

Signed-off-by: Cervenka Dusan <[email protected]>

Author: Hadatko

erpc_c/port/erpc_setup_extensions_freertos.cpp

@@ -28,7 +28,7 @@ void erpc::erpc_pre_cb_default(void)
     (void)s_erpc_call_in_progress->get(s_erpc_call_in_progress->kWaitForever);
     erpc_assert(s_erpc_call_timer_cb &&
            "If you want use default pre cb action, do not forget call erpc_init_call_progress_detection_default.");
-    xTimerStart(s_erpc_call_timer_cb, 0);
+    (void)xTimerStart(s_erpc_call_timer_cb, 0);
 }
 
 void erpc::erpc_post_cb_default(void)

erpc_c/port/erpc_threading_freertos.cpp

@@ -271,16 +271,17 @@ bool Semaphore::get(uint32_t timeout)
     {
         timeout = portMAX_DELAY;
     }
+    else if (timeout > (portMAX_DELAY - 1))
+    {
+        timeout = portMAX_DELAY - 1;
+    }
     else
     {
-        if (timeout > (portMAX_DELAY - 1))
-        {
-            timeout = portMAX_DELAY - 1;
-        }
+        /* Misra condition */
     }
 #endif
 
-    return (pdTRUE == xSemaphoreTake(m_sem, timeout / 1000U / portTICK_PERIOD_MS));
+    return (pdTRUE == xSemaphoreTake(m_sem, timeout));
 }
 
 int Semaphore::getCount(void) const