* [PATCH V4 3/3] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC
@ 2013-06-12 10:20 Jingoo Han
2013-06-12 10:55 ` Arnd Bergmann
0 siblings, 1 reply; 3+ messages in thread
From: Jingoo Han @ 2013-06-12 10:20 UTC (permalink / raw)
To: 'Kukjin Kim', 'Bjorn Helgaas'
Cc: linux-samsung-soc, linux-pci, devicetree-discuss,
linux-arm-kernel, linux-kernel, 'Grant Likely',
'Andrew Murray', 'Thomas Petazzoni',
'Thierry Reding', 'Jason Gunthorpe',
'Arnd Bergmann', 'Surendranath Gurivireddy Balla',
'Siva Reddy Kallam', 'Thomas Abraham', Jingoo Han
Exynos5440 has two PCIe controllers which can be used as root complex
for PCIe interface.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
Tested on Exynos5440.
Changes since v3:
- Removed 'bus-range' property from DT
- Added 'interrupt-map-mask', 'interrupt-map' properties to DT
- Fixed the start address of MEM space in DT
- Increased the size of I/O space to 64kB in DT
- Added 'clocks', 'clock-names' properties to DT
arch/arm/boot/dts/exynos5440-ssdk5440.dts | 8 ++++++
arch/arm/boot/dts/exynos5440.dtsi | 38 +++++++++++++++++++++++++++++
2 files changed, 46 insertions(+)
diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
index d55042b..efe7d39 100644
--- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
+++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
@@ -30,4 +30,12 @@
clock-frequency = <50000000>;
};
};
+
+ pcie0@40000000 {
+ reset-gpio = <5>;
+ };
+
+ pcie1@60000000 {
+ reset-gpio = <22>;
+ };
};
diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
index f6b1c89..2c15f9d 100644
--- a/arch/arm/boot/dts/exynos5440.dtsi
+++ b/arch/arm/boot/dts/exynos5440.dtsi
@@ -216,4 +216,42 @@
clock-names = "rtc";
status = "disabled";
};
+
+ pcie0@0x290000 {
+ compatible = "samsung,exynos5440-pcie";
+ reg = <0x290000 0x1000
+ 0x270000 0x1000
+ 0x271000 0x40>;
+ interrupts = <0 20 0>, <0 21 0>, <0 22 0>;
+ clocks = <&clock 28>, <&clock 27>;
+ clock-names = "pcie", "pcie_bus";
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ ranges = <0x00000800 0 0x40000000 0x40000000 0 0x00200000 /* configuration space */
+ 0x81000000 0 0 0x40200000 0 0x00010000 /* downstream I/O */
+ 0x82000000 0 0x40210000 0x40210000 0 0x10000000>; /* non-prefetchable memory */
+ #interrupt-cells = <1>;
+ interrupt-map-mask = <0 0 0 0>;
+ interrupt-map = <0x0 0 &gic 53>;
+ };
+
+ pcie1@2a0000 {
+ compatible = "samsung,exynos5440-pcie";
+ reg = <0x2a0000 0x1000
+ 0x272000 0x1000
+ 0x271040 0x40>;
+ interrupts = <0 23 0>, <0 24 0>, <0 25 0>;
+ clocks = <&clock 29>, <&clock 27>;
+ clock-names = "pcie", "pcie_bus";
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ ranges = <0x00000800 0 0x60000000 0x60000000 0 0x00200000 /* configuration space */
+ 0x81000000 0 0 0x60200000 0 0x00010000 /* downstream I/O */
+ 0x82000000 0 0x60210000 0x60210000 0 0x10000000>; /* non-prefetchable memory */
+ #interrupt-cells = <1>;
+ interrupt-map-mask = <0 0 0 0>;
+ interrupt-map = <0x0 0 &gic 56>;
+ };
};
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH V4 3/3] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC
2013-06-12 10:20 [PATCH V4 3/3] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC Jingoo Han
@ 2013-06-12 10:55 ` Arnd Bergmann
2013-06-13 5:34 ` Jingoo Han
0 siblings, 1 reply; 3+ messages in thread
From: Arnd Bergmann @ 2013-06-12 10:55 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Jingoo Han, 'Kukjin Kim', 'Bjorn Helgaas',
'Thomas Petazzoni', 'Jason Gunthorpe',
linux-samsung-soc, 'Siva Reddy Kallam', linux-pci,
devicetree-discuss, 'Thierry Reding', linux-kernel,
'Grant Likely', 'Surendranath Gurivireddy Balla',
'Thomas Abraham', 'Andrew Murray'
Thanks for the update! A few comments again:
On Wednesday 12 June 2013 19:20:00 Jingoo Han wrote:
>
> diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> index d55042b..efe7d39 100644
> --- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> +++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> @@ -30,4 +30,12 @@
> clock-frequency = <50000000>;
> };
> };
> +
> + pcie0@40000000 {
> + reset-gpio = <5>;
> + };
> +
> + pcie1@60000000 {
> + reset-gpio = <22>;
> + };
> };
As mentioned before, please use the gpio binding to pass gpio numbers.
> diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
> index f6b1c89..2c15f9d 100644
> --- a/arch/arm/boot/dts/exynos5440.dtsi
> +++ b/arch/arm/boot/dts/exynos5440.dtsi
> @@ -216,4 +216,42 @@
> clock-names = "rtc";
> status = "disabled";
> };
> +
> + pcie0@0x290000 {
> + compatible = "samsung,exynos5440-pcie";
> + reg = <0x290000 0x1000
> + 0x270000 0x1000
> + 0x271000 0x40>;
> + interrupts = <0 20 0>, <0 21 0>, <0 22 0>;
> + clocks = <&clock 28>, <&clock 27>;
> + clock-names = "pcie", "pcie_bus";
> + #address-cells = >;
> + #size-cells = <2>;
> + device_type = "pci";
> + ranges = <0x00000800 0 0x40000000 0x40000000 0 0x00200000 /* configuration space */
> + 0x81000000 0 0 0x40200000 0 0x00010000 /* downstream I/O */
> + 0x82000000 0 0x40210000 0x40210000 0 0x10000000>; /* non-prefetchable memory */
I think you did not reply to my question regarding the size of the
memory space. Does it extend from 0x40210000 to 0x50210000,
or from 0x40210000 to 0x50000000. You probably meant the latter
but wrote the former. If not, please add a comment for clarification.
> + #interrupt-cells = <1>;
> + interrupt-map-mask = <0 0 0 0>;
> + interrupt-map = <0x0 0 &gic 53>;
> + };
So all PCI IntA interrupts are mapped to a single gic interrupt? That
sounds like a bottleneck when you have a lot of devices on the bus.
Do you have MSI support?
Arnd
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH V4 3/3] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC
2013-06-12 10:55 ` Arnd Bergmann
@ 2013-06-13 5:34 ` Jingoo Han
0 siblings, 0 replies; 3+ messages in thread
From: Jingoo Han @ 2013-06-13 5:34 UTC (permalink / raw)
To: 'Arnd Bergmann', linux-arm-kernel
Cc: 'Kukjin Kim', 'Bjorn Helgaas',
'Thomas Petazzoni', 'Jason Gunthorpe',
linux-samsung-soc, 'Siva Reddy Kallam', linux-pci,
devicetree-discuss, 'Thierry Reding', linux-kernel,
'Grant Likely', 'Surendranath Gurivireddy Balla',
'Thomas Abraham', 'Andrew Murray', Jingoo Han
On Wednesday, June 12, 2013 7:56 PM, Arnd Bergmann wrote:
>
> Thanks for the update! A few comments again:
>
> On Wednesday 12 June 2013 19:20:00 Jingoo Han wrote:
> >
> > diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> > index d55042b..efe7d39 100644
> > --- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> > +++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> > @@ -30,4 +30,12 @@
> > clock-frequency = <50000000>;
> > };
> > };
> > +
> > + pcie0@40000000 {
> > + reset-gpio = <5>;
> > + };
> > +
> > + pcie1@60000000 {
> > + reset-gpio = <22>;
> > + };
> > };
>
> As mentioned before, please use the gpio binding to pass gpio numbers.
OK, I will use gpio binding.
>
> > diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
> > index f6b1c89..2c15f9d 100644
> > --- a/arch/arm/boot/dts/exynos5440.dtsi
> > +++ b/arch/arm/boot/dts/exynos5440.dtsi
> > @@ -216,4 +216,42 @@
> > clock-names = "rtc";
> > status = "disabled";
> > };
> > +
> > + pcie0@0x290000 {
> > + compatible = "samsung,exynos5440-pcie";
> > + reg = <0x290000 0x1000
> > + 0x270000 0x1000
> > + 0x271000 0x40>;
> > + interrupts = <0 20 0>, <0 21 0>, <0 22 0>;
> > + clocks = <&clock 28>, <&clock 27>;
> > + clock-names = "pcie", "pcie_bus";
> > + #address-cells = >;
> > + #size-cells = <2>;
> > + device_type = "pci";
> > + ranges = <0x00000800 0 0x40000000 0x40000000 0 0x00200000 /* configuration space */
> > + 0x81000000 0 0 0x40200000 0 0x00010000 /* downstream I/O */
> > + 0x82000000 0 0x40210000 0x40210000 0 0x10000000>; /* non-prefetchable memory */
>
> I think you did not reply to my question regarding the size of the
> memory space. Does it extend from 0x40210000 to 0x50210000,
> or from 0x40210000 to 0x50000000. You probably meant the latter
> but wrote the former. If not, please add a comment for clarification.
OK, I see.
It extends to 0x60000000. I will modify it.
>
> > + #interrupt-cells = <1>;
> > + interrupt-map-mask = <0 0 0 0>;
> > + interrupt-map = <0x0 0 &gic 53>;
> > + };
>
> So all PCI IntA interrupts are mapped to a single gic interrupt? That
> sounds like a bottleneck when you have a lot of devices on the bus.
> Do you have MSI support?
INTA, INTB, INTC, and INTD are mapped to a single gic interrupt.
Exynos5440 PCIe has MSI support; however, MSI support patch will
be posted later.
>
> Arnd
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-13 5:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-12 10:20 [PATCH V4 3/3] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC Jingoo Han
2013-06-12 10:55 ` Arnd Bergmann
2013-06-13 5:34 ` Jingoo Han
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox