[U-Boot] [PATCH] net: fix typo in arp clean up

[U-Boot] [PATCH] net: fix typo in arp clean up

[Mike Frysinger]

The clean up patch missed an &, so we end up passing an int rather than
a pointer to the sprintf function.

arp.c: In function 'ArpReceive':
arp.c:197: warning: format '%p' expects type 'void *', but argument 3 has type 'int'

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
 net/arp.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/arp.c b/net/arp.c
index bac4cab..8e1d2ed 100644
--- a/net/arp.c
+++ b/net/arp.c
@@ -194,7 +194,7 @@ void ArpReceive(struct ethernet_hdr *et, struct ip_udp_hdr *ip, int len)
 #ifdef CONFIG_KEEP_SERVERADDR
 		if (NetServerIP == NetArpWaitPacketIP) {
 			char buf[20];
-			sprintf(buf, "%pM", arp->ar_sha);
+			sprintf(buf, "%pM", &arp->ar_sha);
 			setenv("serveraddr", buf);
 		}
 #endif
-- 
1.7.7.3

[U-Boot] [PATCH] net: fix typo in arp clean up

[Joe Hershberger]

Hi Mike,

On Tue, Jul 10, 2012 at 5:41 AM, Mike Frysinger <vapier@gentoo.org> wrote:
> The clean up patch missed an &, so we end up passing an int rather than
> a pointer to the sprintf function.
>
> arp.c: In function 'ArpReceive':
> arp.c:197: warning: format '%p' expects type 'void *', but argument 3 has type 'int'
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> ---

Applied, thanks.

-Joe