Re: IBM405GP/GPr OPB bus_space endian (powerpc/ibm4xx/dev/opb.c)

[Shigeyuki Fukushima <shige%netbsd.org@localhost>]

Simon Burge wrote:
>>> Other 405 processors have GPIO as well (I just checked the 405CR and
>>> 405EP datasheets).
>> Hmm, then opb (On-chip Peripheral Bus) is better?
> opbgpio works for me.

I agree it.

>>> I'd suggest all of files.ibm405gp should really be
>>> in files.ibm4xx, and any gp prefixs be renamed.  I can't think of what
>>> to rename them too right now though :-/
>> I agree that files.ibm405gp should be merged into files.ibm4xx,
>> but it may require:
>> - options IBM405GP (or something) is needed to specify ibm405gp.c
>>   (or should it be renamed to obppci.c??)
> 
> PCI doesn't need to be 405gp specific.  It attaches to the PLB
> (processor local bus), so plbpci.c would be better.

ibm4xx/pci/pchb.c + ibm4xx/dev/ibm405gp.c = plbpci.c ?

And...
Two functions in ibm4xx/pci/pci_machdep.c
(pci_intr_map()/pci_conf_interrupt()) are machine-dependent.
We should move them to evbppc/<board>/pci_machdep.c.
I think that other functions to powerpc/dev/plbpci.c. Is it ok?

>> - declaration of rtc.c should be conditional by kernel config
> There's still some cleanup to be done here.  Note that the first 405gp
> port (walnut) doesn't actually use files.ibm405gp(!).  I don't think
> that issue is going to be hard to fix.

OpenBlockS266 port use files.ibm405gp, but perhaps it isn't hard to fix.


There's still 405GP-specific part (opb_tag declaration) in
sys/arch/powerpc/ibm4xx/dev/opb.c.

-- 
Kind Regards,
--- shige
Shigeyuki Fukushima <shige@{FreeBSD,jp.FreeBSD,NetBSD}.org>