embestmx6boards: Increase boot delay to 3s
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
From: Nikolay Dimitrov <[email protected]>
--- include/configs/embestmx6boards.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
include/configs/embestmx6boards.h +1 -1
@@ -120,7 +120,7 @@#define CONFIG_CMD_SETEXPR #undef CONFIG_CMD_IMLS -#define CONFIG_BOOTDELAY 1 +#define CONFIG_BOOTDELAY 3#define CONFIG_LOADADDR 0x12000000#define CONFIG_SYS_TEXT_BASE 0x17800000
Signature · 1 line
1.7.10.4
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :
Quoted 12 lines
From: Nikolay Dimitrov <[email protected]>
Signed-off-by Nikolay Dimitrov <[email protected]>--- include/configs/embestmx6boards.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)include/configs/embestmx6boards.h +0
-0
include/configs/embestmx6boards.h line 126
Quoted 8 lines
include/configs/embestmx6boards.h +1
-1@@ -120,7 +120,7 @@#define CONFIG_CMD_SETEXPR #undef CONFIG_CMD_IMLS -#define CONFIG_BOOTDELAY 1 +#define CONFIG_BOOTDELAY 3#define CONFIG_LOADADDR 0x12000000#define CONFIG_SYS_TEXT_BASE 0x17800000
This will increase the boot time by 2 seconds : why is that needed ?
Best regards,
Eric
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi
Quoted 5 lines · Eric B?nard
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :
Can you just change your enviroment? if I remember correctly you can override boot delay
Michael
Quoted 29 lines
Quoted 12 lines
From: Nikolay Dimitrov <[email protected]>
Signed-off-by Nikolay Dimitrov <[email protected]>--- include/configs/embestmx6boards.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)include/configs/embestmx6boards.h +0
-0include/configs/embestmx6boards.h line 126
Quoted 8 lines
include/configs/embestmx6boards.h +1
-1@@ -120,7 +120,7 @@#define CONFIG_CMD_SETEXPR #undef CONFIG_CMD_IMLS -#define CONFIG_BOOTDELAY 1 +#define CONFIG_BOOTDELAY 3#define CONFIG_LOADADDR 0x12000000#define CONFIG_SYS_TEXT_BASE 0x17800000This will increase the boot time by 2 seconds : why is that needed ?
Best regards,
Eric
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi Michael,
Quoted 11 lines · Michael Trimarchi
Hi
Quoted 5 lines · Eric B?nard
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :Can you just change your enviroment? if I remember correctly you can override boot delay
Of course I can change the environment, but my point was towards improving the default "out-of-the-box" behavior.
Kind regards,
Nikolay
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi
Il 14/dic/2014 00:07 "Nikolay Dimitrov" <[email protected]> ha scritto:
Quoted 16 lines
Hi Michael,
Quoted 11 lines · Michael Trimarchi
Hi
Quoted 6 lines · Eric B?nard
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :
Quoted 7 lines
Quoted 2 lines
Can you just change your enviroment? if I remember correctly you can override boot delay
Of course I can change the environment, but my point was towards improving the default "out-of-the-box" behavior.
I understand your point but I prefer a config option in configuration instead of change it. I'm not have this board but most of the time I always change something I don't like create my own environment or config.
Michael
Quoted 2 lines
Kind regards,
Nikolay
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi Michael,
Quoted 25 lines · Michael Trimarchi
Hi
Il 14/dic/2014 00:07 "Nikolay Dimitrov" <[email protected] <mailto:[email protected]>> ha scritto:
Quoted 9 lines
Hi Michael,
Quoted 4 lines · Michael Trimarchi
Hi
On Sat, Dec 13, 2014 at 10:16 PM, Eric B?nard <[email protected]
Quoted 11 lines · <mailto:[email protected]>> wrote:
Quoted 9 lines
Quoted 6 lines
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg <mailto:[email protected]> a ?crit :Can you just change your enviroment? if I remember correctly you can override boot delay
Quoted 7 lines
Quoted 3 lines
Of course I can change the environment, but my point was towards improving the default "out-of-the-box" behavior.
I understand your point but I prefer a config option in configuration instead of change it. I'm not have this board but most of the time I always change something I don't like create my own environment or config.
Well, the "bootdelay" env-var is still there, so everyone can take advantage of it.
I have both RIoTboard & MarsBoard, and my practical experience with them motivated me to propose this optimisation of the default U-Boot behavior.
Kind regards,
Nikolay
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
In reply to Eric BénardHi Eric,
Quoted 20 lines · Eric B?nard
Hi Nikolay,
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :Quoted 15 lines
From: Nikolay Dimitrov <[email protected]>
Signed-off-by Nikolay Dimitrov <[email protected]>--- include/configs/embestmx6boards.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)include/configs/embestmx6boards.h +0
-0@@ -120,7 +120,7 @@#define CONFIG_CMD_SETEXPR #undef CONFIG_CMD_IMLS
Quoted 7 lines
include/configs/embestmx6boards.h line 126
Quoted 5 lines
include/configs/embestmx6boards.h +1
-1-#define CONFIG_BOOTDELAY 1 +#define CONFIG_BOOTDELAY 3#define CONFIG_LOADADDR 0x12000000#define CONFIG_SYS_TEXT_BASE 0x17800000This will increase the boot time by 2 seconds : why is that needed ?
Thanks for commenting. My intent is to give by default a reasonable time for the end-user to react and break the boot-sequence if needed (and avoid the nasty racing against the clock for this precious key-press).
My motivation for this is that I see RIoTboard/MarsBoard mostly as boards that people can use for experimentation/prototyping, and not as a finished end-product, thus my thinking that the slightly increased boot-time won't be a critical issue for such applications. Still everyone can change the delay at will if they want to optimize the boot.
Other hobby boards also have similarly handy boot-timings, like Wandboard (5s), Novena (5s), Udoo (3s).
Kind regards,
Nikolay
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi Nikolay,
Le Sat, 13 Dec 2014 18:05:15 +0200,
Nikolay Dimitrov <[email protected]> a ?crit :
Quoted 32 lines · Eric B?nard
Quoted 18 lines
Le Fri, 12 Dec 2014 20:16:19 +0200,
picmaster at mail.bg a ?crit :Quoted 15 lines
From: Nikolay Dimitrov <[email protected]>
Signed-off-by Nikolay Dimitrov <[email protected]>--- include/configs/embestmx6boards.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)include/configs/embestmx6boards.h +0
-0@@ -120,7 +120,7 @@#define CONFIG_CMD_SETEXPR #undef CONFIG_CMD_IMLSQuoted 7 lines
include/configs/embestmx6boards.h line 126
Quoted 5 lines
include/configs/embestmx6boards.h +1
-1-#define CONFIG_BOOTDELAY 1 +#define CONFIG_BOOTDELAY 3#define CONFIG_LOADADDR 0x12000000#define CONFIG_SYS_TEXT_BASE 0x17800000This will increase the boot time by 2 seconds : why is that needed ?
Thanks for commenting. My intent is to give by default a reasonable time for the end-user to react and break the boot-sequence if needed (and avoid the nasty racing against the clock for this precious key-press).
Quoted 9 lines
My motivation for this is that I see RIoTboard/MarsBoard mostly as boards that people can use for experimentation/prototyping, and not as a finished end-product, thus my thinking that the slightly increased boot-time won't be a critical issue for such applications. Still everyone can change the delay at will if they want to optimize the boot.
Other hobby boards also have similarly handy boot-timings, like Wandboard (5s), Novena (5s), Udoo (3s).
OK seems reasonable.
Eric
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Quoted 19 lines · Eric B?nard
Hi Nikolay,
Le Sat, 13 Dec 2014 18:05:15 +0200,
Nikolay Dimitrov <[email protected]> a ?crit :Quoted 15 lines · Eric B?nard
Quoted 2 lines
This will increase the boot time by 2 seconds : why is that needed ?
Thanks for commenting. My intent is to give by default a reasonable time for the end-user to react and break the boot-sequence if needed (and avoid the nasty racing against the clock for this precious key-press).
My motivation for this is that I see RIoTboard/MarsBoard mostly as boards that people can use for experimentation/prototyping, and not as a finished end-product, thus my thinking that the slightly increased boot-time won't be a critical issue for such applications. Still everyone can change the delay at will if they want to optimize the boot.
Quoted 4 lines
Quoted 3 lines
Other hobby boards also have similarly handy boot-timings, like Wandboard (5s), Novena (5s), Udoo (3s).
OK seems reasonable.
I disagree. I think it's an unnecessary, unneeded, unreasonable tripling of the boot time.
Rather than leaving people to patch out this unwanted behaviour, I'd suggest promoting CONFIG_BOOTDELAY into a Kconfig option, leaving the default as-is.
This allows the people who want it to configure it easily.
In any case the rationale given, that the Riotboard is a dev board used for experimenting, doesn't imply a change like this one. It implies a conversion to use config_distro_bootcmd.h like the patches I've just posted. This also leads to a slightly less unreasonable increase in bootdelay to 2s from config_distro_defaults.h
Using config_distro_bootcmd also means that we can largely ignore the u-boot shell altogether and just use extlinux.conf instead
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Hi
Quoted 23 lines · Iain Paton
Quoted 22 lines · Eric B?nard
Hi Nikolay,
Le Sat, 13 Dec 2014 18:05:15 +0200,
Nikolay Dimitrov <[email protected]> a ?crit :Quoted 18 lines · Eric B?nard
Quoted 2 lines
This will increase the boot time by 2 seconds : why is that needed ?
Thanks for commenting. My intent is to give by default a reasonable time for the end-user to react and break the boot-sequence if needed (and avoid the nasty racing against the clock for this precious key-press).
My motivation for this is that I see RIoTboard/MarsBoard mostly as boards that people can use for experimentation/prototyping, and not as a finished end-product, thus my thinking that the slightly increased boot-time won't be a critical issue for such applications. Still everyone can change the delay at will if they want to optimize the boot.
Other hobby boards also have similarly handy boot-timings, like Wandboard (5s), Novena (5s), Udoo (3s).
Quoted 8 lines
Quoted 1 line
OK seems reasonable.
I disagree. I think it's an unnecessary, unneeded, unreasonable tripling of the boot time.
Rather than leaving people to patch out this unwanted behaviour, I'd suggest promoting
CONFIG_BOOTDELAYinto aKconfigoption, leaving the default as-is.
I'm agree and this is what I say in my previous email
Michael
Quoted 14 lines
This allows the people who want it to configure it easily.
In any case the rationale given, that the Riotboard is a dev board used for experimenting, doesn't imply a change like this one. It implies a conversion to use
config_distro_bootcmd.hlike the patches I've just posted. This also leads to a slightly less unreasonable increase in bootdelay to 2s fromconfig_distro_defaults.hUsing
config_distro_bootcmdalso means that we can largely ignore the u-boot shell altogether and just use extlinux.conf instead
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Signature · 4 lines
| Michael Nazzareno Trimarchi Amarula Solutions BV | | COO - Founder Cruquiuskade 47 | | +31(0)851119172 Amsterdam 1018 AM NL | | [`as] http://www.amarulasolutions.com |
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
In reply to Iain PatonHi Iain,
Quoted 20 lines · Iain Paton
Quoted 19 lines · Eric B?nard
Hi Nikolay,
Le Sat, 13 Dec 2014 18:05:15 +0200,
Nikolay Dimitrov <[email protected]> a ?crit :Quoted 15 lines · Eric B?nard
Quoted 2 lines
This will increase the boot time by 2 seconds : why is that needed ?
Thanks for commenting. My intent is to give by default a reasonable time for the end-user to react and break the boot-sequence if needed (and avoid the nasty racing against the clock for this precious key-press).
My motivation for this is that I see RIoTboard/MarsBoard mostly as boards that people can use for experimentation/prototyping, and not as a finished end-product, thus my thinking that the slightly increased boot-time won't be a critical issue for such applications. Still everyone can change the delay at will if they want to optimize the boot.
Quoted 7 lines
Quoted 4 lines
Quoted 3 lines
Other hobby boards also have similarly handy boot-timings, like Wandboard (5s), Novena (5s), Udoo (3s).
OK seems reasonable.
I disagree. I think it's an unnecessary, unneeded, unreasonable tripling of the boot time.
I have 2 things to say about this:
1. In theory we can also question why the initial boot delay was selected to be 1s.
2. You imply that the "users" want ultra-fast boot times for riotboard. Being myself one of the riotboard users, I can assure you that such generalization is excessive.
Quoted 2 lines
Rather than leaving people to patch out this unwanted behaviour, I'd suggest promoting
CONFIG_BOOTDELAYinto aKconfigoption, leaving the default as-is.
This is a good idea in general, I don't mind it.
Quoted 2 lines
This also leads to a slightly less unreasonable increase in bootdelay to 2s from
config_distro_defaults.h
Well, this is again subjective - we can always argue that 2s is both too long and too short for selected use cases, so I don't understand the qualification "less unreasonable".
In summary - I won't carry a war for this change, as it doesn't worth it. I just tried to contribute a tiny bit of improvement to the already excellent work of Eric, so we can have a silk-smooth out-of-the-box U-Boot user experience, and I'm doing this as an actual riotboard user. That's all.
Thanks for sharing your viewpoint.
Kind regards,
Nikolay
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
Le Sun, 14 Dec 2014 22:56:21 +0200,
Nikolay Dimitrov <[email protected]> a ?crit :
Quoted 8 lines · Iain Paton
Quoted 2 lines
This also leads to a slightly less unreasonable increase in bootdelay to 2s from
config_distro_defaults.hWell, this is again subjective - we can always argue that 2s is both too long and too short for selected use cases, so I don't understand the qualification "less unreasonable".
Quoted 6 lines
In summary - I won't carry a war for this change, as it doesn't worth it. I just tried to contribute a tiny bit of improvement to the already excellent work of Eric, so we can have a silk-smooth out-of-the-box U-Boot user experience, and I'm doing this as an actual riotboard user. That's all.
well if Iain's patch configure the bootdelay to 2 seconds by default this will be a perfect average between the actual state and your proposal so I suggest you test Iain's patch and if its OK for you send your ack as I have no time to test on Riot or Mars board before next year ;-)
Thanks !
Eric
PATCH 1/1 embestmx6boards: Increase boot delay to 3s
In reply to Nikolay DimitrovQuoted 5 lines · Nikolay Dimitrov
In summary - I won't carry a war for this change, as it doesn't worth it. I just tried to contribute a tiny bit of improvement to the already excellent work of Eric, so we can have a silk-smooth out-of-the-box U-Boot user experience, and I'm doing this as an actual riotboard user. That's all.
I'm a riotboard user too, and have contributed towards it's success in my own ways. The choice of 1s hasn't been a problem for the many people using the Debian images I provided.
Anyway, it's in the interest of Riot/Mars to move towards the config_distro_* way of doing things longer term as it's likely to gain easy support by the general distros that way.
Assuming those patches get accepted, the bootdelay ends up in the common config shared by all boards using config_distro_defaults. Over time I suspect that's likely to include most of the general dev boards.
It's still likely to be worth moving it to Kconfig as both of us will probably want to change it to our own preferences regardless :)