Patchwork [U-Boot] eb_cpux9k2: add ram target configuration

login
register
mail settings
Submitter Jens Scharsig
Date April 29, 2012, 6:01 p.m.
Message ID <1335722507-14664-1-git-send-email-js_at_ng@scharsoft.de>
Download mbox | patch
Permalink /patch/155759/
State Changes Requested
Delegated to: Andreas Bießmann
Headers show

Comments

Jens Scharsig - April 29, 2012, 6:01 p.m.
* add ram target for EB+CPUx9k2 board (eb_cpux9k2_ram_config)

Signed-off-by: Jens Scharsig <js_at_ng@scharsoft.de>
---
 boards.cfg                   |    3 ++-
 include/configs/eb_cpux9k2.h |    5 +++++
 2 files changed, 7 insertions(+), 1 deletions(-)
esw@bus-elektronik.de - Sept. 3, 2012, 11:02 a.m.
Dear Andreas Bießmann,

Please could you also apply this open patch.
I've just tested this against the current atmel and master tree.

Thanks

best regards

Jens Scharsig

Am 2012-04-29 20:01, schrieb Jens Scharsig:
> * add ram target for EB+CPUx9k2 board (eb_cpux9k2_ram_config)
> 
> Signed-off-by: Jens Scharsig <js_at_ng@scharsoft.de>
> ---
>  boards.cfg                   |    3 ++-
>  include/configs/eb_cpux9k2.h |    5 +++++
>  2 files changed, 7 insertions(+), 1 deletions(-)
> 
> diff --git a/boards.cfg b/boards.cfg
> index 3cf75c3..efca4fc 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -54,7 +54,8 @@ integratorcp_cm920t          arm         arm920t     integrator          armltd
>  a320evb                      arm         arm920t     -                   faraday        a320
>  at91rm9200ek                 arm         arm920t     at91rm9200ek        atmel          at91        at91rm9200ek
>  at91rm9200ek_ram             arm         arm920t     at91rm9200ek        atmel          at91        at91rm9200ek:RAMBOOT
> -eb_cpux9k2                   arm         arm920t     -                   BuS            at91
> +eb_cpux9k2                   arm         arm920t     eb_cpux9k2          BuS            at91        eb_cpux9k2
> +eb_cpux9k2_ram               arm         arm920t     eb_cpux9k2          BuS            at91        eb_cpux9k2:RAMBOOT
>  cpuat91                      arm         arm920t     cpuat91             eukrea         at91        cpuat91
>  cpuat91_ram                  arm         arm920t     cpuat91             eukrea         at91        cpuat91:RAMBOOT
>  mx1ads                       arm         arm920t     -                   -              imx
> diff --git a/include/configs/eb_cpux9k2.h b/include/configs/eb_cpux9k2.h
> index eb05e2a..d7b6b57 100644
> --- a/include/configs/eb_cpux9k2.h
> +++ b/include/configs/eb_cpux9k2.h
> @@ -44,7 +44,12 @@
>  #define MACH_TYPE_EB_CPUX9K2		1977
>  #define CONFIG_MACH_TYPE		MACH_TYPE_EB_CPUX9K2
>  /*--------------------------------------------------------------------------*/
> +#ifndef CONFIG_RAMBOOT
>  #define CONFIG_SYS_TEXT_BASE 		0x00000000
> +#else
> +#define CONFIG_SKIP_LOWLEVEL_INIT
> +#define CONFIG_SYS_TEXT_BASE 		0x21f00000
> +#endif
>  #define CONFIG_SYS_LOAD_ADDR		0x21000000  /* default load address */
>  
>  #define CONFIG_SYS_BOOT_SIZE		0x00 /* 0 KBytes */
>
Andreas Bießmann - Sept. 3, 2012, 8:09 p.m.
Dear Jens Scharsig,

On 03.09.12 13:02, Jens Scharsig wrote:
> Dear Andreas Bießmann,
> 
> Please could you also apply this open patch.

sorry, this one slipped through. Please add some at91/atmel to the
subject line next time to get my attention.

> I've just tested this against the current atmel and master tree.

Ok, but this one seems to have style problems, please fix.

---8<---
WARNING: please, no space before tabs
#47: FILE: include/configs/eb_cpux9k2.h:51:
+#define CONFIG_SYS_TEXT_BASE ^I^I0x21f00000$

total: 0 errors, 1 warnings, 21 lines checked

NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
MULTISTATEMENT_MACRO_USE_DO_WHILE

U-Boot-eb_cpux9k2-add-ram-target-configuration.patch has style problems,
please review.
--->8---

Best regards

Andreas Bießmann
esw@bus-elektronik.de - Sept. 4, 2012, 8:24 a.m.
Dear Andreas Bießmann,

>>
>> Please could you also apply this open patch.
> 
> sorry, this one slipped through. Please add some at91/atmel to the
> subject line next time to get my attention.

Blame it on myself. I forgot the arch comment in subject

Best regards

Jens Scharsig

Patch

diff --git a/boards.cfg b/boards.cfg
index 3cf75c3..efca4fc 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -54,7 +54,8 @@  integratorcp_cm920t          arm         arm920t     integrator          armltd
 a320evb                      arm         arm920t     -                   faraday        a320
 at91rm9200ek                 arm         arm920t     at91rm9200ek        atmel          at91        at91rm9200ek
 at91rm9200ek_ram             arm         arm920t     at91rm9200ek        atmel          at91        at91rm9200ek:RAMBOOT
-eb_cpux9k2                   arm         arm920t     -                   BuS            at91
+eb_cpux9k2                   arm         arm920t     eb_cpux9k2          BuS            at91        eb_cpux9k2
+eb_cpux9k2_ram               arm         arm920t     eb_cpux9k2          BuS            at91        eb_cpux9k2:RAMBOOT
 cpuat91                      arm         arm920t     cpuat91             eukrea         at91        cpuat91
 cpuat91_ram                  arm         arm920t     cpuat91             eukrea         at91        cpuat91:RAMBOOT
 mx1ads                       arm         arm920t     -                   -              imx
diff --git a/include/configs/eb_cpux9k2.h b/include/configs/eb_cpux9k2.h
index eb05e2a..d7b6b57 100644
--- a/include/configs/eb_cpux9k2.h
+++ b/include/configs/eb_cpux9k2.h
@@ -44,7 +44,12 @@ 
 #define MACH_TYPE_EB_CPUX9K2		1977
 #define CONFIG_MACH_TYPE		MACH_TYPE_EB_CPUX9K2
 /*--------------------------------------------------------------------------*/
+#ifndef CONFIG_RAMBOOT
 #define CONFIG_SYS_TEXT_BASE 		0x00000000
+#else
+#define CONFIG_SKIP_LOWLEVEL_INIT
+#define CONFIG_SYS_TEXT_BASE 		0x21f00000
+#endif
 #define CONFIG_SYS_LOAD_ADDR		0x21000000  /* default load address */
 
 #define CONFIG_SYS_BOOT_SIZE		0x00 /* 0 KBytes */