warning cleanup: mark unused parameters with ARG_UNUSED macro

Use the ARG_UNUSED macro provided by Zephyr to mark function parameters
as intentionally unused to prevent compiler warnings.

Signed-off-by: Luca Burelli <l.burelli@arduino.cc>

Author: pillo79

libraries/SocketWrapper/SocketHelpers.cpp

@@ -10,6 +10,8 @@ void NetworkInterface::event_handler(struct net_mgmt_event_callback *cb, uint64_
 									 struct net_if *iface) {
 	int i = 0;
 
+	ARG_UNUSED(cb);
+
 	if (mgmt_event != NET_EVENT_IPV4_ADDR_ADD) {
 		return;
 	}
@@ -38,6 +40,10 @@ void NetworkInterface::option_handler(struct net_dhcpv4_option_callback *cb, siz
 									  enum net_dhcpv4_msg_type msg_type, struct net_if *iface) {
 	char buf[NET_IPV4_ADDR_LEN];
 
+	ARG_UNUSED(length);
+	ARG_UNUSED(msg_type);
+	ARG_UNUSED(iface);
+
 	LOG_INF("DHCP Option %d: %s", cb->option, net_addr_ntop(AF_INET, cb->data, buf, sizeof(buf)));
 }
 
@@ -185,9 +191,9 @@ void NetworkInterface::setGatewayIP(const IPAddress gateway) {
 	gw_addr.s_addr = gateway;
 	net_if_ipv4_set_gw(netif, &gw_addr);
 	LOG_INF("Gateway IP set: %s", gateway.toString().c_str());
-	return;
 }
 
 void NetworkInterface::setDnsServerIP(const IPAddress dns_server) {
-	return; // DNS server dynamic configuration is not supported
+	// DNS server dynamic configuration is not supported
+	ARG_UNUSED(dns_server);
 }

libraries/SocketWrapper/ZephyrServer.h

@@ -40,6 +40,8 @@ class ZephyrServer : public arduino::Server, ZephyrSocketWrapper {
 	}
 
 	ZephyrClient accept(uint8_t *status = nullptr) {
+		ARG_UNUSED(status);
+
 		ZephyrClient client;
 		int sock = ZephyrSocketWrapper::accept();
 		client.setSocket(sock);

libraries/WiFi/src/WiFi.cpp

@@ -12,6 +12,8 @@ String WiFiClass::firmwareVersion() {
 
 int WiFiClass::begin(const char *ssid, const char *passphrase, wl_enc_type security,
 					 bool blocking) {
+	ARG_UNUSED(security); // currently unsupported
+
 	sta_iface = net_if_get_wifi_sta();
 	netif = sta_iface;
 	sta_config.ssid = (const uint8_t *)ssid;

libraries/Wire/Wire.cpp

@@ -100,6 +100,8 @@ uint8_t arduino::ZephyrI2C::endTransmission(bool stopBit) {
 	size_t max = ring_buf_capacity_get(&txRingBuffer.rb);
 	size_t len = ring_buf_get_claim(&txRingBuffer.rb, &buf, max);
 
+	ARG_UNUSED(stopBit);
+
 	ret = i2c_write(i2c_dev, buf, len, _address);
 
 	// Must be called even if 0 bytes claimed.
@@ -115,8 +117,11 @@ uint8_t arduino::ZephyrI2C::endTransmission(void) {
 size_t arduino::ZephyrI2C::requestFrom(uint8_t address, size_t len_in, bool stopBit) {
 	int ret = -EIO;
 	uint8_t *buf = NULL;
-	size_t len = ring_buf_put_claim(&rxRingBuffer.rb, &buf, len_in);
+	size_t len;
+
+	ARG_UNUSED(stopBit);
 
+	len = ring_buf_put_claim(&rxRingBuffer.rb, &buf, len_in);
 	if (len && buf) {
 		ret = i2c_read(i2c_dev, buf, len, address);
 	}
@@ -171,6 +176,8 @@ void arduino::ZephyrI2C::onRequest(voidFuncPtr cb) {
 }
 
 int arduino::ZephyrI2C::writeRequestedCallback(struct i2c_target_config *config) {
+	ARG_UNUSED(config);
+
 	// Reset the buffer on write requests.
 	ring_buf_reset(&rxRingBuffer.rb);
 	return 0;
@@ -180,6 +187,8 @@ int arduino::ZephyrI2C::writeReceivedCallback(struct i2c_target_config *config,
 	size_t len = ring_buf_size_get(&rxRingBuffer.rb);
 	size_t max = ring_buf_capacity_get(&rxRingBuffer.rb);
 
+	ARG_UNUSED(config);
+
 	// If the buffer is about to overflow, invoke the callback
 	// with the current length.
 	if (onReceiveCb && ((len + 1) > max)) {
@@ -201,6 +210,8 @@ int arduino::ZephyrI2C::readRequestedCallback(struct i2c_target_config *config,
 }
 
 int arduino::ZephyrI2C::readProcessedCallback(struct i2c_target_config *config, uint8_t *val) {
+	ARG_UNUSED(config);
+
 	*val = 0xFF;
 	ring_buf_get(&txRingBuffer.rb, val, 1);
 	// Returning a negative value here is not handled gracefully and
@@ -209,6 +220,8 @@ int arduino::ZephyrI2C::readProcessedCallback(struct i2c_target_config *config,
 }
 
 int arduino::ZephyrI2C::stopCallback(struct i2c_target_config *config) {
+	ARG_UNUSED(config);
+
 	// If the RX buffer is not empty invoke the callback with the
 	// remaining data length.
 	if (onReceiveCb) {

libraries/Zephyr_SDRAM/examples/SDRAM_operations/SDRAM_operations.ino

@@ -36,17 +36,17 @@ void frameBuffer() {
     // simply initialize the memory as
 
     SDRAM.begin(SDRAM_START_ADDRESS + 2 * 1024 * 1024);
-    // 2MB of contiguous memory available at the beginning
 
+    // 2MB of contiguous memory available at the beginning
     uint32_t* framebuffer = (uint32_t*)SDRAM_START_ADDRESS;
 
-    // We can't allocate anymore the huge 7MB array
-
+    // We can't allocate the huge 7MB array anymore
     uint8_t* myVeryBigArray = (uint8_t*)SDRAM.malloc(7 * 1024 * 1024);
     if (myVeryBigArray == NULL) {
         Serial.println("Oops, too big :)");
     }
 
+    ARG_UNUSED(framebuffer); // suppress unused variable warning
 }
 
 void setup() {
@@ -65,4 +65,4 @@ void setup() {
 
 void loop() {
 
-}
+}