freertos: update module handles to struct type

This commit updates the handles for the timer, task and event_group
modules to struct type. This matches upstream freertos source.

Signed-off-by: Sudeep Mohanty <sudeep.mohanty@espressif.com>

components/freertos/FreeRTOS-Kernel/include/freertos/event_groups.h

@@ -84,11 +84,8 @@
  * \ingroup EventGroup
  */
 struct EventGroupDef_t;
-#ifdef ESP_PLATFORM // IDF-3770
-typedef void * EventGroupHandle_t;
-#else
-typedef struct EventGroupDef_t * EventGroupHandle_t;
-#endif // ESP_PLATFORM
+typedef struct EventGroupDef_t   * EventGroupHandle_t;
+
 /*
  * The type that holds event bits always matches TickType_t - therefore the
  * number of bits it holds is set by configUSE_16_BIT_TICKS (16 bits if set to 1,

components/freertos/FreeRTOS-Kernel/include/freertos/task.h

@@ -82,12 +82,9 @@
  * \ingroup Tasks
  */
 struct tskTaskControlBlock;     /* The old naming convention is used to prevent breaking kernel aware debuggers. */
-#ifdef ESP_PLATFORM // IDF-3769
-typedef void* TaskHandle_t;
-#else
-typedef struct tskTaskControlBlock* TaskHandle_t;
-#endif // ESP_PLATFORM
-/**
+typedef struct tskTaskControlBlock * TaskHandle_t;
+
+/*
  * Defines the prototype to which the application task hook function must
  * conform.
  */

components/freertos/FreeRTOS-Kernel/include/freertos/timers.h

@@ -75,11 +75,8 @@
  * (for example, xTimerStart(), xTimerReset(), etc.).
  */
 struct tmrTimerControl; /* The old naming convention is used to prevent breaking kernel aware debuggers. */
-#ifdef ESP_PLATFORM // IDF-3768
-typedef void* TimerHandle_t;
-#else
 typedef struct tmrTimerControl * TimerHandle_t;
-#endif // ESP_PLATFORM
+
 /*
  * Defines the prototype to which timer callback functions must conform.
  */

components/freertos/FreeRTOS-Kernel/tasks.c

@@ -5315,7 +5315,7 @@ TickType_t uxTaskResetEventItemValue( void )
 
 #if ( configUSE_MUTEXES == 1 )
 
-    void *pvTaskIncrementMutexHeldCount( void )
+    TaskHandle_t pvTaskIncrementMutexHeldCount( void )
     {
         TCB_t * curTCB;