Subject: Re: [PATCH 6/7] [POWERPC] mpc8568mds.dts: fix PCI/PCIe nodes

From: Sergei Shtylyov <[email-masked]>
Date: Fri, 05 Oct 2007 21:56:46 +0400

Hello.

Anton Vorontsov wrote:

> Commit 5bece127f0666996ca90772229e00332a34e516c tried to fix
> PCI/PCIe nodes, but actually it broke them even harder. ;-)

    Of course. But shouldn't those be the subnoses of the "soc" type node?
I don't suppose the PCI controllers could hang off the root node... :-/

> Signed-off-by: Anton Vorontsov <[email-masked]>

> diff --git a/arch/powerpc/boot/dts/mpc8568mds.dts b/arch/powerpc/boot/dts/mpc8568mds.dts
> index b4aa5e7..5439437 100644
> --- a/arch/powerpc/boot/dts/mpc8568mds.dts
> +++ b/arch/powerpc/boot/dts/mpc8568mds.dts
> @@ -410,7 +410,7 @@
>  
>  	};
>  
> -	pci@8000 {
> +	pci@e0008000 {
>  		interrupt-map-mask = <f800 0 0 7>;
>  		interrupt-map = <
>  			/* IDSEL 0x12 AD18 */
> @@ -434,13 +434,13 @@
>  		#interrupt-cells = <1>;
>  		#size-cells = <2>;
>  		#address-cells = <3>;
> -		reg = <8000 1000>;
> +		reg = <e0008000 1000>;
>  		compatible = "fsl,mpc8540-pci";
>  		device_type = "pci";
>  	};
>  
>  	/* PCI Express */
> -	pcie@a000 {
> +	pcie@e000a000 {
>  		interrupt-map-mask = <f800 0 0 7>;
>  		interrupt-map = <
>  
> @@ -459,7 +459,7 @@
>  		#interrupt-cells = <1>;
>  		#size-cells = <2>;
>  		#address-cells = <3>;
> -		reg = <a000 1000>;
> +		reg = <e000a000 1000>;
>  		compatible = "fsl,mpc8548-pcie";
>  		device_type = "pci";
>  		pcie@0 {

WBR, Sergei