From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 27 Aug 2007 11:54:17 +1000 From: David Gibson To: Vitaly Bordug Subject: Re: [PATCH 2/3] [POWERPC] Add pci node to sequoia dts Message-ID: <20070827015417.GB12804@localhost.localdomain> References: <20070825091440.4087.1428.stgit@localhost.localdomain> <20070825092954.4087.95333.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20070825092954.4087.95333.stgit@localhost.localdomain> Cc: linuxppc-dev , Stefan Roese List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, Aug 25, 2007 at 01:29:54PM +0400, Vitaly Bordug wrote: > > Signed-off-by: Vitaly Bordug > Signed-off-by: Stefan Roese > > --- > > arch/powerpc/boot/dts/sequoia.dts | 26 ++++++++++++++++++++++++++ > 1 files changed, 26 insertions(+), 0 deletions(-) > > diff --git a/arch/powerpc/boot/dts/sequoia.dts b/arch/powerpc/boot/dts/sequoia.dts > index ef6f41c..8eb258f 100644 > --- a/arch/powerpc/boot/dts/sequoia.dts > +++ b/arch/powerpc/boot/dts/sequoia.dts > @@ -92,6 +92,32 @@ > #size-cells = <1>; > ranges; > clock-frequency = <0>; /* Filled in by zImage */ > + > + pci { > + /* irqs are routed to irq67, dependless of devsel/PIRQx */ > + interrupt-map-mask = <0 0 0 0>; > + interrupt-map = <0 0 0 0 &UIC2 3 8>; > + > + interrupt-parent = <&UIC2>; > + interrupts = <3 8>; > + > + bus-range = <0 0>; > + > + /* > + * mem is at 80000000 set up indirectly > + * io is at 0001_e800_0000 > + */ > + ranges = <02000000 0 80000000 1 80000000 0 10000000 > + 01000000 0 00000000 1 e8000000 0 00100000>; > + > + #interrupt-cells = <1>; > + #size-cells = <2>; > + #address-cells = <3>; > + > + reg = <1 eec00000 40 1 ef400000 40>; /* phb cfg, phb reg */ > + compatible = "ibm, 440epx"; > + device_type = "pci"; I usually put device_type, compatible and reg at the top of the block, to announce what the node actually is before giving all the details. Also, apart from the stray space in the compatible, I'm guessing that the 440EPx bridge is actually more-or-less like the PCI bridges on other 4xx chips, so we should have a more general compatible string too. Is the 440EPx a vanilla PCI or a PCI-X bridge? If the later that should be reflected in the name and compatible as well. > + }; > > SDRAM0: sdram { > device_type = "memory-controller"; > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@ozlabs.org > https://ozlabs.org/mailman/listinfo/linuxppc-dev > -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson