[U-Boot] commit 620776d causes TFTP error: 'Unsupported option(s) requested' (8)

[Pavel Machek]

On Tue 2015-08-25 14:13:17, Hannes Schmelzer wrote:
> Hi,
> 
> i've same problem.
> 
> Running tftp-hpa on Linux Mint 17.
> 
> cat /etc/default/tftpd-hpa
> # /etc/default/tftpd-hpa
> RUN_DAEMON="yes"
> TFTP_USERNAME="tftp"
> TFTP_DIRECTORY="/tftpboot/tseries"
> TFTP_ADDRESS="0.0.0.0:69"
> TFTP_OPTIONS="-l -s"

Yes. Please try patch I mailed half an hour ago... it should fix
that. (Actually here it is again, without changelog).

Sorry for the trouble,
								Pavel


Date: Tue, 25 Aug 2015 13:44:25 +0200
From: Pavel Machek <pavel at denx.de>
To: Bin Meng <bmeng.cn at gmail.com>
Cc: Joe Hershberger <joe.hershberger at ni.com>, Tom Rini
<trini at konsulko.com>, U-Boot Mailing
List <u-boot at lists.denx.de>
Subject: [PATCH] Change default tftp timeout to be	rfc-compliant
	

diff --git a/net/tftp.c b/net/tftp.c
index 18ce84c..e919638 100644
--- a/net/tftp.c
+++ b/net/tftp.c
@@ -18,8 +18,9 @@
 
 /* Well known TFTP port # */
 #define WELL_KNOWN_PORT	69
-/* Millisecs to timeout for lost pkt */
-#define TIMEOUT		100UL
+/* Millisecs to timeout for lost pkt. Anything below 1000msec is against RFC, and
+   some servers will refuse it. */
+#define TIMEOUT		1000UL
 #ifndef	CONFIG_NET_RETRY_COUNT
 /* # of timeouts before giving up */
 # define TIMEOUT_COUNT	1000

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html