* [U-Boot] [PATCH 2/8] mx53loco: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 3/8] mx6qsabrelite: " Fabio Estevam
` (6 subsequent siblings)
7 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx53loco.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx53loco.h b/include/configs/mx53loco.h
index a62ea78..37f9d69 100644
--- a/include/configs/mx53loco.h
+++ b/include/configs/mx53loco.h
@@ -108,7 +108,7 @@
#undef CONFIG_CMD_IMLS
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_ETHPRIME "FEC0"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 3/8] mx6qsabrelite: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 2/8] mx53loco: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-19 3:28 ` Liu Hui-R64343
2012-11-16 15:09 ` [U-Boot] [PATCH 4/8] mx6qsabre_common: " Fabio Estevam
` (5 subsequent siblings)
7 siblings, 1 reply; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx6qsabrelite.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx6qsabrelite.h b/include/configs/mx6qsabrelite.h
index 318f857..759275a 100644
--- a/include/configs/mx6qsabrelite.h
+++ b/include/configs/mx6qsabrelite.h
@@ -144,7 +144,7 @@
#undef CONFIG_CMD_IMLS
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_PREBOOT ""
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 3/8] mx6qsabrelite: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 ` [U-Boot] [PATCH 3/8] mx6qsabrelite: " Fabio Estevam
@ 2012-11-19 3:28 ` Liu Hui-R64343
0 siblings, 0 replies; 20+ messages in thread
From: Liu Hui-R64343 @ 2012-11-19 3:28 UTC (permalink / raw)
To: u-boot
>-----Original Message-----
>From: Fabio Estevam [mailto:festevam at gmail.com]
>Sent: Friday, November 16, 2012 11:09 PM
>To: sbabic at denx.de
>Cc: Liu Hui-R64343; u-boot at lists.denx.de; Estevam Fabio-R49496
>Subject: [PATCH 3/8] mx6qsabrelite: Configure CONFIG_BOOTDELAY to one
>second
>
>From: Fabio Estevam <fabio.estevam@freescale.com>
>
>One second is enough time for users to react in case they want to stop the
>booting process.
>
>Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Acked-by: Jason Liu <r64343@freescale.com>
>---
> include/configs/mx6qsabrelite.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/configs/mx6qsabrelite.h
>b/include/configs/mx6qsabrelite.h index 318f857..759275a 100644
>--- a/include/configs/mx6qsabrelite.h
>+++ b/include/configs/mx6qsabrelite.h
>@@ -144,7 +144,7 @@
>
> #undef CONFIG_CMD_IMLS
>
>-#define CONFIG_BOOTDELAY 3
>+#define CONFIG_BOOTDELAY 1
>
> #define CONFIG_PREBOOT ""
>
>--
>1.7.9.5
>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 4/8] mx6qsabre_common: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 2/8] mx53loco: " Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 3/8] mx6qsabrelite: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-19 3:28 ` Liu Hui-R64343
2012-11-16 15:09 ` [U-Boot] [PATCH 5/8] mx51evk: " Fabio Estevam
` (4 subsequent siblings)
7 siblings, 1 reply; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx6qsabre_common.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx6qsabre_common.h b/include/configs/mx6qsabre_common.h
index 50f3b01..b8621b8 100644
--- a/include/configs/mx6qsabre_common.h
+++ b/include/configs/mx6qsabre_common.h
@@ -76,7 +76,7 @@
#define CONFIG_CMD_BOOTZ
#undef CONFIG_CMD_IMLS
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_LOADADDR 0x10800000
#define CONFIG_SYS_TEXT_BASE 0x17800000
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 4/8] mx6qsabre_common: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 ` [U-Boot] [PATCH 4/8] mx6qsabre_common: " Fabio Estevam
@ 2012-11-19 3:28 ` Liu Hui-R64343
0 siblings, 0 replies; 20+ messages in thread
From: Liu Hui-R64343 @ 2012-11-19 3:28 UTC (permalink / raw)
To: u-boot
>-----Original Message-----
>From: Fabio Estevam [mailto:festevam at gmail.com]
>Sent: Friday, November 16, 2012 11:09 PM
>To: sbabic at denx.de
>Cc: Liu Hui-R64343; u-boot at lists.denx.de; Estevam Fabio-R49496
>Subject: [PATCH 4/8] mx6qsabre_common: Configure CONFIG_BOOTDELAY to
>one second
>
>From: Fabio Estevam <fabio.estevam@freescale.com>
>
>One second is enough time for users to react in case they want to stop the
>booting process.
>
>Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Acked-by: Jason Liu <r64343@freescale.com>
>---
> include/configs/mx6qsabre_common.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/configs/mx6qsabre_common.h
>b/include/configs/mx6qsabre_common.h
>index 50f3b01..b8621b8 100644
>--- a/include/configs/mx6qsabre_common.h
>+++ b/include/configs/mx6qsabre_common.h
>@@ -76,7 +76,7 @@
> #define CONFIG_CMD_BOOTZ
> #undef CONFIG_CMD_IMLS
>
>-#define CONFIG_BOOTDELAY 3
>+#define CONFIG_BOOTDELAY 1
>
> #define CONFIG_LOADADDR 0x10800000
> #define CONFIG_SYS_TEXT_BASE 0x17800000
>--
>1.7.9.5
>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 5/8] mx51evk: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
` (2 preceding siblings ...)
2012-11-16 15:09 ` [U-Boot] [PATCH 4/8] mx6qsabre_common: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 6/8] mx25pdk: " Fabio Estevam
` (3 subsequent siblings)
7 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx51evk.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx51evk.h b/include/configs/mx51evk.h
index 89feaed..e8c6618 100644
--- a/include/configs/mx51evk.h
+++ b/include/configs/mx51evk.h
@@ -149,7 +149,7 @@
#define CONFIG_CMD_DATE
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_ETHPRIME "FEC0"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 6/8] mx25pdk: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
` (3 preceding siblings ...)
2012-11-16 15:09 ` [U-Boot] [PATCH 5/8] mx51evk: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 7/8] mx31pdk: " Fabio Estevam
` (2 subsequent siblings)
7 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx25pdk.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx25pdk.h b/include/configs/mx25pdk.h
index a4bd8b0..b5338a0 100644
--- a/include/configs/mx25pdk.h
+++ b/include/configs/mx25pdk.h
@@ -130,7 +130,7 @@
#define CONFIG_CMD_MII
#define CONFIG_CMD_NET
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_LOADADDR 0x81000000 /* loadaddr env var */
#define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 7/8] mx31pdk: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
` (4 preceding siblings ...)
2012-11-16 15:09 ` [U-Boot] [PATCH 6/8] mx25pdk: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-16 15:09 ` [U-Boot] [PATCH 8/8] mx35pdk: " Fabio Estevam
2012-11-19 7:51 ` [U-Boot] [PATCH 1/8] mx28evk: " Stefano Babic
7 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx31pdk.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx31pdk.h b/include/configs/mx31pdk.h
index 223b5b0..6572676 100644
--- a/include/configs/mx31pdk.h
+++ b/include/configs/mx31pdk.h
@@ -106,7 +106,7 @@
#define CONFIG_BOARD_LATE_INIT
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_EXTRA_ENV_SETTINGS \
"bootargs_base=setenv bootargs console=ttymxc0,115200\0" \
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 8/8] mx35pdk: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
` (5 preceding siblings ...)
2012-11-16 15:09 ` [U-Boot] [PATCH 7/8] mx31pdk: " Fabio Estevam
@ 2012-11-16 15:09 ` Fabio Estevam
2012-11-19 7:51 ` [U-Boot] [PATCH 1/8] mx28evk: " Stefano Babic
7 siblings, 0 replies; 20+ messages in thread
From: Fabio Estevam @ 2012-11-16 15:09 UTC (permalink / raw)
To: u-boot
From: Fabio Estevam <fabio.estevam@freescale.com>
One second is enough time for users to react in case they want to stop the
booting process.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
include/configs/mx35pdk.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/configs/mx35pdk.h b/include/configs/mx35pdk.h
index d89db7a..44fb1cd 100644
--- a/include/configs/mx35pdk.h
+++ b/include/configs/mx35pdk.h
@@ -118,7 +118,7 @@
#define CONFIG_CMD_EXT2
#define CONFIG_CMD_FAT
-#define CONFIG_BOOTDELAY 3
+#define CONFIG_BOOTDELAY 1
#define CONFIG_LOADADDR 0x80800000 /* loadaddr env var */
--
1.7.9.5
^ permalink raw reply related [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-16 15:09 [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second Fabio Estevam
` (6 preceding siblings ...)
2012-11-16 15:09 ` [U-Boot] [PATCH 8/8] mx35pdk: " Fabio Estevam
@ 2012-11-19 7:51 ` Stefano Babic
2012-11-19 23:52 ` Marek Vasut
7 siblings, 1 reply; 20+ messages in thread
From: Stefano Babic @ 2012-11-19 7:51 UTC (permalink / raw)
To: u-boot
On 16/11/2012 16:09, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
>
> One second is enough time for users to react in case they want to stop the
> booting process.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> include/configs/mx28evk.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> index 2916c71..8b89b25 100644
> --- a/include/configs/mx28evk.h
> +++ b/include/configs/mx28evk.h
> @@ -238,7 +238,7 @@
> */
> #define CONFIG_CMDLINE_TAG
> #define CONFIG_SETUP_MEMORY_TAGS
> -#define CONFIG_BOOTDELAY 3
> +#define CONFIG_BOOTDELAY 1
> #define CONFIG_BOOTFILE "uImage"
> #define CONFIG_LOADADDR 0x42000000
> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
>
Applied (whole series) to u-boot-imx, thanks.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 20+ messages in thread* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-19 7:51 ` [U-Boot] [PATCH 1/8] mx28evk: " Stefano Babic
@ 2012-11-19 23:52 ` Marek Vasut
2012-11-20 15:44 ` Stefano Babic
2012-11-20 23:00 ` Scott Wood
0 siblings, 2 replies; 20+ messages in thread
From: Marek Vasut @ 2012-11-19 23:52 UTC (permalink / raw)
To: u-boot
Dear Stefano Babic,
> On 16/11/2012 16:09, Fabio Estevam wrote:
> > From: Fabio Estevam <fabio.estevam@freescale.com>
> >
> > One second is enough time for users to react in case they want to stop
> > the booting process.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> >
> > include/configs/mx28evk.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > index 2916c71..8b89b25 100644
> > --- a/include/configs/mx28evk.h
> > +++ b/include/configs/mx28evk.h
> > @@ -238,7 +238,7 @@
> >
> > */
> >
> > #define CONFIG_CMDLINE_TAG
> > #define CONFIG_SETUP_MEMORY_TAGS
> >
> > -#define CONFIG_BOOTDELAY 3
> > +#define CONFIG_BOOTDELAY 1
> >
> > #define CONFIG_BOOTFILE "uImage"
> > #define CONFIG_LOADADDR 0x42000000
> > #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
>
> Applied (whole series) to u-boot-imx, thanks.
What about making some generic ... config-mx28.h AND config-fsl.h ... and
including those in board-specific configs?
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-19 23:52 ` Marek Vasut
@ 2012-11-20 15:44 ` Stefano Babic
2012-11-20 17:50 ` Otavio Salvador
2012-11-20 23:00 ` Scott Wood
1 sibling, 1 reply; 20+ messages in thread
From: Stefano Babic @ 2012-11-20 15:44 UTC (permalink / raw)
To: u-boot
On 20/11/2012 00:52, Marek Vasut wrote:
> Dear Stefano Babic,
>
>> On 16/11/2012 16:09, Fabio Estevam wrote:
>>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>>
>>> One second is enough time for users to react in case they want to stop
>>> the booting process.
>>>
>>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>>> ---
>>>
>>> include/configs/mx28evk.h | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
>>> index 2916c71..8b89b25 100644
>>> --- a/include/configs/mx28evk.h
>>> +++ b/include/configs/mx28evk.h
>>> @@ -238,7 +238,7 @@
>>>
>>> */
>>>
>>> #define CONFIG_CMDLINE_TAG
>>> #define CONFIG_SETUP_MEMORY_TAGS
>>>
>>> -#define CONFIG_BOOTDELAY 3
>>> +#define CONFIG_BOOTDELAY 1
>>>
>>> #define CONFIG_BOOTFILE "uImage"
>>> #define CONFIG_LOADADDR 0x42000000
>>> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
>>
>> Applied (whole series) to u-boot-imx, thanks.
>
> What about making some generic ... config-mx28.h AND config-fsl.h ... and
> including those in board-specific configs?
Any clean-up is welcome. Then maybe (and this is not related to i.MX
only) we could have a cpu-config, a vendor config and on the top a board
config.
#include <cpu-config.h>
#include <vendor-config.h>
#include <board-config.h>
The main problem is that an exception can breaks our castle, and we are
constrained to add #undef in board configuration file.
Best regards,
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-20 15:44 ` Stefano Babic
@ 2012-11-20 17:50 ` Otavio Salvador
2012-11-20 18:09 ` Marek Vasut
0 siblings, 1 reply; 20+ messages in thread
From: Otavio Salvador @ 2012-11-20 17:50 UTC (permalink / raw)
To: u-boot
On Tue, Nov 20, 2012 at 1:44 PM, Stefano Babic <sbabic@denx.de> wrote:
> On 20/11/2012 00:52, Marek Vasut wrote:
>> Dear Stefano Babic,
>>
>>> On 16/11/2012 16:09, Fabio Estevam wrote:
>>>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>>>
>>>> One second is enough time for users to react in case they want to stop
>>>> the booting process.
>>>>
>>>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>>>> ---
>>>>
>>>> include/configs/mx28evk.h | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
>>>> index 2916c71..8b89b25 100644
>>>> --- a/include/configs/mx28evk.h
>>>> +++ b/include/configs/mx28evk.h
>>>> @@ -238,7 +238,7 @@
>>>>
>>>> */
>>>>
>>>> #define CONFIG_CMDLINE_TAG
>>>> #define CONFIG_SETUP_MEMORY_TAGS
>>>>
>>>> -#define CONFIG_BOOTDELAY 3
>>>> +#define CONFIG_BOOTDELAY 1
>>>>
>>>> #define CONFIG_BOOTFILE "uImage"
>>>> #define CONFIG_LOADADDR 0x42000000
>>>> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
>>>
>>> Applied (whole series) to u-boot-imx, thanks.
>>
>> What about making some generic ... config-mx28.h AND config-fsl.h ... and
>> including those in board-specific configs?
>
> Any clean-up is welcome. Then maybe (and this is not related to i.MX
> only) we could have a cpu-config, a vendor config and on the top a board
> config.
>
> #include <cpu-config.h>
> #include <vendor-config.h>
> #include <board-config.h>
>
> The main problem is that an exception can breaks our castle, and we are
> constrained to add #undef in board configuration file.
I support this idea; currently I think we ought to have as much as
common code by vendor so it is easy to document things and avoid
duplication.
Currently we cannot expect a board to have same (or near the same) set
of features. Some provide a good default environment, others doesn't.
--
Otavio Salvador O.S. Systems
E-mail: otavio at ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-20 17:50 ` Otavio Salvador
@ 2012-11-20 18:09 ` Marek Vasut
2012-11-20 18:19 ` Otavio Salvador
0 siblings, 1 reply; 20+ messages in thread
From: Marek Vasut @ 2012-11-20 18:09 UTC (permalink / raw)
To: u-boot
Dear Otavio Salvador,
> On Tue, Nov 20, 2012 at 1:44 PM, Stefano Babic <sbabic@denx.de> wrote:
> > On 20/11/2012 00:52, Marek Vasut wrote:
> >> Dear Stefano Babic,
> >>
> >>> On 16/11/2012 16:09, Fabio Estevam wrote:
> >>>> From: Fabio Estevam <fabio.estevam@freescale.com>
> >>>>
> >>>> One second is enough time for users to react in case they want to stop
> >>>> the booting process.
> >>>>
> >>>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> >>>> ---
> >>>>
> >>>> include/configs/mx28evk.h | 2 +-
> >>>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>>
> >>>> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> >>>> index 2916c71..8b89b25 100644
> >>>> --- a/include/configs/mx28evk.h
> >>>> +++ b/include/configs/mx28evk.h
> >>>> @@ -238,7 +238,7 @@
> >>>>
> >>>> */
> >>>>
> >>>> #define CONFIG_CMDLINE_TAG
> >>>> #define CONFIG_SETUP_MEMORY_TAGS
> >>>>
> >>>> -#define CONFIG_BOOTDELAY 3
> >>>> +#define CONFIG_BOOTDELAY 1
> >>>>
> >>>> #define CONFIG_BOOTFILE "uImage"
> >>>> #define CONFIG_LOADADDR 0x42000000
> >>>> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> >>>
> >>> Applied (whole series) to u-boot-imx, thanks.
> >>
> >> What about making some generic ... config-mx28.h AND config-fsl.h ...
> >> and including those in board-specific configs?
> >
> > Any clean-up is welcome. Then maybe (and this is not related to i.MX
> > only) we could have a cpu-config, a vendor config and on the top a board
> > config.
> >
> > #include <cpu-config.h>
> > #include <vendor-config.h>
> > #include <board-config.h>
> >
> > The main problem is that an exception can breaks our castle, and we are
> > constrained to add #undef in board configuration file.
>
> I support this idea; currently I think we ought to have as much as
> common code by vendor so it is easy to document things and avoid
> duplication.
WFM
> Currently we cannot expect a board to have same (or near the same) set
> of features. Some provide a good default environment, others doesn't.
Let's leave ENV out of this discussion. Moreover, this needs to be done in
proper incremental steps.
Furthermore, new thread should be started.
Just an idea, to avoid undef in board-files, we can have #ifdef CONFIG...
#define CONFIG... #endif constructions in the CPU-specific files. This will
allow simple overriding (of course, these files would have to be included at the
bottom of the board config file then).
Otavio, any plans to finish u-boot/mx23 soon?
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-20 18:09 ` Marek Vasut
@ 2012-11-20 18:19 ` Otavio Salvador
2012-11-20 18:43 ` Marek Vasut
0 siblings, 1 reply; 20+ messages in thread
From: Otavio Salvador @ 2012-11-20 18:19 UTC (permalink / raw)
To: u-boot
On Tue, Nov 20, 2012 at 4:09 PM, Marek Vasut <marex@denx.de> wrote:
> Dear Otavio Salvador,
>
>> On Tue, Nov 20, 2012 at 1:44 PM, Stefano Babic <sbabic@denx.de> wrote:
>> > On 20/11/2012 00:52, Marek Vasut wrote:
>> >> Dear Stefano Babic,
>> >>
>> >>> On 16/11/2012 16:09, Fabio Estevam wrote:
>> >>>> From: Fabio Estevam <fabio.estevam@freescale.com>
>> >>>>
>> >>>> One second is enough time for users to react in case they want to stop
>> >>>> the booting process.
>> >>>>
>> >>>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>> >>>> ---
>> >>>>
>> >>>> include/configs/mx28evk.h | 2 +-
>> >>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >>>>
>> >>>> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
>> >>>> index 2916c71..8b89b25 100644
>> >>>> --- a/include/configs/mx28evk.h
>> >>>> +++ b/include/configs/mx28evk.h
>> >>>> @@ -238,7 +238,7 @@
>> >>>>
>> >>>> */
>> >>>>
>> >>>> #define CONFIG_CMDLINE_TAG
>> >>>> #define CONFIG_SETUP_MEMORY_TAGS
>> >>>>
>> >>>> -#define CONFIG_BOOTDELAY 3
>> >>>> +#define CONFIG_BOOTDELAY 1
>> >>>>
>> >>>> #define CONFIG_BOOTFILE "uImage"
>> >>>> #define CONFIG_LOADADDR 0x42000000
>> >>>> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
>> >>>
>> >>> Applied (whole series) to u-boot-imx, thanks.
>> >>
>> >> What about making some generic ... config-mx28.h AND config-fsl.h ...
>> >> and including those in board-specific configs?
>> >
>> > Any clean-up is welcome. Then maybe (and this is not related to i.MX
>> > only) we could have a cpu-config, a vendor config and on the top a board
>> > config.
>> >
>> > #include <cpu-config.h>
>> > #include <vendor-config.h>
>> > #include <board-config.h>
>> >
>> > The main problem is that an exception can breaks our castle, and we are
>> > constrained to add #undef in board configuration file.
>>
>> I support this idea; currently I think we ought to have as much as
>> common code by vendor so it is easy to document things and avoid
>> duplication.
>
> WFM
>
>> Currently we cannot expect a board to have same (or near the same) set
>> of features. Some provide a good default environment, others doesn't.
>
> Let's leave ENV out of this discussion. Moreover, this needs to be done in
> proper incremental steps.
So you prefer it to manage features only, at least for now?
> Furthermore, new thread should be started.
Agreed.
> Just an idea, to avoid undef in board-files, we can have #ifdef CONFIG...
> #define CONFIG... #endif constructions in the CPU-specific files. This will
> allow simple overriding (of course, these files would have to be included at the
> bottom of the board config file then).
So basically a global feature-aware setting file.
> Otavio, any plans to finish u-boot/mx23 soon?
No reply yet from the guy who got it further. I am almost getting onto
it without his patches ...
--
Otavio Salvador O.S. Systems
E-mail: otavio at ossystems.com.br http://www.ossystems.com.br
Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-20 18:19 ` Otavio Salvador
@ 2012-11-20 18:43 ` Marek Vasut
0 siblings, 0 replies; 20+ messages in thread
From: Marek Vasut @ 2012-11-20 18:43 UTC (permalink / raw)
To: u-boot
Dear Otavio Salvador,
> On Tue, Nov 20, 2012 at 4:09 PM, Marek Vasut <marex@denx.de> wrote:
> > Dear Otavio Salvador,
> >
> >> On Tue, Nov 20, 2012 at 1:44 PM, Stefano Babic <sbabic@denx.de> wrote:
> >> > On 20/11/2012 00:52, Marek Vasut wrote:
> >> >> Dear Stefano Babic,
> >> >>
> >> >>> On 16/11/2012 16:09, Fabio Estevam wrote:
> >> >>>> From: Fabio Estevam <fabio.estevam@freescale.com>
> >> >>>>
> >> >>>> One second is enough time for users to react in case they want to
> >> >>>> stop the booting process.
> >> >>>>
> >> >>>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> >> >>>> ---
> >> >>>>
> >> >>>> include/configs/mx28evk.h | 2 +-
> >> >>>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >> >>>>
> >> >>>> diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> >> >>>> index 2916c71..8b89b25 100644
> >> >>>> --- a/include/configs/mx28evk.h
> >> >>>> +++ b/include/configs/mx28evk.h
> >> >>>> @@ -238,7 +238,7 @@
> >> >>>>
> >> >>>> */
> >> >>>>
> >> >>>> #define CONFIG_CMDLINE_TAG
> >> >>>> #define CONFIG_SETUP_MEMORY_TAGS
> >> >>>>
> >> >>>> -#define CONFIG_BOOTDELAY 3
> >> >>>> +#define CONFIG_BOOTDELAY 1
> >> >>>>
> >> >>>> #define CONFIG_BOOTFILE "uImage"
> >> >>>> #define CONFIG_LOADADDR 0x42000000
> >> >>>> #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> >> >>>
> >> >>> Applied (whole series) to u-boot-imx, thanks.
> >> >>
> >> >> What about making some generic ... config-mx28.h AND config-fsl.h ...
> >> >> and including those in board-specific configs?
> >> >
> >> > Any clean-up is welcome. Then maybe (and this is not related to i.MX
> >> > only) we could have a cpu-config, a vendor config and on the top a
> >> > board config.
> >> >
> >> > #include <cpu-config.h>
> >> > #include <vendor-config.h>
> >> > #include <board-config.h>
> >> >
> >> > The main problem is that an exception can breaks our castle, and we
> >> > are constrained to add #undef in board configuration file.
> >>
> >> I support this idea; currently I think we ought to have as much as
> >> common code by vendor so it is easy to document things and avoid
> >> duplication.
> >
> > WFM
> >
> >> Currently we cannot expect a board to have same (or near the same) set
> >> of features. Some provide a good default environment, others doesn't.
> >
> > Let's leave ENV out of this discussion. Moreover, this needs to be done
> > in proper incremental steps.
>
> So you prefer it to manage features only, at least for now?
For now, yes. And let's not do it all in one massive swipe, but rather slowly.
First pull out CPU-specific goo, then sep-out manufacturer stuff etc.
> > Furthermore, new thread should be started.
>
> Agreed.
>
> > Just an idea, to avoid undef in board-files, we can have #ifdef CONFIG...
> > #define CONFIG... #endif constructions in the CPU-specific files. This
> > will allow simple overriding (of course, these files would have to be
> > included at the bottom of the board config file then).
>
> So basically a global feature-aware setting file.
More or less, yes.
> > Otavio, any plans to finish u-boot/mx23 soon?
>
> No reply yet from the guy who got it further. I am almost getting onto
> it without his patches ...
Yes, I suspect we might have lost him :(
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-19 23:52 ` Marek Vasut
2012-11-20 15:44 ` Stefano Babic
@ 2012-11-20 23:00 ` Scott Wood
2012-11-21 0:15 ` Marek Vasut
1 sibling, 1 reply; 20+ messages in thread
From: Scott Wood @ 2012-11-20 23:00 UTC (permalink / raw)
To: u-boot
On 11/19/2012 05:52:07 PM, Marek Vasut wrote:
> Dear Stefano Babic,
>
> > On 16/11/2012 16:09, Fabio Estevam wrote:
> > > From: Fabio Estevam <fabio.estevam@freescale.com>
> > >
> > > One second is enough time for users to react in case they want to
> stop
> > > the booting process.
> > >
> > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > > ---
> > >
> > > include/configs/mx28evk.h | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > > index 2916c71..8b89b25 100644
> > > --- a/include/configs/mx28evk.h
> > > +++ b/include/configs/mx28evk.h
> > > @@ -238,7 +238,7 @@
> > >
> > > */
> > >
> > > #define CONFIG_CMDLINE_TAG
> > > #define CONFIG_SETUP_MEMORY_TAGS
> > >
> > > -#define CONFIG_BOOTDELAY 3
> > > +#define CONFIG_BOOTDELAY 1
> > >
> > > #define CONFIG_BOOTFILE "uImage"
> > > #define CONFIG_LOADADDR 0x42000000
> > > #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> >
> > Applied (whole series) to u-boot-imx, thanks.
>
> What about making some generic ... config-mx28.h AND config-fsl.h ...
> and
> including those in board-specific configs?
What configuration subset would be applicable to every "fsl" target,
including i.MX, mpc83xx, mpc85xx, mpc5xxx, m68k, etc?
-Scott
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-20 23:00 ` Scott Wood
@ 2012-11-21 0:15 ` Marek Vasut
2012-11-21 0:20 ` Scott Wood
0 siblings, 1 reply; 20+ messages in thread
From: Marek Vasut @ 2012-11-21 0:15 UTC (permalink / raw)
To: u-boot
Dear Scott Wood,
> On 11/19/2012 05:52:07 PM, Marek Vasut wrote:
> > Dear Stefano Babic,
> >
> > > On 16/11/2012 16:09, Fabio Estevam wrote:
> > > > From: Fabio Estevam <fabio.estevam@freescale.com>
> > > >
> > > > One second is enough time for users to react in case they want to
> >
> > stop
> >
> > > > the booting process.
> > > >
> > > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > > > ---
> > > >
> > > > include/configs/mx28evk.h | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/include/configs/mx28evk.h b/include/configs/mx28evk.h
> > > > index 2916c71..8b89b25 100644
> > > > --- a/include/configs/mx28evk.h
> > > > +++ b/include/configs/mx28evk.h
> > > > @@ -238,7 +238,7 @@
> > > >
> > > > */
> > > >
> > > > #define CONFIG_CMDLINE_TAG
> > > > #define CONFIG_SETUP_MEMORY_TAGS
> > > >
> > > > -#define CONFIG_BOOTDELAY 3
> > > > +#define CONFIG_BOOTDELAY 1
> > > >
> > > > #define CONFIG_BOOTFILE "uImage"
> > > > #define CONFIG_LOADADDR 0x42000000
> > > > #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> > >
> > > Applied (whole series) to u-boot-imx, thanks.
> >
> > What about making some generic ... config-mx28.h AND config-fsl.h ...
> > and
> > including those in board-specific configs?
>
> What configuration subset would be applicable to every "fsl" target,
> including i.MX, mpc83xx, mpc85xx, mpc5xxx, m68k, etc?
That's up to you to think about ... anyway, move it to a separate thread please.
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second
2012-11-21 0:15 ` Marek Vasut
@ 2012-11-21 0:20 ` Scott Wood
0 siblings, 0 replies; 20+ messages in thread
From: Scott Wood @ 2012-11-21 0:20 UTC (permalink / raw)
To: u-boot
On 11/20/2012 06:15:25 PM, Marek Vasut wrote:
> Dear Scott Wood,
>
> > On 11/19/2012 05:52:07 PM, Marek Vasut wrote:
> > > Dear Stefano Babic,
> > >
> > > > On 16/11/2012 16:09, Fabio Estevam wrote:
> > > > > From: Fabio Estevam <fabio.estevam@freescale.com>
> > > > >
> > > > > One second is enough time for users to react in case they
> want to
> > >
> > > stop
> > >
> > > > > the booting process.
> > > > >
> > > > > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > > > > ---
> > > > >
> > > > > include/configs/mx28evk.h | 2 +-
> > > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/include/configs/mx28evk.h
> b/include/configs/mx28evk.h
> > > > > index 2916c71..8b89b25 100644
> > > > > --- a/include/configs/mx28evk.h
> > > > > +++ b/include/configs/mx28evk.h
> > > > > @@ -238,7 +238,7 @@
> > > > >
> > > > > */
> > > > >
> > > > > #define CONFIG_CMDLINE_TAG
> > > > > #define CONFIG_SETUP_MEMORY_TAGS
> > > > >
> > > > > -#define CONFIG_BOOTDELAY 3
> > > > > +#define CONFIG_BOOTDELAY 1
> > > > >
> > > > > #define CONFIG_BOOTFILE "uImage"
> > > > > #define CONFIG_LOADADDR 0x42000000
> > > > > #define CONFIG_SYS_LOAD_ADDR CONFIG_LOADADDR
> > > >
> > > > Applied (whole series) to u-boot-imx, thanks.
> > >
> > > What about making some generic ... config-mx28.h AND config-fsl.h
> ...
> > > and
> > > including those in board-specific configs?
> >
> > What configuration subset would be applicable to every "fsl" target,
> > including i.MX, mpc83xx, mpc85xx, mpc5xxx, m68k, etc?
>
> That's up to you to think about ... anyway, move it to a separate
> thread please.
I think the answer to that question is "nothing" -- and you can change
the thread title at any time you wish; I'm not going to go create some
unrelated thread to respond to a point that you raised in this thread.
-Scott
^ permalink raw reply [flat|nested] 20+ messages in thread