Reduce the number of compiler warnings

KurtE · KurtE · commit 75c3586affa6 · 2023-08-11T14:19:46.000-07:00
Resolves #77 At least partially. I did not go into the API class to fix the printf statement issue.
diff --git a/cores/arduino/IRQManager.cpp b/cores/arduino/IRQManager.cpp
@@ -342,7 +342,7 @@ bool IRQManager::addPeripheral(Peripheral_t p, void *cfg) {
     if(p == IRQ_USB && cfg != NULL) {
         USBIrqCfg_t *irqcfg = (USBIrqCfg_t *)cfg;
 
-        if (irqcfg->first_irq_number != FSP_INVALID_VECTOR) {
+        if (irqcfg->first_irq_number != (uint32_t)FSP_INVALID_VECTOR) {
             // already configured, return
             goto end_config;
         }
@@ -405,7 +405,7 @@ bool IRQManager::addPeripheral(Peripheral_t p, void *cfg) {
     else if(p == IRQ_USB_HS && cfg != NULL) {
         USBIrqCfg_t *irqcfg = (USBIrqCfg_t *)cfg;
 
-        if (irqcfg->first_irq_number != FSP_INVALID_VECTOR) {
+        if (irqcfg->first_irq_number != (uint32_t)FSP_INVALID_VECTOR) {
             // already configured, return
             goto end_config;
         }
@@ -1659,7 +1659,7 @@ void IRQManager::set_can_tx_link_event(int li, int ch)
 #endif
 }
 
-void IRQManager::set_canfd_error_link_event(int li, int ch)
+void IRQManager::set_canfd_error_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)
 {
   if (0) {}
 #ifdef ELC_EVENT_CAN0_CHERR
@@ -1674,7 +1674,7 @@ void IRQManager::set_canfd_error_link_event(int li, int ch)
 #endif
 }
 
-void IRQManager::set_canfd_rx_link_event(int li, int ch)
+void IRQManager::set_canfd_rx_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)
 {
   if (0) {}
 #ifdef ELC_EVENT_CAN0_COMFRX
@@ -1689,7 +1689,7 @@ void IRQManager::set_canfd_rx_link_event(int li, int ch)
 #endif
 }
 
-void IRQManager::set_canfd_tx_link_event(int li, int ch)
+void IRQManager::set_canfd_tx_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)
 {
   if (0) {}
 #ifdef ELC_EVENT_CAN0_TX
diff --git a/cores/arduino/cm_backtrace/cm_backtrace.c b/cores/arduino/cm_backtrace/cm_backtrace.c
@@ -117,9 +117,9 @@ static const char * const print_info[] = {
 #endif
 };
 
-static char* fw_name;
-static char* hw_ver;
-static char* sw_ver;
+static const char* fw_name;
+static const char* hw_ver;
+static const char* sw_ver;
 static uint32_t main_stack_start_addr = 0;
 static size_t main_stack_size = 0;
 static uint32_t code_start_addr = 0;

Original file line number	Diff line number	Diff line change
`@@ -342,7 +342,7 @@ bool IRQManager::addPeripheral(Peripheral_t p, void *cfg) {`
`342`	`342`	`if(p == IRQ_USB && cfg != NULL) {`
`343`	`343`	`USBIrqCfg_t irqcfg = (USBIrqCfg_t )cfg;`
`344`	`344`
`345`		`- if (irqcfg->first_irq_number != FSP_INVALID_VECTOR) {`
	`345`	`+ if (irqcfg->first_irq_number != (uint32_t)FSP_INVALID_VECTOR) {`
`346`	`346`	`// already configured, return`
`347`	`347`	`goto end_config;`
`348`	`348`	`}`
`@@ -405,7 +405,7 @@ bool IRQManager::addPeripheral(Peripheral_t p, void *cfg) {`
`405`	`405`	`else if(p == IRQ_USB_HS && cfg != NULL) {`
`406`	`406`	`USBIrqCfg_t irqcfg = (USBIrqCfg_t )cfg;`
`407`	`407`
`408`		`- if (irqcfg->first_irq_number != FSP_INVALID_VECTOR) {`
	`408`	`+ if (irqcfg->first_irq_number != (uint32_t)FSP_INVALID_VECTOR) {`
`409`	`409`	`// already configured, return`
`410`	`410`	`goto end_config;`
`411`	`411`	`}`
`@@ -1659,7 +1659,7 @@ void IRQManager::set_can_tx_link_event(int li, int ch)`
`1659`	`1659`	`#endif`
`1660`	`1660`	`}`
`1661`	`1661`
`1662`		`-void IRQManager::set_canfd_error_link_event(int li, int ch)`
	`1662`	`+void IRQManager::set_canfd_error_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)`
`1663`	`1663`	`{`
`1664`	`1664`	`if (0) {}`
`1665`	`1665`	`#ifdef ELC_EVENT_CAN0_CHERR`
`@@ -1674,7 +1674,7 @@ void IRQManager::set_canfd_error_link_event(int li, int ch)`
`1674`	`1674`	`#endif`
`1675`	`1675`	`}`
`1676`	`1676`
`1677`		`-void IRQManager::set_canfd_rx_link_event(int li, int ch)`
	`1677`	`+void IRQManager::set_canfd_rx_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)`
`1678`	`1678`	`{`
`1679`	`1679`	`if (0) {}`
`1680`	`1680`	`#ifdef ELC_EVENT_CAN0_COMFRX`
`@@ -1689,7 +1689,7 @@ void IRQManager::set_canfd_rx_link_event(int li, int ch)`
`1689`	`1689`	`#endif`
`1690`	`1690`	`}`
`1691`	`1691`
`1692`		`-void IRQManager::set_canfd_tx_link_event(int li, int ch)`
	`1692`	`+void IRQManager::set_canfd_tx_link_event(__attribute__((unused)) int li, __attribute__((unused)) int ch)`
`1693`	`1693`	`{`
`1694`	`1694`	`if (0) {}`
`1695`	`1695`	`#ifdef ELC_EVENT_CAN0_TX`