Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-18 Thread Matthias Kaehlcke
hi, El Tue, Dec 08, 2009 at 09:33:18PM +0100 Wolfgang Denk ha dit: > After relocation we have a full-blown, "normal" C runtime environment. > Before that, you can use C as well, if you mind the restrictions (like > data and bss not being available/writable yet, no real stack, etc.). as suggested

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-08 Thread Matthias Kaehlcke
hi wolfgang, El Tue, Dec 08, 2009 at 09:33:18PM +0100 Wolfgang Denk ha dit: > In message <20091208151358.gd31...@darwin> you wrote: > > > > i am starting to look at this issue and it seems i need some more guidance: > > > > before relocating U-Boot to RAM for ARM920T processors a jump to the >

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-08 Thread Wolfgang Denk
Dear "kevin.morf...@fearnside-systems.co.uk", In message <4b1e76e9.2050...@fearnside-systems.co.uk> you wrote: > > I would've thought that you need a working stack before you can run > 'C' code so you'd need the RAM timing set up before you could run In general this is correct, but then - what

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-08 Thread Wolfgang Denk
Dear Matthias Kaehlcke, In message <20091208151358.gd31...@darwin> you wrote: > > i am starting to look at this issue and it seems i need some more guidance: > > before relocating U-Boot to RAM for ARM920T processors a jump to the > label *lowlevel_init* is performed, where RAM timings are suppo

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-08 Thread kevin.morf...@fearnside-systems.co.uk
Matthias Kaehlcke wrote: > Hello Wolfgang, > > El Tue, Dec 08, 2009 at 12:51:47AM +0100 Wolfgang Denk ha dit: > U-Boot uses assembler only when ther eis no reasonable way to implement the code in C, and I don't see any such justification here. Please rewrite all this in C. >>> i'

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-08 Thread Matthias Kaehlcke
Hello Wolfgang, El Tue, Dec 08, 2009 at 12:51:47AM +0100 Wolfgang Denk ha dit: > > > U-Boot uses assembler only when ther eis no reasonable way to > > > implement the code in C, and I don't see any such justification here. > > > Please rewrite all this in C. > > > > i'm just starting to get my f

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-07 Thread Wolfgang Denk
Dear Matthias Kaehlcke, In message <20091207233313.ga31...@darwin> you wrote: > > > Please don't do scripting in the Makefile. Move this logic into your > > board config file instead. > > i got the inspiration to handle it this way from the U-Boot Makefile > (Total5100, TQM5200, et al). could yo

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-07 Thread Matthias Kaehlcke
hi wolfgang, thanks a lot for you review, below some comments and questions El Mon, Dec 07, 2009 at 08:46:29PM +0100 Wolfgang Denk ha dit: > In message <20091206145444.ge22...@darwin> you wrote: > > Add support for the Cirrus EP93xx platform and EDB93xx boards > > In addition to Tom's comments:

Re: [U-Boot] [PATCH] ARM: Add support for edb93xx boards

2009-12-07 Thread Wolfgang Denk
Dear Matthias Kaehlcke, In message <20091206145444.ge22...@darwin> you wrote: > Add support for the Cirrus EP93xx platform and EDB93xx boards In addition to Tom's comments: > @@ -2429,6 +2429,42 @@ TQM834x_config:unconfig > vme8349_config: unconfig > @$(MKCONFIG) $(@: