[RESEND,v3,1/3] ARM: dts: DRA7: Add "linux,pci-domain" property for pci dt nodes
diff mbox

Message ID 1470832400-11929-2-git-send-email-kishon@ti.com
State New
Headers show

Commit Message

Kishon Vijay Abraham I Aug. 10, 2016, 12:33 p.m. UTC
Since DRA7 has multiple PCIe Rootcomplex, add "linux,pci-domain"
property to assign a PCI domain number to each of the host
bridges.

Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
---
 arch/arm/boot/dts/dra7.dtsi |    2 ++
 1 file changed, 2 insertions(+)

Comments

Tony Lindgren Aug. 26, 2016, 3:30 p.m. UTC | #1
* Kishon Vijay Abraham I <kishon@ti.com> [160810 05:34]:
> Since DRA7 has multiple PCIe Rootcomplex, add "linux,pci-domain"
> property to assign a PCI domain number to each of the host
> bridges.

Is this patch safe to apply without the other changes?

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kishon Vijay Abraham I Aug. 30, 2016, 11:05 a.m. UTC | #2
Hi Tony,

On Friday 26 August 2016 09:00 PM, Tony Lindgren wrote:
> * Kishon Vijay Abraham I <kishon@ti.com> [160810 05:34]:
>> Since DRA7 has multiple PCIe Rootcomplex, add "linux,pci-domain"
>> property to assign a PCI domain number to each of the host
>> bridges.
> 
> Is this patch safe to apply without the other changes?

Yes. I verified pcie on dra7x by applying only this patch and it doesn't break
anything.

Thanks
Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tony Lindgren Aug. 30, 2016, 6:47 p.m. UTC | #3
* Kishon Vijay Abraham I <kishon@ti.com> [160830 04:06]:
> Hi Tony,
> 
> On Friday 26 August 2016 09:00 PM, Tony Lindgren wrote:
> > * Kishon Vijay Abraham I <kishon@ti.com> [160810 05:34]:
> >> Since DRA7 has multiple PCIe Rootcomplex, add "linux,pci-domain"
> >> property to assign a PCI domain number to each of the host
> >> bridges.
> > 
> > Is this patch safe to apply without the other changes?
> 
> Yes. I verified pcie on dra7x by applying only this patch and it doesn't break
> anything.

OK thanks for testing, I'm picking the dts change into omap-for-v4.9/dt
branch.

The other patches you probably should upload to RMKs patch system
if no more comments.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch
diff mbox

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index d9bfb94..8a45385 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -301,6 +301,7 @@ 
 					  0x82000000 0 0x20013000 0x13000 0 0xffed000>;
 				#interrupt-cells = <1>;
 				num-lanes = <1>;
+				linux,pci-domain = <0>;
 				ti,hwmods = "pcie1";
 				phys = <&pcie1_phy>;
 				phy-names = "pcie-phy0";
@@ -336,6 +337,7 @@ 
 					  0x82000000 0 0x30013000 0x13000 0 0xffed000>;
 				#interrupt-cells = <1>;
 				num-lanes = <1>;
+				linux,pci-domain = <1>;
 				ti,hwmods = "pcie2";
 				phys = <&pcie2_phy>;
 				phy-names = "pcie-phy0";