From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/3] net: Make TFTP server timeout configurable
Date: Mon, 22 Sep 2008 22:42:13 +0200 [thread overview]
Message-ID: <20080922204213.87B33248EE@gemini.denx.de> (raw)
In-Reply-To: <48D33E23.3020502@semihalf.com>
Dear Bartek,
In message <48D33E23.3020502@semihalf.com> you wrote:
>
> > Are seconds an appropriate scale factor for the timeout?
...
> The patch didn't introduce changes in this regard -- TFTP timeouts were
> defined in seconds originally. The patch makes the timeout behavior
> configurable, instead of being hardcoded, but the units remain the same:
But the argument was a perfectly valid one. I am, too, concerned that
second resolution is not optimal any more (it was as long as we needed
just one timeout long enough, but this situation will be changed by
your other patches).
So please extend that patch to change this into milliseconds as
discussed before.
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Humans do claim a great deal for that particular emotion (love).
-- Spock, "The Lights of Zetar", stardate 5725.6
next prev parent reply other threads:[~2008-09-22 20:42 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-18 15:03 [U-Boot] Automatic software updates in U-Boot Bartlomiej Sieka
2008-09-18 15:03 ` [U-Boot] [PATCH 1/3] net: Make TFTP server timeout configurable Bartlomiej Sieka
2008-09-18 15:17 ` Jerry Van Baren
2008-09-18 21:54 ` Graeme Russ
2008-09-19 1:37 ` Jerry Van Baren
2008-09-19 5:52 ` Bartlomiej Sieka
2008-09-22 20:42 ` Wolfgang Denk [this message]
2008-09-25 6:17 ` Bartlomiej Sieka
2008-09-18 15:03 ` [U-Boot] [PATCH 2/3] Automatic software update from TFTP server Bartlomiej Sieka
2008-09-22 21:03 ` Wolfgang Denk
2008-09-25 8:16 ` Bartlomiej Sieka
2008-09-25 8:55 ` Wolfgang Denk
2008-09-25 9:49 ` Bartlomiej Sieka
2008-09-25 10:01 ` Wolfgang Denk
2008-09-26 8:46 ` Detlev Zundel
2008-09-25 15:33 ` Andrew Dyer
2008-09-25 16:28 ` Kim Phillips
2008-09-25 16:24 ` Jerry Van Baren
2008-09-25 18:17 ` Wolfgang Denk
2008-09-25 19:21 ` Kim Phillips
2008-09-26 8:08 ` Bartlomiej Sieka
2008-09-26 8:29 ` Wolfgang Denk
2008-09-26 8:29 ` Bartlomiej Sieka
2008-09-18 15:03 ` [U-Boot] [PATCH 3/3] FIT: output image load address for type 'firmware', fix debug msg while there Bartlomiej Sieka
2008-10-01 13:26 ` [U-Boot] Automatic software updates in U-Boot -- version 2 Bartlomiej Sieka
2008-10-13 22:57 ` Wolfgang Denk
2008-10-14 12:42 ` Bartlomiej Sieka
2008-10-14 13:47 ` Wolfgang Denk
2008-10-14 20:07 ` Wolfgang Denk
2008-10-15 9:45 ` Bartlomiej Sieka
2008-10-01 13:26 ` [U-Boot] [PATCH v2 1/6] flash: factor out adjusting of Flash address to the end of sector Bartlomiej Sieka
2008-10-08 7:39 ` Bartlomiej Sieka
2008-10-08 7:47 ` Stefan Roese
2008-10-08 8:05 ` Bartlomiej Sieka
2008-10-08 8:16 ` Wolfgang Denk
2008-10-08 12:02 ` Stefan Roese
2008-10-13 22:28 ` Wolfgang Denk
2008-10-08 17:38 ` Ben Warren
2008-10-13 22:30 ` Wolfgang Denk
2008-10-01 13:26 ` [U-Boot] [PATCH v2 2/6] net: express the first argument to NetSetTimeout() in milliseconds Bartlomiej Sieka
2008-10-06 5:13 ` Ben Warren
2008-10-01 13:26 ` [U-Boot] [PATCH v2 3/6] net: Make TFTP server timeout configurable Bartlomiej Sieka
2008-10-06 5:13 ` Ben Warren
2008-10-01 13:26 ` [U-Boot] [PATCH v2 4/6] Restore alphabetic ordering in common/Makefile Bartlomiej Sieka
2008-10-08 12:29 ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-08 12:58 ` Bartlomiej Sieka
2008-10-08 14:50 ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-09 8:40 ` Bartlomiej Sieka
2008-10-01 13:26 ` [U-Boot] [PATCH v2 5/6] Automatic software update from TFTP server Bartlomiej Sieka
2008-10-13 22:37 ` Wolfgang Denk
2008-10-01 13:26 ` [U-Boot] [PATCH v2 6/6] FIT: output image load address for type 'firmware', fix message while there Bartlomiej Sieka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080922204213.87B33248EE@gemini.denx.de \
--to=wd@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox