* [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
@ 2009-11-16 17:36 Curtis Wald
2009-11-30 13:41 ` Josh Boyer
0 siblings, 1 reply; 5+ messages in thread
From: Curtis Wald @ 2009-11-16 17:36 UTC (permalink / raw)
To: jwboyer, mporter; +Cc: linuxppc-dev
Yosemite.dts stanza for PCI was copied from Bamboo which has four PCI
slots; Yosemite only has one PCI slot which is mapped to
IDSEL 12, ADDR 22, IRQ2 Vector 25, INTA.
Signed-off-by: Curtis Wald <cwald@watchguardvideo.com>
---
arch/powerpc/boot/dts/yosemite.dts | 17 +++++------------
1 files changed, 5 insertions(+), 12 deletions(-)
diff --git a/arch/powerpc/boot/dts/yosemite.dts
b/arch/powerpc/boot/dts/yosemite.dts
index 1fa3cb4..8328be5 100644
--- a/arch/powerpc/boot/dts/yosemite.dts
+++ b/arch/powerpc/boot/dts/yosemite.dts
@@ -276,26 +276,19 @@
* later cannot be changed. Chip supports a
second
* IO range but we don't use it for now
*/
+
+ /* u32 u64
u64 u64 */
+ /* pci_space pci_addr
cpu_addr size */
ranges =3D <0x02000000 0x00000000 0xa0000000
0x00000000 0xa0000000 0x00000000 0x20000000
0x01000000 0x00000000 0x00000000
0x00000000 0xe8000000 0x00000000 0x00010000>;
=20
/* Inbound 2GB range starting at 0 */
dma-ranges =3D <0x42000000 0x0 0x0 0x0 0x0 0x0
0x80000000>;
=20
- /* Bamboo has all 4 IRQ pins tied together per
slot */
interrupt-map-mask =3D <0xf800 0x0 0x0 0x0>;
interrupt-map =3D <
- /* 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, ADDR 22, INTA, IRQ2 =3D
Vector 25, 0x19 */
+ 0x6000 0x0 0x0 0x0 &UIC0 0x19 0x8
>;
};
};
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
2009-11-16 17:36 [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS Curtis Wald
@ 2009-11-30 13:41 ` Josh Boyer
2009-11-30 15:25 ` Curtis Wald
0 siblings, 1 reply; 5+ messages in thread
From: Josh Boyer @ 2009-11-30 13:41 UTC (permalink / raw)
To: Curtis Wald; +Cc: linuxppc-dev
On Mon, Nov 16, 2009 at 11:36:40AM -0600, Curtis Wald wrote:
>Yosemite.dts stanza for PCI was copied from Bamboo which has four PCI
>slots; Yosemite only has one PCI slot which is mapped to
>IDSEL 12, ADDR 22, IRQ2 Vector 25, INTA.
>
>Signed-off-by: Curtis Wald <cwald@watchguardvideo.com>
Your patch looks correct (I think), but is very badly word wrapped and
whitespace damaged. Could you resend?
josh
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
2009-11-30 13:41 ` Josh Boyer
@ 2009-11-30 15:25 ` Curtis Wald
2009-12-04 12:07 ` Josh Boyer
0 siblings, 1 reply; 5+ messages in thread
From: Curtis Wald @ 2009-11-30 15:25 UTC (permalink / raw)
To: Josh Boyer; +Cc: linuxppc-dev
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.=20
The comments added before the ranges declaration provides info on the
array element as processed by the kernel - there is no runtime
difference for this change, but adds some background. Since the ranges
declaration is beyond 80 columns the comments added are lined up with
the entries and when viewed as 80 column will wrap. If you have
suggestions on correcting this I will implement.=20
Thanks,
-Curtis
The stanza for PCI was copied from Bamboo which has four PCI
slots; Yosemite only has one PCI slot which is mapped to
IDSEL 12, ADDR 22, IRQ2 Vector 25, INTA.
Signed-off-by: Curtis Wald <cwald@watchguardvideo.com>
---
arch/powerpc/boot/dts/yosemite.dts | 17 +++++------------
1 files changed, 5 insertions(+), 12 deletions(-)
diff --git a/arch/powerpc/boot/dts/yosemite.dts
b/arch/powerpc/boot/dts/yosemite.dts
index 1fa3cb4..8328be5 100644
--- a/arch/powerpc/boot/dts/yosemite.dts
+++ b/arch/powerpc/boot/dts/yosemite.dts
@@ -276,27 +276,19 @@
* later cannot be changed. Chip supports a second
* IO range but we don't use it for now
*/
+
+ /* u32 u64 u64
u64 */
+ /* pci_space pci_addr cpu_addr
size */
ranges =3D <0x02000000 0x00000000 0xa0000000 0x00000000 0xa0000000
0x00000000 0x20000000
0x01000000 0x00000000 0x00000000 0x00000000 0xe8000000
0x00000000 0x00010000>;
=20
/* Inbound 2GB range starting at 0 */
dma-ranges =3D <0x42000000 0x0 0x0 0x0 0x0 0x0 0x80000000>;
=20
- /* Bamboo has all 4 IRQ pins tied together per slot */
interrupt-map-mask =3D <0xf800 0x0 0x0 0x0>;
interrupt-map =3D <
- /* 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, ADDR 22, INTA, IRQ2 =3D Vector 25, 0x19 */
+ 0x6000 0x0 0x0 0x0 &UIC0 0x19 0x8
>;
};
};
--=20
1.5.5.6
> -----Original Message-----
> From: Josh Boyer [mailto:jwboyer@linux.vnet.ibm.com]
> Sent: Monday, November 30, 2009 7:42 AM
> To: Curtis Wald
> Cc: mporter@kernel.crashing.org; linuxppc-dev@ozlabs.org
> Subject: Re: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
>=20
> On Mon, Nov 16, 2009 at 11:36:40AM -0600, Curtis Wald wrote:
> >Yosemite.dts stanza for PCI was copied from Bamboo which has four PCI
> >slots; Yosemite only has one PCI slot which is mapped to
> >IDSEL 12, ADDR 22, IRQ2 Vector 25, INTA.
> >
> >Signed-off-by: Curtis Wald <cwald@watchguardvideo.com>
>=20
> Your patch looks correct (I think), but is very badly word wrapped and
> whitespace damaged. Could you resend?
>=20
> josh
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
2009-11-30 15:25 ` Curtis Wald
@ 2009-12-04 12:07 ` Josh Boyer
2009-12-04 14:19 ` Curtis Wald
0 siblings, 1 reply; 5+ messages in thread
From: Josh Boyer @ 2009-12-04 12:07 UTC (permalink / raw)
To: Curtis Wald; +Cc: linuxppc-dev
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
>;
};
};
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
2009-12-04 12:07 ` Josh Boyer
@ 2009-12-04 14:19 ` Curtis Wald
0 siblings, 0 replies; 5+ messages in thread
From: Curtis Wald @ 2009-12-04 14:19 UTC (permalink / raw)
To: Josh Boyer; +Cc: linuxppc-dev
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@linux.vnet.ibm.com]
> Sent: Friday, December 04, 2009 6:08 AM
> To: Curtis Wald
> Cc: mporter@kernel.crashing.org; linuxppc-dev@ozlabs.org
> Subject: Re: [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS
>=20
> 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.
>=20
> 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:
>=20
> http://patchwork.ozlabs.org/patch/39810/
>=20
> 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.
>=20
> josh
>=20
> ---
>=20
> 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 =3D <0x42000000 0x0 0x0 0x0 0x0 0x0
> 0x80000000>;
>=20
> - /* Bamboo has all 4 IRQ pins tied together per
slot
> */
> interrupt-map-mask =3D <0xf800 0x0 0x0 0x0>;
> interrupt-map =3D <
> - /* 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
> >;
> };
> };
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-12-04 14:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-16 17:36 [PATCH] POWERPC 4xx: Fix PCI in AMCC 440EP Yosemite DTS Curtis Wald
2009-11-30 13:41 ` Josh Boyer
2009-11-30 15:25 ` Curtis Wald
2009-12-04 12:07 ` Josh Boyer
2009-12-04 14:19 ` Curtis Wald
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox