cores/arduino: warning squash

Silence a number of trivial compiler warnings in the Arduino core:
- unused arguments
- "constexpr const"
- signed/unsigned mismatches

Signed-off-by: Luca Burelli <[email protected]>

Author: pillo79

cores/arduino/USB.cpp

@@ -14,6 +14,7 @@
 const struct device *const usb_dev = DEVICE_DT_GET(DT_PHANDLE_BY_IDX(DT_PATH(zephyr_user), cdc_acm, 0));
 
 void usb_status_cb(enum usb_dc_status_code cb_status, const uint8_t *param) {
+    (void)param; // unused
     if (cb_status == USB_DC_CONFIGURED) {
 
     }
@@ -34,6 +35,7 @@ void arduino::SerialUSB_::_baudChangeHandler()
 
 static void _baudChangeHandler(const struct device *dev, uint32_t rate)
 {
+    (void)dev; // unused
     if (rate == 1200) {
         usb_disable();
         _on_1200_bps();

cores/arduino/abi.cpp

@@ -6,8 +6,12 @@
 
 #include <stdlib.h>
 
-extern "C" void __cxa_pure_virtual(void) __attribute__ ((__noreturn__));
-extern "C" void __cxa_deleted_virtual(void) __attribute__ ((__noreturn__));
+extern "C" void __cxa_pure_virtual(void) {}
+extern "C" void __cxa_deleted_virtual(void) {}
+extern "C" int __cxa_atexit(void (*func) (void *), void * arg, void * dso_handle) {
+  (void)func; (void)arg; (void)dso_handle; // unused
+  return 0;
+}
 
 namespace std {
   [[gnu::weak, noreturn]] void terminate() {

cores/arduino/main.cpp

@@ -68,7 +68,9 @@ static void __libc_init_array (void)
     __init_array_start[i] ();
 }
 
-extern "C" __attribute__((section(".entry_point"), used)) void entry_point(k_thread_stack_t* stack, size_t stack_size) {
+extern "C" __attribute__((section(".entry_point"), used)) void entry_point(struct k_heap* stack, size_t stack_size) {
+  (void)stack; (void)stack_size; // unused
+
   // copy .data in the right place
   // .bss should already be in the right place
   // call constructors
@@ -84,11 +86,9 @@ extern "C" __attribute__((section(".entry_point"), used)) void entry_point(k_thr
 
   //__asm volatile ("cpsie i");
 
-  const size_t alignment = 4096;
   printk("System Heap end: %p\n", &__heap_end);
   printk("System Heap start: %p\n", &__heap_start);
   printk("Sketch Heap start: %p, size %p\n", &kheap_llext_heap, &kheap_llext_heap_size);
-  // __heap_start = (__heap_start + (alignment - 1)) & ~(alignment - 1);
 
   memcpy(&_sdata, &_sidata, (&_edata - &_sdata) * sizeof(uint32_t));
   memset(&_sbss, 0, &_ebss - &_sbss);

cores/arduino/new.cpp

@@ -46,14 +46,17 @@ void * operator new[](std::size_t size) {
 
 #if __cplusplus >= 201703L
 void* operator new(std::size_t count, std::align_val_t al) {
+  (void)al; // unused
   return operator new(count);
 }
 
 void* operator new[](std::size_t count, std::align_val_t al) {
+  (void)al; // unused
   return operator new(count);
 }
 
 void * operator new(std::size_t size, std::align_val_t al, const std::nothrow_t tag) noexcept {
+  (void)al; (void)tag; // unused
 #if defined(NEW_TERMINATES_ON_FAILURE)
   // Cannot call throwing operator new as standard suggests, so call
   // new_helper directly then
@@ -64,6 +67,7 @@ void * operator new(std::size_t size, std::align_val_t al, const std::nothrow_t
 }
 
 void * operator new[](std::size_t size, std::align_val_t al, const std::nothrow_t& tag) noexcept {
+  (void)al; (void)tag; // unused
 #if defined(NEW_TERMINATES_ON_FAILURE)
   // Cannot call throwing operator new[] as standard suggests, so call
   // malloc directly then
@@ -75,6 +79,7 @@ void * operator new[](std::size_t size, std::align_val_t al, const std::nothrow_
 #endif
 
 void * operator new(std::size_t size, const std::nothrow_t tag) noexcept {
+  (void)tag; // unused
 #if defined(NEW_TERMINATES_ON_FAILURE)
   // Cannot call throwing operator new as standard suggests, so call
   // new_helper directly then
@@ -84,6 +89,7 @@ void * operator new(std::size_t size, const std::nothrow_t tag) noexcept {
 #endif
 }
 void * operator new[](std::size_t size, const std::nothrow_t& tag) noexcept {
+  (void)tag; // unused
 #if defined(NEW_TERMINATES_ON_FAILURE)
   // Cannot call throwing operator new[] as standard suggests, so call
   // malloc directly then
@@ -111,17 +117,21 @@ void operator delete[](void * ptr) noexcept {
 
 #if __cplusplus >= 201402L
 void operator delete(void* ptr, std::size_t size) noexcept {
+  (void)size; // unused
   operator delete(ptr);
 }
 void operator delete[](void * ptr, std::size_t size) noexcept {
+  (void)size; // unused
   operator delete[](ptr);
 }
 #endif // __cplusplus >= 201402L
 
 void operator delete(void* ptr, const std::nothrow_t& tag) noexcept {
+  (void)tag; // unused
   operator delete(ptr);
 }
 void operator delete[](void* ptr, const std::nothrow_t& tag) noexcept {
+  (void)tag; // unused
   operator delete[](ptr);
 }
 

cores/arduino/zephyrCommon.cpp

@@ -68,24 +68,24 @@ inline int global_gpio_pin_configure(pin_size_t pinNumber, int flags) {
   return gpio_pin_configure(local_gpio_port(pinNumber), local_gpio_pin(pinNumber), flags);
 }
 
-template <class N, class Head> constexpr const N sum_of_list(const N sum, const Head &head)
+template <class N, class Head> constexpr N sum_of_list(const N sum, const Head &head)
 {
   return sum + head;
 }
 
 template <class N, class Head, class... Tail>
-constexpr const N sum_of_list(const N sum, const Head &head, const Tail &...tail)
+constexpr N sum_of_list(const N sum, const Head &head, const Tail &...tail)
 {
   return sum_of_list(sum + head, tail...);
 }
 
-template <class N, class Head> constexpr const N max_in_list(const N max, const Head &head)
+template <class N, class Head> constexpr N max_in_list(const N max, const Head &head)
 {
   return (max >= head) ? max : head;
 }
 
 template <class N, class Head, class... Tail>
-constexpr const N max_in_list(const N max, const Head &head, const Tail &...tail)
+constexpr N max_in_list(const N max, const Head &head, const Tail &...tail)
 {
   return max_in_list((max >= head) ? max : head, tail...);
 }

cores/arduino/zephyrSerial.cpp

@@ -111,6 +111,7 @@ void arduino::ZephyrSerial::IrqHandler()
 
 void arduino::ZephyrSerial::IrqDispatch(const struct device *dev, void *data)
 {
+	(void)dev; // unused
 	reinterpret_cast<ZephyrSerial *>(data)->IrqHandler();
 }
 
@@ -149,7 +150,7 @@ int arduino::ZephyrSerial::read()
 
 size_t arduino::ZephyrSerial::write(const uint8_t *buffer, size_t size)
 {
-	int ret;
+	size_t idx = 0;
 
 	k_sem_take(&tx.sem, K_FOREVER);
 	ret = ring_buf_put(&tx.ringbuf, buffer, size);