powerpc/913x: Add config flag for bootdelay

---
 include/configs/BSC9131RDB.h |    1 +
 include/configs/BSC9132QDS.h |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)

index adb8146..6f1022e 100644
@@ -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"						\

index 2722a32..66ca461 100644
@@ -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

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++

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 <[email protected]>; u-boot at lists.denx.de
Subject: RE: [PATCH] powerpc/913x: Add config flag for bootdelay

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++

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:[email protected]>>; u-boot at lists.denx.de<mailto:[email protected]>
Subject: RE: [PATCH] powerpc/913x: Add config flag for bootdelay

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++

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

Thanks for the review. Let me look into this

Harry++

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

Applied to u-boot-mpc85xx master branch, awaiting upstream.

York