From: Luuk Paulussen <luuk.paulus...@alliedtelesis.co.nz> When we use the ntpserverip environment variable argv[1] may not be set. Printing the error message using the NetNtpServerIP variable ensures the correct output in both cases.
Signed-off-by: Luuk Paulussen <luuk.paulus...@alliedtelesis.co.nz> Acked-by: Chris Packham <chris.pack...@alliedtelesis.co.nz> Cc: Ben Warren <biggerbadder...@gmail.com> --- Changes since v1: - run through checkpatch.pl, fix line > 80 chars common/cmd_net.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/common/cmd_net.c b/common/cmd_net.c index 8c6f5c8..fae3c7f 100644 --- a/common/cmd_net.c +++ b/common/cmd_net.c @@ -324,7 +324,8 @@ int do_sntp (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) else NetTimeOffset = simple_strtol (toff, NULL, 10); if (NetLoop(SNTP) < 0) { - printf("SNTP failed: host %s not responding\n", argv[1]); + printf("SNTP failed: host %pI4 not responding\n", + &NetNtpServerIP); return 1; } -- 1.7.4.1 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot