commit 2e32e815babb1a9dea747926ab9e49a1b460b749
author Andrew Boie <andrew.p.boie@intel.com> Mon Aug 14 13:46:52 2017 -0700
committer Andrew Boie <andrewboie@gmail.com> Wed Aug 16 10:59:10 2017 -0700
tree 2d50d1f9f025162e90624a44bfc02c6fdd345f6e
parent d4f116ffde0120a0d82508339820e8a36f3cf8c5

net: fix net_tcp_state_str() declaration

The second 'const' is misguided, indicating that the returns pointer
value itself cannot be changed, but since pointers are passed by value
anyway this is not useful and was generating warnings with XCC.

The leading 'const' indicates that the memory pointed to is constant,
which is all we needed.

Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>

subsys/net/ip/tcp.c

diff --git a/subsys/net/ip/tcp.c b/subsys/net/ip/tcp.c
index 6acf906..c8229ed 100644
--- a/subsys/net/ip/tcp.c
+++ b/subsys/net/ip/tcp.c
@@ -641,7 +641,7 @@
 	return 0;
 }
 
-const char * const net_tcp_state_str(enum net_tcp_state state)
+const char *net_tcp_state_str(enum net_tcp_state state)
 {
 #if defined(CONFIG_NET_DEBUG_TCP)
 	switch (state) {

subsys/net/ip/tcp.h

diff --git a/subsys/net/ip/tcp.h b/subsys/net/ip/tcp.h
index b523537..3bff901 100644
--- a/subsys/net/ip/tcp.h
+++ b/subsys/net/ip/tcp.h
@@ -215,7 +215,7 @@
 	return sys_rand32_get();
 }
 
-const char * const net_tcp_state_str(enum net_tcp_state state);
+const char *net_tcp_state_str(enum net_tcp_state state);
 
 #if defined(CONFIG_NET_TCP)
 void net_tcp_change_state(struct net_tcp *tcp, enum net_tcp_state new_state);