Cast TickType_t to unsigned long when printing

concatime · concatime · commit 8581594ebf13 · 2025-02-01T18:55:49.000-05:00
Fixes Clang warnings of this type:

&gt; /source/FreeRTOS_DHCP.c:405:87: warning: format specifies type 'unsigned long' but the argument has type 'TickType_t' (aka 'unsigned int') [-Wformat]
&gt;   405 |                         FreeRTOS_debug_printf( ( "vDHCPProcess: timeout %lu ticks\n", EP_DHCPData.xDHCPTxPeriod ) );
&gt;       |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
&gt;       |                                                                         %u
&gt; /source/FreeRTOS_DHCP.c:59:33: note: expanded from macro 'EP_DHCPData'
&gt;    59 |     #define EP_DHCPData         pxEndPoint-&gt;xDHCPData               /**&lt; Temporary define to make /single source similar to /multi version. */
&gt;       |                                 ^
&gt; /build/src/FreeRTOSIPConfig.h:60:12: note: expanded from macro 'FreeRTOS_debug_printf'
&gt;    60 |     printf X
&gt;       |            ^
diff --git a/source/FreeRTOS_DHCP.c b/source/FreeRTOS_DHCP.c
@@ -402,7 +402,7 @@
 
                     if( prvSendDHCPDiscover( pxEndPoint ) == pdPASS )
                     {
-                        FreeRTOS_debug_printf( ( "vDHCPProcess: timeout %lu ticks\n", EP_DHCPData.xDHCPTxPeriod ) );
+                        FreeRTOS_debug_printf( ( "vDHCPProcess: timeout %lu ticks\n", ( unsigned long ) EP_DHCPData.xDHCPTxPeriod ) );
                     }
                     else
                     {
@@ -419,7 +419,7 @@
             }
             else
             {
-                FreeRTOS_debug_printf( ( "vDHCPProcess: giving up %lu > %lu ticks\n", EP_DHCPData.xDHCPTxPeriod, ipconfigMAXIMUM_DISCOVER_TX_PERIOD ) );
+                FreeRTOS_debug_printf( ( "vDHCPProcess: giving up %lu > %lu ticks\n", ( unsigned long ) EP_DHCPData.xDHCPTxPeriod, ( unsigned long ) ipconfigMAXIMUM_DISCOVER_TX_PERIOD ) );
 
                 #if ( ipconfigDHCP_FALL_BACK_AUTO_IP != 0 )
                 {
@@ -950,7 +950,7 @@
 
             /* Create the DHCP socket if it has not already been created. */
             prvCreateDHCPSocket( pxEndPoint );
-            FreeRTOS_debug_printf( ( "prvInitialiseDHCP: start after %lu ticks\n", dhcpINITIAL_TIMER_PERIOD ) );
+            FreeRTOS_debug_printf( ( "prvInitialiseDHCP: start after %lu ticks\n", ( unsigned long ) dhcpINITIAL_TIMER_PERIOD ) );
             vDHCP_RATimerReload( pxEndPoint, dhcpINITIAL_TIMER_PERIOD );
         }
         else
diff --git a/source/FreeRTOS_IP_Timers.c b/source/FreeRTOS_IP_Timers.c
@@ -522,7 +522,7 @@ static void prvIPTimerReload( IPTimer_t * pxTimer,
     void vDHCP_RATimerReload( NetworkEndPoint_t * pxEndPoint,
                               TickType_t uxClockTicks )
     {
-        FreeRTOS_printf( ( "vDHCP_RATimerReload: %lu\n", uxClockTicks ) );
+        FreeRTOS_printf( ( "vDHCP_RATimerReload: %lu\n", ( unsigned long ) uxClockTicks ) );
         prvIPTimerReload( &( pxEndPoint->xDHCP_RATimer ), uxClockTicks );
     }
 #endif /* ( ipconfigUSE_DHCP == 1 ) || ( ipconfigUSE_RA == 1 ) */

Original file line number	Diff line number	Diff line change
`@@ -402,7 +402,7 @@`
`402`	`402`
`403`	`403`	`if( prvSendDHCPDiscover( pxEndPoint ) == pdPASS )`
`404`	`404`	`{`
`405`		`- FreeRTOS_debug_printf( ( "vDHCPProcess: timeout %lu ticks\n", EP_DHCPData.xDHCPTxPeriod ) );`
	`405`	`+ FreeRTOS_debug_printf( ( "vDHCPProcess: timeout %lu ticks\n", ( unsigned long ) EP_DHCPData.xDHCPTxPeriod ) );`
`406`	`406`	`}`
`407`	`407`	`else`
`408`	`408`	`{`
`@@ -419,7 +419,7 @@`
`419`	`419`	`}`
`420`	`420`	`else`
`421`	`421`	`{`
`422`		`- FreeRTOS_debug_printf( ( "vDHCPProcess: giving up %lu > %lu ticks\n", EP_DHCPData.xDHCPTxPeriod, ipconfigMAXIMUM_DISCOVER_TX_PERIOD ) );`
	`422`	`+ FreeRTOS_debug_printf( ( "vDHCPProcess: giving up %lu > %lu ticks\n", ( unsigned long ) EP_DHCPData.xDHCPTxPeriod, ( unsigned long ) ipconfigMAXIMUM_DISCOVER_TX_PERIOD ) );`
`423`	`423`
`424`	`424`	`#if ( ipconfigDHCP_FALL_BACK_AUTO_IP != 0 )`
`425`	`425`	`{`
`@@ -950,7 +950,7 @@`
`950`	`950`
`951`	`951`	`/* Create the DHCP socket if it has not already been created. */`
`952`	`952`	`prvCreateDHCPSocket( pxEndPoint );`
`953`		`- FreeRTOS_debug_printf( ( "prvInitialiseDHCP: start after %lu ticks\n", dhcpINITIAL_TIMER_PERIOD ) );`
	`953`	`+ FreeRTOS_debug_printf( ( "prvInitialiseDHCP: start after %lu ticks\n", ( unsigned long ) dhcpINITIAL_TIMER_PERIOD ) );`
`954`	`954`	`vDHCP_RATimerReload( pxEndPoint, dhcpINITIAL_TIMER_PERIOD );`
`955`	`955`	`}`
`956`	`956`	`else`
Original file line number	Diff line number	Diff line change
`@@ -522,7 +522,7 @@ static void prvIPTimerReload( IPTimer_t * pxTimer,`
`522`	`522`	`void vDHCP_RATimerReload( NetworkEndPoint_t * pxEndPoint,`
`523`	`523`	`TickType_t uxClockTicks )`
`524`	`524`	`{`
`525`		`- FreeRTOS_printf( ( "vDHCP_RATimerReload: %lu\n", uxClockTicks ) );`
	`525`	`+ FreeRTOS_printf( ( "vDHCP_RATimerReload: %lu\n", ( unsigned long ) uxClockTicks ) );`
`526`	`526`	`prvIPTimerReload( &( pxEndPoint->xDHCP_RATimer ), uxClockTicks );`
`527`	`527`	`}`
`528`	`528`	`#endif /* ( ipconfigUSE_DHCP == 1 ) \|\| ( ipconfigUSE_RA == 1 ) */`