[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 2/8] mx53loco: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 3/8] mx6qsabrelite: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 4/8] mx6qsabre_common: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 5/8] mx51evk: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 6/8] mx25pdk: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 7/8] mx31pdk: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 8/8] mx35pdk: Configure CONFIG_BOOTDELAY to one second

[Fabio Estevam]

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

[U-Boot] [PATCH 3/8] mx6qsabrelite: Configure CONFIG_BOOTDELAY to one second

[Liu Hui-R64343]

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

[U-Boot] [PATCH 4/8] mx6qsabre_common: Configure CONFIG_BOOTDELAY to one second

[Liu Hui-R64343]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[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.

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Marek Vasut]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Stefano Babic]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[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.

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Marek Vasut]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[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?

> 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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Marek Vasut]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[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?

-Scott

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Marek Vasut]

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

[U-Boot] [PATCH 1/8] mx28evk: Configure CONFIG_BOOTDELAY to one second

[Scott Wood]

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