Try to make gcc work without -Wno-address

apps/ping/ping.c

@@ -189,7 +189,7 @@ ping_recv(int s)
       if(from.ss_family == AF_INET) {
         struct sockaddr_in *from4 = (struct sockaddr_in*)&from;
         inet_addr_to_ip4addr(ip_2_ip4(&fromaddr), &from4->sin_addr);
-        IP_SET_TYPE(&fromaddr, IPADDR_TYPE_V4);
+        IP_SET_TYPE_VAL(fromaddr, IPADDR_TYPE_V4);
       }
 #endif /* LWIP_IPV4 */
 
@@ -197,12 +197,12 @@ ping_recv(int s)
       if(from.ss_family == AF_INET6) {
         struct sockaddr_in6 *from6 = (struct sockaddr_in6*)&from;
         inet6_addr_to_ip6addr(ip_2_ip6(&fromaddr), &from6->sin6_addr);
-        IP_SET_TYPE(&fromaddr, IPADDR_TYPE_V6);
+        IP_SET_TYPE_VAL(fromaddr, IPADDR_TYPE_V6);
       }
 #endif /* LWIP_IPV6 */
       
       LWIP_DEBUGF( PING_DEBUG, ("ping: recv "));
-      ip_addr_debug_print(PING_DEBUG, &fromaddr);
+      ip_addr_debug_print_val(PING_DEBUG, fromaddr);
       LWIP_DEBUGF( PING_DEBUG, (" %"U32_F" ms\n", (sys_now() - ping_time)));
 
       /* todo: support ICMP6 echo */

apps/shell/shell.c

@@ -886,7 +886,7 @@ com_udpb(struct command *com)
   }
 
 #if LWIP_IPV4
-  if (IP_IS_V6(&ipaddr)) {
+  if (IP_IS_V6_VAL(ipaddr)) {
     err = netconn_bind(conns[i], &ip_addr_broadcast, lport);
     if (err != ERR_OK) {
       netconn_delete(conns[i]);

ports/Common.allports.mk

@@ -37,7 +37,7 @@ CFLAGS+=-g -Wall -DLWIP_DEBUG -pedantic -Werror \
 	-Wparentheses -Wsequence-point -Wswitch-default \
 	-Wextra -Wundef -Wshadow -Wpointer-arith -Wcast-qual \
 	-Wc++-compat -Wwrite-strings -Wold-style-definition -Wcast-align \
-	-Wmissing-prototypes -Wredundant-decls -Wnested-externs -Wno-address \
+	-Wmissing-prototypes -Wredundant-decls -Wnested-externs \
 	-Wunreachable-code -Wuninitialized
 
 ifeq (,$(findstring clang,$(CC)))

ports/unix/port/include/arch/sys_arch.h

@@ -39,18 +39,22 @@ typedef u32_t sys_prot_t;
 
 struct sys_sem;
 typedef struct sys_sem * sys_sem_t;
-#define sys_sem_valid(sem)       (((sem) != NULL) && (*(sem) != NULL))
-#define sys_sem_set_invalid(sem) do { if((sem) != NULL) { *(sem) = NULL; }}while(0)
+#define sys_sem_valid(sem)             (((sem) != NULL) && (*(sem) != NULL))
+#define sys_sem_valid_val(sem)         ((sem) != NULL)
+#define sys_sem_set_invalid(sem)       do { if((sem) != NULL) { *(sem) = NULL; }}while(0)
+#define sys_sem_set_invalid_val(sem)   do { (sem) = NULL; }while(0)
 
 struct sys_mutex;
 typedef struct sys_mutex * sys_mutex_t;
-#define sys_mutex_valid(mutex)       sys_sem_valid(mutex)
-#define sys_mutex_set_invalid(mutex) sys_sem_set_invalid(mutex)
+#define sys_mutex_valid(mutex)         sys_sem_valid(mutex)
+#define sys_mutex_set_invalid(mutex)   sys_sem_set_invalid(mutex)
 
 struct sys_mbox;
 typedef struct sys_mbox * sys_mbox_t;
-#define sys_mbox_valid(mbox)       sys_sem_valid(mbox)
-#define sys_mbox_set_invalid(mbox) sys_sem_set_invalid(mbox)
+#define sys_mbox_valid(mbox)           sys_sem_valid(mbox)
+#define sys_mbox_valid_val(mbox)       sys_sem_valid_val(mbox)
+#define sys_mbox_set_invalid(mbox)     sys_sem_set_invalid(mbox)
+#define sys_mbox_set_invalid_val(mbox) sys_sem_set_invalid_val(mbox)
 
 struct sys_thread;
 typedef struct sys_thread * sys_thread_t;