Merge pull request #440 from pennam/eth-warnings

C33 Ethernet fix warnings

Author: pennam

libraries/Ethernet/src/Ethernet.cpp

@@ -123,6 +123,8 @@ int CEthernet::begin(uint8_t *mac_address, IPAddress local_ip, IPAddress dns_ser
 /* -------------------------------------------------------------------------- */
 int CEthernet::begin(uint8_t *mac, IPAddress local_ip, IPAddress dns_server, IPAddress gateway, IPAddress subnet, unsigned long timeout, unsigned long responseTimeout) {
 /* -------------------------------------------------------------------------- */  
+  (void)responseTimeout;
+  (void)timeout;
   CLwipIf::getInstance().setMacAddress(NI_ETHERNET, mac);
   return begin(local_ip, dns_server, gateway, subnet);
 }

libraries/Ethernet/src/EthernetClock.cpp

@@ -49,17 +49,14 @@ fsp_err_t EthernetClock::stop() {
   fsp_err_t err = R_AGT_Stop(&this->TIMER_ETHERNET_ctrl);
   if (err != FSP_SUCCESS) {
     return err;
-  } else {
-    err = R_AGT_Close(&this->TIMER_ETHERNET_ctrl);
-    if (err != FSP_SUCCESS) {
-      return err;
-    } else {
-      err = R_AGT_Disable(&this->TIMER_ETHERNET_ctrl);
-      if (err != FSP_SUCCESS) {
-        return err;
-      }
-    }
   }
+
+  err = R_AGT_Close(&this->TIMER_ETHERNET_ctrl);
+  if (err != FSP_SUCCESS) {
+    return err;
+  }
+
+  return R_AGT_Disable(&this->TIMER_ETHERNET_ctrl);
 }
 
 #else

libraries/Ethernet/src/EthernetDriver.cpp

@@ -195,7 +195,6 @@ void eth_reset_due_to_ADE_bit() {
 /* -------------------------------------------------------------------------- */
 void EthernetDriver::irq_callback(ether_callback_args_t * p_args) {
 /* -------------------------------------------------------------------------- */  
-    p_args->status_ecsr;
     uint32_t reg_eesr = p_args->status_eesr;
     if(p_args->channel == ETHERNET_CHANNEL) {
         if(p_args->event == ETHER_EVENT_WAKEON_LAN) {