Two functions are declared "static" in forward declaration, but not in the actual code, so sparse complains. Signed-off-by: Stephen Hemminger <shemminger@xxxxxxxxxxxxxxxxxxxx> --- a/drivers/net/slhc.c 2007-08-06 04:25:48.000000000 -0400 +++ b/drivers/net/slhc.c 2007-08-17 14:55:24.000000000 -0400 @@ -174,7 +174,7 @@ put16(unsigned char *cp, unsigned short /* Encode a number */ -unsigned char * +static unsigned char * encode(unsigned char *cp, unsigned short n) { if(n >= 256 || n == 0){ @@ -199,7 +199,7 @@ pull16(unsigned char **cpp) } /* Decode a number */ -long +static long decode(unsigned char **cpp) { register int x; - To unsubscribe from this list: send the line "unsubscribe linux-ppp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html