[PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS

[Curtis Wald]

Josh,
Yes, the patch you provided below is exactly what is needed for PCI
functionality.

I've verified using a 2 port Silicon Image 3512 PCI to SATA controller
card on Yosemite.

-Curtis


> -----Original Message-----
> From: Josh Boyer [mailto:jwboyer at linux.vnet.ibm.com]
> Sent: Friday, December 04, 2009 6:08 AM
> To: Curtis Wald
> Cc: mporter at kernel.crashing.org; linuxppc-dev at ozlabs.org
> Subject: Re: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
> 
> On Mon, Nov 30, 2009 at 09:25:51AM -0600, Curtis Wald wrote:
> >Josh,
> >Here is a resend of the Yosemite.dts patch, deleting tabs and spaces
> in
> >the IDSEL section that should look better when viewing as 80 column.
> 
> Something is still eating your patches when you send them out.  They
> get
> corrupted to the point of being unusable as you can see here:
> 
> http://patchwork.ozlabs.org/patch/39810/
> 
> However, your intentions are pretty clear.  Could you look at the
patch
> below
> and see if it was what you intended?  If so, I'll include it in my
> 'next'
> branch today.
> 
> josh
> 
> ---
> 
> diff --git a/arch/powerpc/boot/dts/yosemite.dts
> b/arch/powerpc/boot/dts/yosemite.dts
> index 1fa3cb4..6492324 100644
> --- a/arch/powerpc/boot/dts/yosemite.dts
> +++ b/arch/powerpc/boot/dts/yosemite.dts
> @@ -282,20 +282,10 @@
>  			/* Inbound 2GB range starting at 0 */
>  			dma-ranges = <0x42000000 0x0 0x0 0x0 0x0 0x0
> 0x80000000>;
> 
> -			/* Bamboo has all 4 IRQ pins tied together per
slot
> */
>  			interrupt-map-mask = <0xf800 0x0 0x0 0x0>;
>  			interrupt-map = <
> -				/* IDSEL 1 */
> -				0x800 0x0 0x0 0x0 &UIC0 0x1c 0x8
> -
> -				/* IDSEL 2 */
> -				0x1000 0x0 0x0 0x0 &UIC0 0x1b 0x8
> -
> -				/* IDSEL 3 */
> -				0x1800 0x0 0x0 0x0 &UIC0 0x1a 0x8
> -
> -				/* IDSEL 4 */
> -				0x2000 0x0 0x0 0x0 &UIC0 0x19 0x8
> +				/* IDSEL 12 */
> +				0x6000 0x0 0x0 0x0 &UIC0 0x19 0x8
>  			>;
>  		};
>  	};