public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
@ 2003-05-06  7:26 Robert Schwebel
  2003-05-06  7:44 ` Wolfgang Denk
  2003-05-06 12:19 ` Kyle Harris
  0 siblings, 2 replies; 7+ messages in thread
From: Robert Schwebel @ 2003-05-06  7:26 UTC (permalink / raw)
  To: u-boot

Hi, 

I would like to propose that we move the memsetup.S code from the board
directories for the Innokom and CSB226 (soon also Gealog) plattform to
the cpu/xscale directory. 

- the code is absolutely generic, can be configured via CFG_ values
- it has been cleaned up
- I see no reason why it should not work on every PXA250 plattform 
- it would avoid code quadruplication ;)
- the other board maintainers probably had to test it

Comments? Shall I submit a patch? 

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Braunschweiger Str. 79,  31134 Hildesheim, Germany
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
    Phone: +49-5121-28619-0 |  Fax: +49-5121-28619-4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06  7:26 [U-Boot-Users] [RFD] move memsetup.S to CPU directory Robert Schwebel
@ 2003-05-06  7:44 ` Wolfgang Denk
  2003-05-06 12:19 ` Kyle Harris
  1 sibling, 0 replies; 7+ messages in thread
From: Wolfgang Denk @ 2003-05-06  7:44 UTC (permalink / raw)
  To: u-boot

In message <20030506072640.GO2263@pengutronix.de> you wrote:
> 
> I would like to propose that we move the memsetup.S code from the board
> directories for the Innokom and CSB226 (soon also Gealog) plattform to
> the cpu/xscale directory. 

OK.

> Comments? Shall I submit a patch? 

Yes, please.

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
That said, there may be good reasons for what you did beyond obsequi-
ous sycophantic parody. Perhaps you might be so kind as to elucidate.
         -- Tom Christiansen in <5ldjbm$jtk$1@csnews.cs.colorado.edu>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06  7:26 [U-Boot-Users] [RFD] move memsetup.S to CPU directory Robert Schwebel
  2003-05-06  7:44 ` Wolfgang Denk
@ 2003-05-06 12:19 ` Kyle Harris
  2003-05-06 12:35   ` Robert Schwebel
  1 sibling, 1 reply; 7+ messages in thread
From: Kyle Harris @ 2003-05-06 12:19 UTC (permalink / raw)
  To: u-boot

On Tuesday 06 May 2003 03:26 am, Robert Schwebel wrote:

> Hi,
>
> I would like to propose that we move the memsetup.S code from the board
> directories for the Innokom and CSB226 (soon also Gealog) plattform to
> the cpu/xscale directory.

There are other Xscale CPUs. Is the memsetup.S generic enough to support them 
as well. I don't know the answer to this yet. I will be porting u-boot to 
IXP420 in the next few weeks and will have a better idea. Maybe someone else 
already knows.

Kyle.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06 12:19 ` Kyle Harris
@ 2003-05-06 12:35   ` Robert Schwebel
  2003-05-06 12:55     ` Kyle Harris
  0 siblings, 1 reply; 7+ messages in thread
From: Robert Schwebel @ 2003-05-06 12:35 UTC (permalink / raw)
  To: u-boot

On Tue, May 06, 2003 at 08:19:44AM -0400, Kyle Harris wrote:
> There are other Xscale CPUs. Is the memsetup.S generic enough to
> support them as well. I don't know the answer to this yet. I will be
> porting u-boot to IXP420 in the next few weeks and will have a better
> idea. Maybe someone else already knows.

My impression is that at the moment everything that's labled "XScale" in
u-boot is "PXA" in reality, so an audit has to be done anyway.

Do you have information about how the memory controller of the IXP is
different from the PXAs? 

Robert
-- 
 Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
 Pengutronix - Linux Solutions for Science and Industry
   Braunschweiger Str. 79,  31134 Hildesheim, Germany
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
    Phone: +49-5121-28619-0 |  Fax: +49-5121-28619-4

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06 12:35   ` Robert Schwebel
@ 2003-05-06 12:55     ` Kyle Harris
  2003-05-06 14:00       ` Wolfgang Denk
  0 siblings, 1 reply; 7+ messages in thread
From: Kyle Harris @ 2003-05-06 12:55 UTC (permalink / raw)
  To: u-boot

On Tuesday 06 May 2003 08:35 am, Robert Schwebel wrote:

> On Tue, May 06, 2003 at 08:19:44AM -0400, Kyle Harris wrote:
> > There are other Xscale CPUs. Is the memsetup.S generic enough to
> > support them as well. I don't know the answer to this yet. I will be
> > porting u-boot to IXP420 in the next few weeks and will have a better
> > idea. Maybe someone else already knows.
>
> My impression is that at the moment everything that's labled "XScale" in
> u-boot is "PXA" in reality, so an audit has to be done anyway.
>
> Do you have information about how the memory controller of the IXP is
> different from the PXAs?

Actually, the mem controller is completely different. The IXP has separate 
SDRAM and expansion buses. Most other parts of cpu/xscale will be different 
as well (e.g., interrupt.c, serial.c). It may be that we need to have ixp and 
pxa subdirs. We can leave xscale (really pxa) as is, move memsetup there, and 
create a new ixp.

Kyle.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06 12:55     ` Kyle Harris
@ 2003-05-06 14:00       ` Wolfgang Denk
  2003-05-06 14:16         ` Kyle Harris
  0 siblings, 1 reply; 7+ messages in thread
From: Wolfgang Denk @ 2003-05-06 14:00 UTC (permalink / raw)
  To: u-boot

In message <E19D26b-0003S5-00@smtp10.atl.mindspring.net> you wrote:
> 
> Actually, the mem controller is completely different. The IXP has separate 
> SDRAM and expansion buses. Most other parts of cpu/xscale will be different 
> as well (e.g., interrupt.c, serial.c). It may be that we need to have ixp and 
> pxa subdirs. We can leave xscale (really pxa) as is, move memsetup there, and 
> create a new ixp.

Then let's rename xscale ==> pxa and add a new ixp directory


Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
Abstainer: A weak person who yields to the temptation of denying him-
self a pleasure. A total abstainer is one who  abstains  from  every-
thing  but  abstention, and especially from inactivity in the affairs
of others.                                           - Ambrose Bierce

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [U-Boot-Users] [RFD] move memsetup.S to CPU directory
  2003-05-06 14:00       ` Wolfgang Denk
@ 2003-05-06 14:16         ` Kyle Harris
  0 siblings, 0 replies; 7+ messages in thread
From: Kyle Harris @ 2003-05-06 14:16 UTC (permalink / raw)
  To: u-boot

On Tuesday 06 May 2003 10:00 am, Wolfgang Denk wrote:

> In message <E19D26b-0003S5-00@smtp10.atl.mindspring.net> you wrote:
> > Actually, the mem controller is completely different. The IXP has
> > separate SDRAM and expansion buses. Most other parts of cpu/xscale will
> > be different as well (e.g., interrupt.c, serial.c). It may be that we
> > need to have ixp and pxa subdirs. We can leave xscale (really pxa) as is,
> > move memsetup there, and create a new ixp.
>
> Then let's rename xscale ==> pxa and add a new ixp directory
>

Yes, renaming xscale to pxa is the correct thing to do. It will be 2-3 weeks 
before I start the ixp port, though.

Kyle.

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2003-05-06 14:16 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-05-06  7:26 [U-Boot-Users] [RFD] move memsetup.S to CPU directory Robert Schwebel
2003-05-06  7:44 ` Wolfgang Denk
2003-05-06 12:19 ` Kyle Harris
2003-05-06 12:35   ` Robert Schwebel
2003-05-06 12:55     ` Kyle Harris
2003-05-06 14:00       ` Wolfgang Denk
2003-05-06 14:16         ` Kyle Harris

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox