On Tue, Sep 23, 2003 at 02:12:11AM +0900, Shigeyuki Fukushima wrote:
> From: Eduardo Horvath <eeh@NetBSD.ORG>
> Subject: Re: Repo-copy request: evbppc port for OpenBlockS266
> Date: Mon, 22 Sep 2003 12:50:04 -0400
> 
> Hi, Eduardo!
> Thank you for your advice.
> 
> > > machdep.c has initppc() function.
> > > common parts(codes) are included in ibm4xx_machdep.c.
> > 
> > You probably want to separate out the 405-specific parts
> > from the bits that are common between 401, 403, and 405
> > cores.  We will also have 440-specific bits at some point,
> > so I'd think this file should be named ibm40x_machdep.c
> > to distinguish it from the 440 stuff.
> 
> Ok. it should be named ibm40x_machdep.c.
> Is it better that ibm4xx_autoconf.c should also be named
> ibm40x_autoconf.c?

The walnut/autoconf.c file explicitly tweaks implementation
DCRs (CPC0_CR1 to be precise) which does exist on the 440GP.
However, these processors are also sold as cores only, so
I'm not certain how relevent this operation is to non-GP
parts.  Maybe you should call this file ibm4xxGPx_autoconf.c?

> > > obs405_start.S is not moved to common directory.
> > > This is future works too.
> > Some of the code in the walnut_start.S (which should be
> > renamed locore.S the way it was originally) and 
> > walnut/machdep.c is there to import board strapping
> > information from OpenBIOS.  That should also be factored
> > out if possible.  BTW, do the OpenBlockS machines use
> > OpenBIOS or something compatible?
> 
> Yes. At least, OpenBlockS266 use OpenBIOS.
> But I don't know about other OpenBlockS products.

Hm...  It might make sense to create a 
sys/arch/powerpc/ibm4xx/openbios directory and move the
files over there.

Eduardo