New 'sandbox' test architecture for U-Boot

This adds required header files for the sandbox architecture, and a basic description of what sandbox is (README.sandbox).

Signed-off-by Simon Glass <[email protected]>

---
Changes in v6:
- Remove volatile keyword from arch/sandbox/include/asm/bitops.h

 arch/sandbox/include/asm/bitops.h         |  162 +++++++++++++++++++++++++++++
 arch/sandbox/include/asm/byteorder.h      |   40 +++++++
 arch/sandbox/include/asm/config.h         |   26 +++++
 arch/sandbox/include/asm/global_data.h    |   66 ++++++++++++
 arch/sandbox/include/asm/io.h             |   41 +++++++
 arch/sandbox/include/asm/posix_types.h    |   57 ++++++++++
 arch/sandbox/include/asm/ptrace.h         |   38 +++++++
 arch/sandbox/include/asm/string.h         |   23 ++++
 arch/sandbox/include/asm/system.h         |   36 +++++++
 arch/sandbox/include/asm/types.h          |   72 +++++++++++++
 arch/sandbox/include/asm/u-boot-sandbox.h |   38 +++++++
 arch/sandbox/include/asm/u-boot.h         |   61 +++++++++++
 arch/sandbox/include/asm/unaligned.h      |   23 ++++
 doc/README.sandbox                        |   53 ++++++++++
 include/common.h                          |    3 +
 15 files changed, 739 insertions(+), 0 deletions(-)
 create mode 100644 arch/sandbox/include/asm/bitops.h
 create mode 100644 arch/sandbox/include/asm/byteorder.h
 create mode 100644 arch/sandbox/include/asm/config.h
 create mode 100644 arch/sandbox/include/asm/global_data.h
 create mode 100644 arch/sandbox/include/asm/io.h
 create mode 100644 arch/sandbox/include/asm/posix_types.h

 create mode 100644 arch/sandbox/include/asm/ptrace.h
 create mode 100644 arch/sandbox/include/asm/string.h
 create mode 100644 arch/sandbox/include/asm/system.h
 create mode 100644 arch/sandbox/include/asm/types.h
 create mode 100644 arch/sandbox/include/asm/u-boot-sandbox.h
 create mode 100644 arch/sandbox/include/asm/u-boot.h
 create mode 100644 arch/sandbox/include/asm/unaligned.h
 create mode 100644 doc/README.sandbox

It is better to use %p in this case.

Signed-off-by Simon Glass <[email protected]>

Acked-by Mike Frysinger <[email protected]>

---
Changes in v2:
- Split this change out from 'Add architecture image support'

 common/image.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

We won't actually load an image with this architecture, but we still need to define it.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v3:
- Add architecture image support for sandbox

Changes in v5:
- Define IH_ARCH_DEFAULT which is now required

 arch/sandbox/include/asm/u-boot.h |    3 +++
 include/image.h                   |    1 +
 2 files changed, 4 insertions(+), 0 deletions(-)

Dear Simon Glass,

Quoted 6 lines

Changes in v5:
- Define IH_ARCH_DEFAULT which is now required

?arch/sandbox/include/asm/u-boot.h | ? ?3 +++
?include/image.h ? ? ? ? ? ? ? ? ? | ? ?1 +
?2 files changed, 4 insertions(+), 0 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

2011/10/18 Wolfgang Denk <[email protected]>:

Quoted 10 lines

Dear Simon Glass,

Quoted 6 lines

Changes in v5:
- Define IH_ARCH_DEFAULT which is now required

 arch/sandbox/include/asm/u-boot.h |    3 +++
 include/image.h                   |    1 +
 2 files changed, 4 insertions(+), 0 deletions(-)

Applied, thanks.

Sorry for late to reply your mail.
I'm doing PATCH v17 fix about NDS32 architecture against to your patches. I"ve found here you've defined IH_ARCH_SANDBOX in /include/image.h Just wondering about don't you need to also add this into ../common/image.c in the following table?
static const table_entry_t uimage_arch[]
[deleted]

        {       IH_ARCH_BLACKFIN,       "blackfin",     "Blackfin",     },
        {       IH_ARCH_AVR32,          "avr32",        "AVR32",        },
        {       -1,                     "",             "",             },

};

Because I'm wondering about if I need to fix this if you need to add IH_ARCH_SANDBOX into this structure.

Quoted 1 line

2011/10/18 Wolfgang Denk <[email protected]>:

Quoted 7 lines

static const table_entry_t uimage_arch[]
[deleted]

        {       IH_ARCH_BLACKFIN,       "blackfin",     "Blackfin",     },
        {       IH_ARCH_AVR32,          "avr32",        "AVR32",        },
        {       -1,                     "",             "",             },

};

Quoted 2 lines

Because I'm wondering about if I need to fix this if you need to add IH_ARCH_SANDBOX into this structure.

for now, i don't think so. "sandbox" isn't an arch you can create uimages for (atm?).
-mike

2011/10/18 Mike Frysinger:

Quoted 2 lines · On Tuesday 18 October 2011 04:28:23 ??? wrote:

Quoted 1 line

2011/10/18 Wolfgang Denk <[email protected]>:

[deleted]

Quoted 13 lines

Quoted 12 lines

static const table_entry_t uimage_arch[]
[deleted]

        {       IH_ARCH_BLACKFIN,       "blackfin",     "Blackfin",    
        }, {       IH_ARCH_AVR32,          "avr32",        "AVR32",    
           }, {       -1,                     "",             "",      
              },

};

Because I'm wondering about if I need to fix this if you need to add IH_ARCH_SANDBOX into this structure.

Quoted 2 lines

for now, i don't think so. "sandbox" isn't an arch you can create uimages for (atm?).

Is there no plan to run Chrome OS from flash storage?
Sorry I'm not familiar about the hardware planning about the Chrome OS. Just wondering if there is planning low-price Chrome netbook, the uimage might be
necessary for such kind of hardware.

Quoted 13 lines · On Tuesday 18 October 2011 11:43:49 ??? wrote:

2011/10/18 Mike Frysinger:

Quoted 2 lines · On Tuesday 18 October 2011 04:28:23 ??? wrote:

Quoted 1 line

2011/10/18 Wolfgang Denk <[email protected]>:

[deleted]

Quoted 8 lines

Quoted 8 lines

static const table_entry_t uimage_arch[]
[deleted]

        {       IH_ARCH_BLACKFIN,       "blackfin",     "Blackfin",
        }, {       IH_ARCH_AVR32,          "avr32",        "AVR32",
           }, {       -1,                     "",             "",
              },

Quoted 7 lines

Quoted 7 lines

Quoted 4 lines

};

Because I'm wondering about if I need to fix this if you need to add IH_ARCH_SANDBOX into this structure.

for now, i don't think so. "sandbox" isn't an arch you can create uimages for (atm?).

Quoted 6 lines

Is there no plan to run Chrome OS from flash storage?
Sorry I'm not familiar about the hardware planning about the Chrome OS. Just wondering if there is planning low-price Chrome netbook, the uimage might be
necessary for such kind of hardware.

"sandbox" isn't actual hardware. it's a native application on your system like ls or mv or mutt. it's purely for testing on your local machine. -mike

This sets __WORDSIZE to 8 correctly on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Update commit message to remove 'temporary'
- Allow __WORDSIZE to be defined in Makefile / elsewhere

 include/compiler.h |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)

This sets __WORDSIZE to 8 correctly on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Update commit message to remove 'temporary'
- Allow __WORDSIZE to be defined in Makefile / elsewhere

 include/compiler.h |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)

Dear Simon Glass,

Quoted 10 lines · In message <[email protected]> you wrote:

This sets __WORDSIZE to 8 correctly on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Update commit message to remove 'temporary'
- Allow __WORDSIZE to be defined in Makefile / elsewhere

 include/compiler.h |   12 +++++++++++-
 1 files changed, 11 insertions(+), 1 deletions(-)

This commit breaks building of all (or at least almost all, probbaly all ARM and PPC) boards:

+ ./MAKEALL ppc
In file included from /home/wd/git/u-boot/work/include/image.h:36,

                 from include/common.h:117:

/home/wd/git/u-boot/work/include/compiler.h:119:2: error: #error "__SIZEOF_LONG__ has unexpected value" /home/wd/git/u-boot/work/include/compiler.h:128:2: error: #error "__WORDSIZE has unexpected value" In file included from /home/wd/git/u-boot/work/include/image.h:36, from include/common.h:117:
...

+ ./MAKEALL arm
In file included from /home/wd/git/u-boot/work/include/image.h:36,

                 from include/common.h:117:

/home/wd/git/u-boot/work/include/compiler.h:119:2: error: #error "__SIZEOF_LONG__ has unexpected value" /home/wd/git/u-boot/work/include/compiler.h:128:2: error: #error "__WORDSIZE has unexpected value" In file included from /home/wd/git/u-boot/work/include/image.h:36, from include/common.h:117:
...

This happend both when building on i686 and on x86_64 build hosts.

Please fix.

Best regards,

Wolfgang Denk

Finally, I'm not alone when it comes to breaking all boards ! :-)

Dear Marek Vasut,

Quoted 2 lines · In message <[email protected]> you wrote:

Finally, I'm not alone when it comes to breaking all boards ! :-)

Guess we all do this, all the time.

Some of us learn from it, others buy higher quality asbestos underwear ;-)

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de By the way, ALL software projects are done by iterative ?prototyping. Some companies call their prototypes "releases", that's all.

Dear Simon Glass,

Quoted 10 lines · In message <[email protected]> you wrote:

This sets __WORDSIZE to 8 correctly on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Update commit message to remove 'temporary'
- Allow __WORDSIZE to be defined in Makefile / elsewhere

?include/compiler.h | ? 12 +++++++++++-
?1 files changed, 11 insertions(+), 1 deletions(-)

This commit breaks building of all (or at least almost all, probbaly all ARM and PPC) boards:

+ ./MAKEALL ppc
In file included from /home/wd/git/u-boot/work/include/image.h:36,
? ? ? ? ? ? ? ? from include/common.h:117:
/home/wd/git/u-boot/work/include/compiler.h:119:2: error: #error "__SIZEOF_LONG__ has unexpected value" /home/wd/git/u-boot/work/include/compiler.h:128:2: error: #error "__WORDSIZE has unexpected value" In file included from /home/wd/git/u-boot/work/include/image.h:36,
? ? ? ? ? ? ? ? from include/common.h:117:
...

+ ./MAKEALL arm
In file included from /home/wd/git/u-boot/work/include/image.h:36,
? ? ? ? ? ? ? ? from include/common.h:117:
/home/wd/git/u-boot/work/include/compiler.h:119:2: error: #error "__SIZEOF_LONG__ has unexpected value" /home/wd/git/u-boot/work/include/compiler.h:128:2: error: #error "__WORDSIZE has unexpected value" In file included from /home/wd/git/u-boot/work/include/image.h:36,
? ? ? ? ? ? ? ? from include/common.h:117:
...

This happend both when building on i686 and on x86_64 build hosts.

Please fix.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de After a time, you may find that "having" is not so pleasing a thing, after all, as "wanting." ?It is not logical, but it is often true. ? ? ? ?-- Spock, "Amok Time", stardate 3372.7

OK I will take a look now.

Dear Simon,

Quoted 2 lines · In message <CAPnjgZ1GpZ6v38pdNVRWW4fKToebFe4_uF9tL906d9kM4qB2Gw@mail.gmail.com>…

OK I will take a look now.

Thanks. ?Note: I haven't pushed this upstream yet, so I can still rebase and merge a fix into this commit.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de My brother sent me a postcard the other day with this ?big ?sattelite photo ?of the entire earth on it. On the back it said: "Wish you were here". ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?- Steven Wright

This is an initial implementation with all functions defined but not working.

The lds file is very simple since we can mostly rely on the linker defaults.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Remove setting of LDSCRIPT (top level Makefile does this anyway)
- Add comment to do_reset() to justify the sandbox implementation
- Move lds script into cpu/sandbox

Changes in v3:
- Change do_reset() back to what it was
- Move cpu files into arch/sandbox/cpu

 arch/sandbox/config.mk      |   20 +++++++++++++
 arch/sandbox/cpu/Makefile   |   47 ++++++++++++++++++++++++++++++++
 arch/sandbox/cpu/cpu.c      |   63 +++++++++++++++++++++++++++++++++++++++++++
 arch/sandbox/cpu/u-boot.lds |   34 +++++++++++++++++++++++
 4 files changed, 164 insertions(+), 0 deletions(-)
 create mode 100644 arch/sandbox/config.mk
 create mode 100644 arch/sandbox/cpu/Makefile
 create mode 100644 arch/sandbox/cpu/cpu.c
 create mode 100644 arch/sandbox/cpu/u-boot.lds

These files are taken from the ARM board implementation and then reduced to remove unneeded cruft.

Ideally we would work towards unifying arch/xxx/lib files, particularly board.c.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Fix commit message typo, sadly
- Remove ARM cruft from Makefile
- Remove version_string and make display_banner() call display_options()
- Tidy up DRAM logic (but I think it is still needed to fit in with U-Boot)
- Remove use of CONFIG_SYS_NO_FLASH
- Make board_init_f() call board_init_r() for now
- Make board_init_r() call main_loop() for now

Changes in v4:
- Add CONFIG_SYS_SDRAM_SIZE for size of emulated SDRAM
- Drop CONFIG_POST support
- Drop call to board_init()
- Remove printing of memory setup which is meaningless in sandbox

 arch/sandbox/lib/Makefile     |   51 ++++++++
 arch/sandbox/lib/board.c      |  276 +++++++++++++++++++++++++++++++++++++++++
 arch/sandbox/lib/interrupts.c |   39 ++++++
 3 files changed, 366 insertions(+), 0 deletions(-)
 create mode 100644 arch/sandbox/lib/Makefile
 create mode 100644 arch/sandbox/lib/board.c
 create mode 100644 arch/sandbox/lib/interrupts.c

This adds basic files for the sandbox board. The lds file is very simple since we can rely mostly on the linker defaults.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Remove clean and dist-clean targets from Makefile
- Move lds script out of the board directory

Changes in v4:
- Remove board/sandbox/common directory as it is not used

Changes in v5:
- Simplify the declaration of gd_t in sandbox's board.c

 board/sandbox/sandbox/Makefile  |   42 +++++++++++++++++++++++++++++++++
 board/sandbox/sandbox/sandbox.c |   49 +++++++++++++++++++++++++++++++++++++++
 boards.cfg                      |    1 +
 3 files changed, 92 insertions(+), 0 deletions(-)
 create mode 100644 board/sandbox/sandbox/Makefile
 create mode 100644 board/sandbox/sandbox/sandbox.c

This adds basic files for the sandbox board. The lds file is very simple since we can rely mostly on the linker defaults.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v5:
- Simplify the declaration of gd_t in sandbox's board.c

?board/sandbox/sandbox/Makefile ?| ? 42 +++++++++++++++++++++++++++++++++
?board/sandbox/sandbox/sandbox.c | ? 49 +++++++++++++++++++++++++++++++++++++++
?boards.cfg ? ? ? ? ? ? ? ? ? ? ?| ? ?1 +

?3 files changed, 92 insertions(+), 0 deletions(-) ?create mode 100644 board/sandbox/sandbox/Makefile

(quoted diff) +1 -0

+include $(TOPDIR)/config.mk

2011/10/11 Simon Glass :

board/sandbox/sandbox/Makefile line 33

Quoted 11 lines

board/sandbox/sandbox/Makefile new file +11 -0

+include $(TOPDIR)/config.mk
+
+LIB    = $(obj)lib$(BOARD).o
+
+COBJS  := $(BOARD).o
+
+SRCS   := $(COBJS:.o=.c)
+OBJS   := $(addprefix $(obj),$(COBJS))
+
+$(LIB):        $(obj).depend $(OBJS)
+       $(AR) $(ARFLAGS) $@ $(OBJS)

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
It has been suggested by Mike that you should use something like the following because $(AR) might broken.

$(LIB): $(OBJS)
$(call cmd_link_o_target, $(OBJS))

Quoted 17 lines · On Tuesday 18 October 2011 04:43:09 ??? wrote:

2011/10/11 Simon Glass :

Review

Quoted 11 lines

(quoted diff) +11 -0

+include $(TOPDIR)/config.mk
+
+LIB ? ?= $(obj)lib$(BOARD).o
+
+COBJS ?:= $(BOARD).o
+
+SRCS ? := $(COBJS:.o=.c)
+OBJS ? := $(addprefix $(obj),$(COBJS))
+
+$(LIB): ? ? ? ?$(obj).depend $(OBJS)
+ ? ? ? $(AR) $(ARFLAGS) $@ $(OBJS)

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
It has been suggested by Mike that you should use something like the following because $(AR) might broken.

Quoted 2 lines

$(LIB): ? ? ? ?$(OBJS)
$(call cmd_link_o_target, $(OBJS))

yeah i missed this chunk of Simon's patch (i caught some others in his patchset). ?i've fixed this in my tree-wide $(AR) fixup, so Simon won't have to worry about posting an update.
-mike

This is required for the bdinfo command to work.

This also cleans up the #ifdef mess for ethernet and lnum a little.

Signed-off-by Simon Glass <[email protected]>

---
 common/cmd_bdinfo.c |   34 ++++++++++++++++++++++++++++++----
 1 files changed, 30 insertions(+), 4 deletions(-)

Dear Simon Glass,

Quoted 2 lines · In message <[email protected]> you wrote:

This is required for the bdinfo command to work.

Quoted 6 lines

This also cleans up the #ifdef mess for ethernet and lnum a little.

Signed-off-by Simon Glass <[email protected]>

---
?common/cmd_bdinfo.c | ? 34 ++++++++++++++++++++++++++++++---- ?1 files changed, 30 insertions(+), 4 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Our way is peace.
? ? ? ?-- Septimus, the Son Worshiper, "Bread and Circuses", ? ? ? ? ? stardate 4040.7.

This is required for the bdinfo command to work.

This also cleans up the #ifdef mess for ethernet and lnum a little.

Signed-off-by Simon Glass <[email protected]>

Dear Simon Glass,

Quoted 4 lines · In message <[email protected]> you wrote:

This is required for the bdinfo command to work.

This also cleans up the #ifdef mess for ethernet and lnum a little.

Quoted 1 line

Signed-off-by Simon Glass <[email protected]>

...

common/cmd_bdinfo.c DECLARE_GLOBAL_DATA_PTR;

Quoted 3 lines

common/cmd_bdinfo.c +2 -1

-#if !(defined(CONFIG_ARM) || defined(CONFIG_M68K)) || defined(CONFIG_CMD_NET)
+#if !(defined(CONFIG_ARM) || defined(CONFIG_M68K) || defined(CONFIG_SANDBOX))
+ ? ? || defined(CONFIG_CMD_NET)

Did you _ever_ try compiling this?

Added '\\' when checking in.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de We, the unwilling, led by the unknowing, are doing the impossible for the ungrateful. We have done so much, for so long, with so little, we are now qualified to do anything with nothing.

Yes, but only with ARM or SANDBOX, sorry. I am set up for PPC now although I do get errors with MAKEALL:

Quoted 2 lines · On Monday 17 October 2011 18:12:13 Simon Glass wrote:

Yes, but only with ARM or SANDBOX, sorry. I am set up for PPC now although I do get errors with MAKEALL:

MAKEALL is a pita for testing all arches. ?i wrote a local script to make build testing a lot simpler. ?see attached.

it takes care of figuring out the right CROSS_COMPILE for the requested target, setting up ccache, and enabling private libgcc when possible. ?the next thing i plan on adding is automatic archiving of compiled objects in a per-board dir so that i can easily compare the compiled differences between patches.

so now i just run:
? ? ? ?../buildall
(this will build *all* arches)

or for a quick cross-arch smoke test:
? ? ? ?../buildall smoke

or it takes any target that MAKEALL does:
? ? ? ?../buildall board board arch board arch etc...

caveat: it assumes a tree layout like so:
/usr/local/src/u-boot/

?|- toolchains/ ? ? ? ? toolchains from http://kernel.org/
?| ?|- arm-linux/
?| ?|- avr32-linux/
?| ?|- ........
?| ?`- mips-linux/

?`- tmpu/ ? ? ? ? ? ? ? ? ? ? ? tree like git://git.denx.de/u-boot.git
-mike

Quoted 8 lines · Mike Frysinger

caveat: it assumes a tree layout like so:
/usr/local/src/u-boot/

 |- toolchains/         toolchains from http://kernel.org/
 |  |- arm-linux/
 |  |- avr32-linux/
 |  |- ........
 |  `- mips-linux/

`- tmpu/ tree like git://git.denx.de/u-boot.git

This looks great. Do you have a full link to one of the toolchains on kernel.org please? - everything seems to have changed there.

Quoted 10 lines · On Monday 17 October 2011 18:47:10 Simon Glass wrote:

Quoted 8 lines · Mike Frysinger

caveat: it assumes a tree layout like so:
/usr/local/src/u-boot/

?|- toolchains/ ? ? ? ? toolchains from http://kernel.org/
?| ?|- arm-linux/
?| ?|- avr32-linux/
?| ?|- ........
?| ?`- mips-linux/

?`- tmpu/ ? ? ? ? ? ? ? ? ? ? ? tree like git://git.denx.de/u-boot.git

Quoted 2 lines

This looks great. Do you have a full link to one of the toolchains on kernel.org please? - everything seems to have changed there.

they don't seem to have been restored yet. ?i'll tar up the trees i have locally and post them.
-mike

Quoted 12 lines · On Monday 17 October 2011 18:47:10 Simon Glass wrote:

Quoted 11 lines · Mike Frysinger

caveat: it assumes a tree layout like so:
/usr/local/src/u-boot/

 |- toolchains/         toolchains from http://kernel.org/
 |
 |  |- arm-linux/
 |  |- avr32-linux/
 |  |- ........
 |  
 |  `- mips-linux/

Quoted 4 lines

Quoted 1 line

`- tmpu/ tree like git://git.denx.de/u-boot.git

This looks great. Do you have a full link to one of the toolchains on kernel.org please? - everything seems to have changed there.

they don't seem to have been restored yet. i'll tar up the trees i have locally and post them.

Quoted 13 lines · On Monday 17 October 2011 18:56:24 Mike Frysinger wrote:

Quoted 12 lines · On Monday 17 October 2011 18:47:10 Simon Glass wrote:

Quoted 11 lines · Mike Frysinger

caveat: it assumes a tree layout like so:
/usr/local/src/u-boot/

?|- toolchains/ ? ? ? ? toolchains from http://kernel.org/
?|
?| ?|- arm-linux/
?| ?|- avr32-linux/
?| ?|- ........
?|
?| ?`- mips-linux/

Quoted 7 lines

Quoted 4 lines

Quoted 1 line

?`- tmpu/ ? ? ? ? ? ? ? ? ? ? ? tree like git://git.denx.de/u-boot.git

This looks great. Do you have a full link to one of the toolchains on kernel.org please? - everything seems to have changed there.

they don't seem to have been restored yet. ?i'll tar up the trees i have locally and post them.

i've posted a bunch here:
http://dev.gentoo.org/~vapier/u-boot/

i need to build/post sh4/sparc ones because the one kernel.org don't build u- boot properly (multilib / libgcc issues). ?i've been using Gentoo built ones. -mike

This adds sandbox architecture support to bootm, although it is probably not useful to load sandbox code into the address space and execute it.

This change at least make the file build correctly on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Fix cast of int to pointer instead of just removing the code

 common/cmd_bootm.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

We prefer to U-Boot's malloc but for now it is easier to use the C library's version.

Signed-off-by Simon Glass <[email protected]>

---
 common/Makefile |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

This is less useful on the sandbox architecture since we can simply link all our code with U-Boot. However some interest has been expressed in implementing this support.

For now, it is disabled.

Signed-off-by Simon Glass <[email protected]>

---
 Makefile |    9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

Dear Simon Glass,

Quoted 4 lines · In message <[email protected]> you wrote:

This is less useful on the sandbox architecture since we can simply link all our code with U-Boot. However some interest has been expressed in implementing this support.

Quoted 6 lines

For now, it is disabled.

Signed-off-by Simon Glass <[email protected]>

---
?Makefile | ? ?9 ++++++---
?1 files changed, 6 insertions(+), 3 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de There is a multi-legged creature crawling on your shoulder. ? ? ? ?-- Spock, "A Taste of Armageddon", stardate 3193.9

By default sections are 16-byte aligned on some architectures, but the command name structure (struct cmd_tbl_s) does not have padding to 16 bytes. We use the linker to collect elements of an array of these structures and then at run-time we want this array to be accessible.

If the linker puts padding between elements then we cannot access the array properly.

This change reduces the alignment to 4-bytes so that the command table can be accessed correctly on any architecture.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v4:
- Add more info to command section alignment commit

 include/command.h |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

We want to keep all OS-dependent code in once place, with a simple interface to U-Boot. For now, this is that place.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Move os layer into arch/sandbox
- Remove clean and dist-clean targets from Makefile
- Try and fail to remove the global -I/usr/include

Changes in v3:
- Add os_exit() which always exits with success for now
- Add function comments to header file
- Call os_exit() in do_reset()
- Move cpu files into arch/sandbox/cpu

Changes in v4:
- Allow os_exit() to exit with a non-zero return code

 arch/sandbox/cpu/Makefile |    5 ++-
 arch/sandbox/cpu/cpu.c    |    7 ++--
 arch/sandbox/cpu/os.c     |   55 ++++++++++++++++++++++++++++++++++
 include/os.h              |   73 +++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 135 insertions(+), 5 deletions(-)
 create mode 100644 arch/sandbox/cpu/os.c
 create mode 100644 include/os.h

Add a main program so that we can run U-Boot.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v4:
- Move main() into arch/sandbox/cpu/start.c

 arch/sandbox/cpu/Makefile |    2 +-
 arch/sandbox/cpu/start.c  |   33 +++++++++++++++++++++++++++++++++
 2 files changed, 34 insertions(+), 1 deletions(-)
 create mode 100644 arch/sandbox/cpu/start.c

This uart simply writes to stdout and reads from stdin. We might imagine instead buffering the data so that a test interface can check output and inject input.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v4:
- Remove serial_exit()

 drivers/serial/Makefile  |    1 +
 drivers/serial/sandbox.c |   63 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 64 insertions(+), 0 deletions(-)
 create mode 100644 drivers/serial/sandbox.c

This basic provides required features along with a basic command set.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Remove CONFIG_SYS_SDRAM_BASE which is always 0 for sandbox boards
- Fix #define<tab>
- Remove CONFIG_LMB

Changes in v4:
- Add CONFIG_SYS_SDRAM_SIZE for size of emulated SDRAM
- Add #ifdef protection around sandbox.h header file

 include/configs/sandbox.h |   85 +++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 85 insertions(+), 0 deletions(-)
 create mode 100644 include/configs/sandbox.h

This fixes a few problems when building on 64-bit machines.

Signed-off-by Simon Glass <[email protected]>

---
 common/cmd_mem.c     |    2 +-
 common/fdt_support.c |    8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

The dependency rules are currently done in a shell 'for' loop. This does not permit Makefile variables to adjust preprocessor flags as is done with normal compile flags, using the CFLAGS_path/file.o syntax.

This change moves the dependency generation into the Makefile itself, and permits a CPPFLAGS_path/file.o to adjust preprocessor flags on a file or directory basis.

The CPPFLAGS_... variable is also folded into CFLAGS during the build.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v3:
- Change U-Boot's dependency generation to permit per-file flags

Changes in v4:
- 'Improve' dependency generation to deal with source files containing paths
- Make sure that clobber target removes all dependency files

Changes in v5:
- Fix dependency output to include $(obj) for out-of-tree builds

 .gitignore |    2 +-
 Makefile   |    2 +-
 config.mk  |    7 +++++++
 rules.mk   |   44 +++++++++++++++++++++++++++++++++-----------
 4 files changed, 42 insertions(+), 13 deletions(-)

The dependency rules are currently done in a shell 'for' loop. This does not permit Makefile variables to adjust preprocessor flags as is done with normal compile flags, using the CFLAGS_path/file.o syntax.

This change moves the dependency generation into the Makefile itself, and permits a CPPFLAGS_path/file.o to adjust preprocessor flags on a file or directory basis.

The CPPFLAGS_... variable is also folded into CFLAGS during the build.

Signed-off-by Simon Glass <[email protected]>

Dear Simon Glass,

Quoted 11 lines · In message <[email protected]> you wrote:

The dependency rules are currently done in a shell 'for' loop. This does not permit Makefile variables to adjust preprocessor flags as is done with normal compile flags, using the CFLAGS_path/file.o syntax.

This change moves the dependency generation into the Makefile itself, and permits a CPPFLAGS_path/file.o to adjust preprocessor flags on a file or directory basis.

The CPPFLAGS_... variable is also folded into CFLAGS during the build.

Signed-off-by Simon Glass <[email protected]>

This commit breaks building a number of PPC boards like that:

Configuring for acadia_nand - Board: acadia, Options: NAND_U_BOOT,SYS_TEXT_BASE=0x01000000
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/start.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/resetvec.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/cache.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/gpio.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/nand_boot.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/nand_ecc.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/memory.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/ndfc.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/pll.c: No such file or directory
ppc_6xx-gcc: no input files
make[1]: *** No rule to make target /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/.depend', needed by all'. ?Stop.

Affected boards:

? ? ? ?MPC8313ERDB_NAND_33 ? ? P1020RDB_NAND ? ? ? ? ? SIMPC8313_LP ? ? ? ?MPC8313ERDB_NAND_66 ? ? P1021RDB-PC_36BIT_NAND ?SIMPC8313_SP ? ? ? ?MPC8315ERDB_NAND ? ? ? ?P1021RDB-PC_NAND ? ? ? ?acadia_nand ? ? ? ?MPC8536DS_NAND ? ? ? ? ?P1023RDS_NAND ? ? ? ? ? bamboo_nand ? ? ? ?MPC8569MDS_NAND ? ? ? ? P1024RDB_NAND ? ? ? ? ? canyonlands_nand ? ? ? ?MPC8572DS_NAND ? ? ? ? ?P1025RDB_NAND ? ? ? ? ? glacier_nand ? ? ? ?P1010RDB_36BIT_NAND ? ? P2010RDB_NAND ? ? ? ? ? haleakala_nand ? ? ? ?P1010RDB_NAND ? ? ? ? ? P2020RDB-PC_36BIT_NAND ?kilauea_nand ? ? ? ?P1011RDB_NAND ? ? ? ? ? P2020RDB-PC_NAND ? ? ? ?rainier_nand ? ? ? ?P1020RDB-PC_36BIT_NAND ?P2020RDB_NAND ? ? ? ? ? sequoia_nand ? ? ? ?P1020RDB-PC_NAND

Please fix.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Documentation is the castor oil of programming.
Managers know it must be good because the programmers hate it so much.

Hi Wolfgang,

Quoted 41 lines · Wolfgang Denk

Dear Simon Glass,

Quoted 11 lines · In message <[email protected]> you wrote:

The dependency rules are currently done in a shell 'for' loop. This does not permit Makefile variables to adjust preprocessor flags as is done with normal compile flags, using the CFLAGS_path/file.o syntax.

This change moves the dependency generation into the Makefile itself, and permits a CPPFLAGS_path/file.o to adjust preprocessor flags on a file or directory basis.

The CPPFLAGS_... variable is also folded into CFLAGS during the build.

Signed-off-by Simon Glass <[email protected]>

This commit breaks building a number of PPC boards like that:

Configuring for acadia_nand - Board: acadia, Options: NAND_U_BOOT,SYS_TEXT_BASE=0x01000000
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/start.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/resetvec.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/cache.S: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/gpio.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/nand_boot.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/nand_ecc.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/memory.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/ndfc.c: No such file or directory
ppc_6xx-gcc: no input files
ppc_6xx-gcc: /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/pll.c: No such file or directory
ppc_6xx-gcc: no input files
make[1]: *** No rule to make target /work/wd/tmp-ppc/nand_spl/board/amcc/acadia/.depend', needed by all'. ?Stop.

Affected boards:

Quoted 12 lines

? ? ? ?MPC8313ERDB_NAND_33 ? ? P1020RDB_NAND ? ? ? ? ? SIMPC8313_LP ? ? ? ?MPC8313ERDB_NAND_66 ? ? P1021RDB-PC_36BIT_NAND ?SIMPC8313_SP ? ? ? ?MPC8315ERDB_NAND ? ? ? ?P1021RDB-PC_NAND ? ? ? ?acadia_nand ? ? ? ?MPC8536DS_NAND ? ? ? ? ?P1023RDS_NAND ? ? ? ? ? bamboo_nand ? ? ? ?MPC8569MDS_NAND ? ? ? ? P1024RDB_NAND ? ? ? ? ? canyonlands_nand ? ? ? ?MPC8572DS_NAND ? ? ? ? ?P1025RDB_NAND ? ? ? ? ? glacier_nand ? ? ? ?P1010RDB_36BIT_NAND ? ? P2010RDB_NAND ? ? ? ? ? haleakala_nand ? ? ? ?P1010RDB_NAND ? ? ? ? ? P2020RDB-PC_36BIT_NAND ?kilauea_nand ? ? ? ?P1011RDB_NAND ? ? ? ? ? P2020RDB-PC_NAND ? ? ? ?rainier_nand ? ? ? ?P1020RDB-PC_36BIT_NAND ?P2020RDB_NAND ? ? ? ? ? sequoia_nand ? ? ? ?P1020RDB-PC_NAND

This might be something to do with SPL - I am looking at it today.

Regards,
Simon

Quoted 1 line

Please fix.

Quoted 12 lines

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Documentation is the castor oil of programming.
Managers know it must be good because the programmers hate it so much.

I have submitted a patch for this, tested on acadia like this:

ARCH=powerpc make O=ppc -s acadia_nand_config && ARCH=powerpc make O> =ppc -s

and similarly without the O= part.

Dear Simon,

Quoted 6 lines · In message <CAPnjgZ1qOjQw2w_oUSOriDhqh4JSpj5tMABBuEcHcpg+PQFHow@mail.gmail.com>…

I have submitted a patch for this, tested on acadia like this:

ARCH=powerpc make O=ppc -s acadia_nand_config && ARCH=powerpc make O> =ppc -s

and similarly without the O= part.

Thanks a lot!

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Microsoft Multitasking:
? ? ? ? ? ? ? ? ? ? several applications can crash at the same time.

Since we want want to have a standard GPIO interface, this adds a definition for this into include/asm-generic/gpio.h.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v5:
- Move the introduction of include/asm-generic/gpio.h into a separate commit
- Add comments to GPIO functions

 include/asm-generic/gpio.h |   74 ++++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 74 insertions(+), 0 deletions(-)
 create mode 100644 include/asm-generic/gpio.h

At this point U-Boot will build and run on x86 under Linux.

The idea is to define a new architecture called 'sandbox', alongside ARM and x86. This runs natively on Linux to suit the host machine. All hardware access is either omitted or emulated.

The purpose of this system is to test the bulk of the non-hardware-specific U-Boot code. We can mock the SPI flash, GPIOs, UART and keyboard, then test that U-Boot behaves as we wish.

Signed-off-by Simon Glass <[email protected]>

---
Changes in v2:
- Rebase to master
- Tidy top-level Makefile to minimise changes

Changes in v4:
- Drop board/sandbox/common/board.c for now (no board_init() needed)
- Move main() into start.c to remove one ifdef from the top level Makefile
- Fix up Makefile for non-sandbox builds

 Makefile |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)