* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
@ 2014-12-15 7:28 Harninder Rai
2014-12-15 16:56 ` York Sun
2015-01-23 0:29 ` York Sun
0 siblings, 2 replies; 9+ messages in thread
From: Harninder Rai @ 2014-12-15 7:28 UTC (permalink / raw)
To: u-boot
Signed-off-by: Harninder Rai <harninder.rai@freescale.com>
---
include/configs/BSC9131RDB.h | 1 +
include/configs/BSC9132QDS.h | 1 +
2 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/include/configs/BSC9131RDB.h b/include/configs/BSC9131RDB.h
index adb8146..6f1022e 100644
--- a/include/configs/BSC9131RDB.h
+++ b/include/configs/BSC9131RDB.h
@@ -430,6 +430,7 @@ extern unsigned long get_sdram_size(void);
#define CONFIG_UBOOTPATH "u-boot.bin" /* U-Boot image on TFTP server */
#define CONFIG_BAUDRATE 115200
+#define CONFIG_BOOTDELAY 10 /* -1 disable auto-boot */
#define CONFIG_EXTRA_ENV_SETTINGS \
"netdev=eth0\0" \
diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index 2722a32..66ca461 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -672,6 +672,7 @@ combinations. this should be removed later
#define CONFIG_UBOOTPATH "u-boot.bin"
#define CONFIG_BAUDRATE 115200
+#define CONFIG_BOOTDELAY 10 /* -1 disable auto-boot */
#ifdef CONFIG_SDCARD
#define CONFIG_DEF_HWCONFIG "hwconfig=usb1:dr_mode=host,phy_type=ulpi\0"
--
1.7.6.GIT
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-15 7:28 [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay Harninder Rai
@ 2014-12-15 16:56 ` York Sun
2014-12-16 6:07 ` Harninder Rai
2015-01-23 0:29 ` York Sun
1 sibling, 1 reply; 9+ messages in thread
From: York Sun @ 2014-12-15 16:56 UTC (permalink / raw)
To: u-boot
On 12/14/2014 11:28 PM, Harninder Rai wrote:
> Signed-off-by: Harninder Rai <harninder.rai@freescale.com>
> ---
> include/configs/BSC9131RDB.h | 1 +
> include/configs/BSC9132QDS.h | 1 +
> 2 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/include/configs/BSC9131RDB.h b/include/configs/BSC9131RDB.h
> index adb8146..6f1022e 100644
> --- a/include/configs/BSC9131RDB.h
> +++ b/include/configs/BSC9131RDB.h
> @@ -430,6 +430,7 @@ extern unsigned long get_sdram_size(void);
> #define CONFIG_UBOOTPATH "u-boot.bin" /* U-Boot image on TFTP server */
>
> #define CONFIG_BAUDRATE 115200
> +#define CONFIG_BOOTDELAY 10 /* -1 disable auto-boot */
>
> #define CONFIG_EXTRA_ENV_SETTINGS \
> "netdev=eth0\0" \
> diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
> index 2722a32..66ca461 100644
> --- a/include/configs/BSC9132QDS.h
> +++ b/include/configs/BSC9132QDS.h
> @@ -672,6 +672,7 @@ combinations. this should be removed later
> #define CONFIG_UBOOTPATH "u-boot.bin"
>
> #define CONFIG_BAUDRATE 115200
> +#define CONFIG_BOOTDELAY 10 /* -1 disable auto-boot */
>
> #ifdef CONFIG_SDCARD
> #define CONFIG_DEF_HWCONFIG "hwconfig=usb1:dr_mode=host,phy_type=ulpi\0"
>
The change is trivial. Do you mind to explain why you need this change?
York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-15 16:56 ` York Sun
@ 2014-12-16 6:07 ` Harninder Rai
2014-12-16 6:15 ` York Sun
0 siblings, 1 reply; 9+ messages in thread
From: Harninder Rai @ 2014-12-16 6:07 UTC (permalink / raw)
To: u-boot
>
> The change is trivial. Do you mind to explain why you need this change?
Thanks for the review. This change is required to give user some time to interrupt the booting process
Also, this will allow the system to boot from flash after power-on (does help in automation testing)
Is this sufficient for a commit message?
Harry++
>
> York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-16 6:07 ` Harninder Rai
@ 2014-12-16 6:15 ` York Sun
2014-12-16 10:55 ` Harninder Rai
0 siblings, 1 reply; 9+ messages in thread
From: York Sun @ 2014-12-16 6:15 UTC (permalink / raw)
To: u-boot
You understand this is the default value of the said variable. You can save your environmental variable when you boot up. You can save it to any value you want, and this default value doesn't matter any more, until you erase it.
York
________________________________
From: Rai Harninder-B01044
Sent: Mon, 15/12/2014 22:07
To: Sun York-R58495 <yorksun@freescale.com>; u-boot at lists.denx.de
Subject: RE: [PATCH] powerpc/913x: Add config flag for bootdelay
>
> The change is trivial. Do you mind to explain why you need this change?
Thanks for the review. This change is required to give user some time to interrupt the booting process
Also, this will allow the system to boot from flash after power-on (does help in automation testing)
Is this sufficient for a commit message?
Harry++
>
> York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-16 6:15 ` York Sun
@ 2014-12-16 10:55 ` Harninder Rai
2014-12-16 16:38 ` York Sun
0 siblings, 1 reply; 9+ messages in thread
From: Harninder Rai @ 2014-12-16 10:55 UTC (permalink / raw)
To: u-boot
I am not sure I followed you completely but if I just use CONFIG_BOOTDELAY then I get the following compilation error
common/autoboot.c: In function 'bootdelay_process':
common/autoboot.c:247:68: error: expected expression before ';' token
Is there something which I am missing?
Thanks and Regards
Harry++
From: Sun York-R58495
Sent: Tuesday, December 16, 2014 11:46 AM
To: Rai Harninder-B01044; u-boot at lists.denx.de
Subject: RE: [PATCH] powerpc/913x: Add config flag for bootdelay
You understand this is the default value of the said variable. You can save your environmental variable when you boot up. You can save it to any value you want, and this default value doesn't matter any more, until you erase it.
York
________________________________
From: Rai Harninder-B01044
Sent: Mon, 15/12/2014 22:07
To: Sun York-R58495 <yorksun at freescale.com<mailto:yorksun@freescale.com>>; u-boot at lists.denx.de<mailto:u-boot@lists.denx.de>
Subject: RE: [PATCH] powerpc/913x: Add config flag for bootdelay
>
> The change is trivial. Do you mind to explain why you need this change?
Thanks for the review. This change is required to give user some time to interrupt the booting process
Also, this will allow the system to boot from flash after power-on (does help in automation testing)
Is this sufficient for a commit message?
Harry++
>
> York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-16 10:55 ` Harninder Rai
@ 2014-12-16 16:38 ` York Sun
2014-12-22 5:22 ` Harninder Rai
0 siblings, 1 reply; 9+ messages in thread
From: York Sun @ 2014-12-16 16:38 UTC (permalink / raw)
To: u-boot
On 12/16/2014 02:55 AM, Rai Harninder-B01044 wrote:
> I am not sure I followed you completely but if I just use CONFIG_BOOTDELAY then
> I get the following compilation error
>
>
>
> common/autoboot.c: In function 'bootdelay_process':
>
> common/autoboot.c:247:68: error: expected expression before ';' token
>
>
>
> Is there something which I am missing?
>
I am guessing you has some extra character for the CONFIG_BOOTDELAY. Anyway,
since you are on this line common/autoboot.c:247, you can see the "bootdelay"
variable one line above. Your purpose of adding delay to boot is easily
implemented by these command under u-boot
setenv bootdelay 10
saveenv
There is no need to change the code for this purpose.
York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-16 16:38 ` York Sun
@ 2014-12-22 5:22 ` Harninder Rai
2015-01-14 21:55 ` York Sun
0 siblings, 1 reply; 9+ messages in thread
From: Harninder Rai @ 2014-12-22 5:22 UTC (permalink / raw)
To: u-boot
> I am guessing you has some extra character for the CONFIG_BOOTDELAY. Anyway,
> since you are on this line common/autoboot.c:247, you can see the "bootdelay"
> variable one line above. Your purpose of adding delay to boot is easily
> implemented by these command under u-boot
>
> setenv bootdelay 10
> saveenv
>
> There is no need to change the code for this purpose.
Thanks for the review. Let me look into this
Harry++
>
> York
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-22 5:22 ` Harninder Rai
@ 2015-01-14 21:55 ` York Sun
0 siblings, 0 replies; 9+ messages in thread
From: York Sun @ 2015-01-14 21:55 UTC (permalink / raw)
To: u-boot
On 12/21/2014 09:22 PM, Rai Harninder-B01044 wrote:
>> I am guessing you has some extra character for the CONFIG_BOOTDELAY. Anyway,
>> since you are on this line common/autoboot.c:247, you can see the "bootdelay"
>> variable one line above. Your purpose of adding delay to boot is easily
>> implemented by these command under u-boot
>>
>> setenv bootdelay 10
>> saveenv
>>
>> There is no need to change the code for this purpose.
> Thanks for the review. Let me look into this
>
I took another look at this patch. It is harmless and adding the default value
is correct. I will accept this patch and add commit message.
York
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay
2014-12-15 7:28 [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay Harninder Rai
2014-12-15 16:56 ` York Sun
@ 2015-01-23 0:29 ` York Sun
1 sibling, 0 replies; 9+ messages in thread
From: York Sun @ 2015-01-23 0:29 UTC (permalink / raw)
To: u-boot
On 12/15/2014 01:28 AM, Harninder Rai wrote:
> Signed-off-by: Harninder Rai <harninder.rai@freescale.com>
> ---
Applied to u-boot-mpc85xx master branch, awaiting upstream.
York
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-01-23 0:29 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-15 7:28 [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay Harninder Rai
2014-12-15 16:56 ` York Sun
2014-12-16 6:07 ` Harninder Rai
2014-12-16 6:15 ` York Sun
2014-12-16 10:55 ` Harninder Rai
2014-12-16 16:38 ` York Sun
2014-12-22 5:22 ` Harninder Rai
2015-01-14 21:55 ` York Sun
2015-01-23 0:29 ` York Sun
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox