From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4] Add initial support for Wandboard dual lite and solo.
Date: Thu, 14 Mar 2013 17:01:59 -0400 [thread overview]
Message-ID: <51423AC7.3000704@ti.com> (raw)
In-Reply-To: <20130314203630.08AE920057D@gemini.denx.de>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 03/14/2013 04:36 PM, Wolfgang Denk wrote:
> Dear Tom,
>
> In message <20130314173648.GX23324@bill-the-cat> you wrote:
>>
>>>> Is there any reason for not chosing the more standard 5
>>>> second delay?
>>>
>>> Ok, so let's go with 3 seconds then ;-)
>>
>> There's a pretty even distribution of 1 3 and 5 second delays
>> (with a few 10s, 2s and 6s). If they want 1, let them have 1,
>> it's not hard to break into, you have U-Boot starting + 1sec.
>
> The question is who "they" are. My vote is for 5 :-)
>
> A quick AWK oneliner in include/configs finds this statistics:
OK, I forgot to sub -1 from 1, oops. So it's not very even, but...
>
> Value Count -1: 49 0: 12 1: 59 2: 30 3: 150 5: 175
> 6: 14 10: 38 20: 1
We have a lot of variation and no "standard" aside from "board
maintainer picks". While I'd strongly question anyone who did > 10 I
think this is a board maintainer decision. Now, if you've convinced
Fabio that 5 or 3 just makes more sense than 1 for all the right
reasons, great. But "I think 1 is too short" isn't a good one. 1 is
plenty of time to get a keypress in and stop the boot given that you
have roughly from u-boot starting to press any key, not just when the
countdown is going. Unless you have a bug in your timer code :)
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/
iQIcBAEBAgAGBQJRQjrHAAoJENk4IS6UOR1WsZIQAJv0BwogMTtfpaijt10frY2Z
9TiVbSz+GLAsHG1KpxmWG8K1D6mFVuQw6/96+8ARdZCeLdcqKXZjPLcydmmO6Ths
HLR22EcGpUDVmfgi15LUGxD/uct/MZYwl0w9NWfEQBeuluEPlGN6Rahv9gtQeZmI
U/3fBLX8S/LhX1WunVDRyDn2YBRedR2AvlLQbkd7JeondPqL72Cspnwr2QgpumJc
mqua9zXHiXzzS0HdLDoQxdwWM9wCF7Xl2+go744gvv3Rr/hot7eWwPy5DjAPDMlL
bjdhgtsg1kcjlJqdvKz3TpV5WC5bkbdHXR7G9xaNCL1ibkN5SWg9YhwelbLYGl+r
Enbov0r0jG5XHfv2QfxutPnIOb1EptH9//926Wu4V7Xc/n5aMeuRLer0Cqzni7Ps
EbjQ8PZSwLbBla9hlW6IjlzjdXxgG74k6n/+St9np5Sfe2jLkfziMoQApkxPairr
ixhCH0w+rfrMeDxZV0vubQWqG9cRlfXwWdKIbdGZm0vpZWC5wFSqbMcqTuz/Edyx
E5H4M6TGmXV+bWG/ShWeQ+ybxBgKmikyMcGYuVr9MtUgaHkwmAf+WEITUWXDi6iX
tUCR+tXwnRvMxQoPu45U1xLA2VedofvTukZaLWrarP5o7yhCU7BE7Kx4gsjQMH2V
E6PGT31eUj3m+Ila3mWd
=PZeB
-----END PGP SIGNATURE-----
next prev parent reply other threads:[~2013-03-14 21:01 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-14 2:32 [U-Boot] [PATCH v4] Add initial support for Wandboard dual lite and solo Fabio Estevam
2013-03-14 12:31 ` Wolfgang Denk
2013-03-14 17:01 ` Fabio Estevam
2013-03-14 17:36 ` Tom Rini
2013-03-14 20:36 ` Wolfgang Denk
2013-03-14 21:01 ` Tom Rini [this message]
2013-03-14 22:17 ` Scott Wood
2013-03-14 20:24 ` Wolfgang Denk
2013-03-15 14:55 ` Stefano Babic
2013-03-15 17:27 ` Wolfgang Denk
2013-03-15 18:26 ` Stefano Babic
2013-03-15 20:48 ` Wolfgang Denk
2013-03-14 20:13 ` Fabio Estevam
2013-03-14 20:37 ` Wolfgang Denk
2013-03-15 18:21 ` Fabio Estevam
2013-03-15 20:32 ` Fabio Estevam
2013-03-15 20:52 ` Wolfgang Denk
2013-03-20 10:12 ` Stefano Babic
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=51423AC7.3000704@ti.com \
--to=trini@ti.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.