Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2015-01-22 Thread York Sun
On 12/15/2014 01:28 AM, Harninder Rai wrote: Signed-off-by: Harninder Rai harninder@freescale.com --- Applied to u-boot-mpc85xx master branch, awaiting upstream. York ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2015-01-14 Thread York Sun
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

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-21 Thread Harninder Rai
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

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-16 Thread Harninder Rai
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

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-16 Thread York Sun
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

[U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-15 Thread Harninder Rai
Signed-off-by: Harninder Rai harninder@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 ---

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-15 Thread York Sun
On 12/14/2014 11:28 PM, Harninder Rai wrote: Signed-off-by: Harninder Rai harninder@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

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-15 Thread York Sun
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

Re: [U-Boot] [PATCH] powerpc/913x: Add config flag for bootdelay

2014-12-15 Thread Harninder Rai
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