* [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds @ 2015-10-13 11:29 Zhiqiang Hou 2015-10-13 11:29 ` [PATCH 2/2] powerpc/85xx: Add PCIe controller support for bsc9132qds Zhiqiang Hou 2015-10-14 20:36 ` [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Scott Wood 0 siblings, 2 replies; 4+ messages in thread From: Zhiqiang Hou @ 2015-10-13 11:29 UTC (permalink / raw) To: linuxppc-dev, scottwood, galak, benh, paulus, mpe, devicetree, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, harninder.rai Cc: Minghuan.Lian, Mingkai.Hu From: Harninder Rai <harninder.rai@freescale.com> Signed-off-by: Harninder Rai <harninder.rai@freescale.com> Signed-off-by: Minghuan Lian <Minghuan.Lian@freescale.com> Change-Id: I4355add4a92d1fcf514843aea5ecadd2e2517969 Reviewed-on: http://git.am.freescale.net:8181/2454 Reviewed-by: Zang Tiefei-R61911 <tie-fei.zang@freescale.com> Reviewed-by: Kushwaha Prabhakar-B32579 <prabhakar@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> --- arch/powerpc/boot/dts/bsc9132qds.dts | 15 ++++++++++++++ arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi | 29 +++++++++++++++++++++++++++ arch/powerpc/boot/dts/fsl/bsc9132si-pre.dtsi | 1 + 3 files changed, 45 insertions(+) diff --git a/arch/powerpc/boot/dts/bsc9132qds.dts b/arch/powerpc/boot/dts/bsc9132qds.dts index 6cab106..940d719 100644 --- a/arch/powerpc/boot/dts/bsc9132qds.dts +++ b/arch/powerpc/boot/dts/bsc9132qds.dts @@ -29,6 +29,21 @@ soc: soc@ff700000 { ranges = <0x0 0x0 0xff700000 0x100000>; }; + + pci0: pcie@ff70a000 { + reg = <0 0xff70a000 0 0x1000>; + ranges = <0x2000000 0x0 0x90000000 0 0x90000000 0x0 0x20000000 + 0x1000000 0x0 0x00000000 0 0xc0010000 0x0 0x10000>; + pcie@0 { + ranges = <0x2000000 0x0 0x90000000 + 0x2000000 0x0 0x90000000 + 0x0 0x20000000 + + 0x1000000 0x0 0x0 + 0x1000000 0x0 0x0 + 0x0 0x100000>; + }; + }; }; /include/ "bsc9132qds.dtsi" diff --git a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi index c723071..78c8f1c 100644 --- a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi @@ -40,6 +40,35 @@ interrupts = <16 2 0 0 20 2 0 0>; }; +/* controller at 0xa000 */ +&pci0 { + compatible = "fsl,bsc9132-pcie", "fsl,qoriq-pcie-v2.2"; + device_type = "pci"; + #size-cells = <2>; + #address-cells = <3>; + bus-range = <0 255>; + clock-frequency = <33333333>; + interrupts = <16 2 0 0>; + + pcie@0 { + reg = <0 0 0 0 0>; + #interrupt-cells = <1>; + #size-cells = <2>; + #address-cells = <3>; + device_type = "pci"; + interrupts = <16 2 0 0>; + interrupt-map-mask = <0xf800 0 0 7>; + + interrupt-map = < + /* IDSEL 0x0 */ + 0000 0x0 0x0 0x1 &mpic 0x0 0x2 0x0 0x0 + 0000 0x0 0x0 0x2 &mpic 0x1 0x2 0x0 0x0 + 0000 0x0 0x0 0x3 &mpic 0x2 0x2 0x0 0x0 + 0000 0x0 0x0 0x4 &mpic 0x3 0x2 0x0 0x0 + >; + }; +}; + &soc { #address-cells = <1>; #size-cells = <1>; diff --git a/arch/powerpc/boot/dts/fsl/bsc9132si-pre.dtsi b/arch/powerpc/boot/dts/fsl/bsc9132si-pre.dtsi index 301a9db..90f7949 100644 --- a/arch/powerpc/boot/dts/fsl/bsc9132si-pre.dtsi +++ b/arch/powerpc/boot/dts/fsl/bsc9132si-pre.dtsi @@ -45,6 +45,7 @@ serial0 = &serial0; ethernet0 = &enet0; ethernet1 = &enet1; + pci0 = &pci0; }; cpus { -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 2/2] powerpc/85xx: Add PCIe controller support for bsc9132qds 2015-10-13 11:29 [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Zhiqiang Hou @ 2015-10-13 11:29 ` Zhiqiang Hou 2015-10-14 20:36 ` [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Scott Wood 1 sibling, 0 replies; 4+ messages in thread From: Zhiqiang Hou @ 2015-10-13 11:29 UTC (permalink / raw) To: linuxppc-dev, scottwood, galak, benh, paulus, mpe, devicetree, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, harninder.rai Cc: Minghuan.Lian, Mingkai.Hu From: Harninder Rai <harninder.rai@freescale.com> 1. Use machine_arch_initcall to hook mpc85xx_common_publish_devices This can ensure before pcibios_init() is called, pci controllers have been probed and added to the hose_list. 2. Add a workaround for errata A-005434 For the BSC9132, PEX_PEXIWARn[TRGT] for all windows defaults to 0xF, which is mapped to CCSRBAR. However, for other products, 0xF is mapped to the local memory. Therefore, for the BSC9132, any default PCI Express access to the local memory (DDR) will now access the CCSRBAR. This patch changes the mapping of targets of inbound windows PEX_PEXIWARn[TRGT] to the Local address space – 0x0 (from 0xF). Signed-off-by: Harninder Rai <harninder.rai@freescale.com> Signed-off-by: Minghuan Lian <Minghuan.Lian@freescale.com> Change-Id: Iebb3d0e057d64d5b96263c130495e5d04caf7948 Reviewed-on: http://git.am.freescale.net:8181/2455 Reviewed-by: Zang Tiefei-R61911 <tie-fei.zang@freescale.com> Reviewed-by: Kushwaha Prabhakar-B32579 <prabhakar@freescale.com> Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> --- arch/powerpc/platforms/85xx/bsc913x_qds.c | 8 +++++++- arch/powerpc/sysdev/fsl_pci.c | 13 +++++++++++++ 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/85xx/bsc913x_qds.c b/arch/powerpc/platforms/85xx/bsc913x_qds.c index f0927e5..dcfafd6 100644 --- a/arch/powerpc/platforms/85xx/bsc913x_qds.c +++ b/arch/powerpc/platforms/85xx/bsc913x_qds.c @@ -17,6 +17,7 @@ #include <linux/pci.h> #include <asm/mpic.h> #include <sysdev/fsl_soc.h> +#include <sysdev/fsl_pci.h> #include <asm/udbg.h> #include "mpc85xx.h" @@ -46,10 +47,12 @@ static void __init bsc913x_qds_setup_arch(void) mpc85xx_smp_init(); #endif + fsl_pci_assign_primary(); + pr_info("bsc913x board from Freescale Semiconductor\n"); } -machine_device_initcall(bsc9132_qds, mpc85xx_common_publish_devices); +machine_arch_initcall(bsc9132_qds, mpc85xx_common_publish_devices); /* * Called very early, device-tree isn't unflattened @@ -67,6 +70,9 @@ define_machine(bsc9132_qds) { .probe = bsc9132_qds_probe, .setup_arch = bsc913x_qds_setup_arch, .init_IRQ = bsc913x_qds_pic_init, +#ifdef CONFIG_PCI + .pcibios_fixup_bus = fsl_pcibios_fixup_bus, +#endif .get_irq = mpic_get_irq, .restart = fsl_rstcr_restart, .calibrate_decr = generic_calibrate_decr, diff --git a/arch/powerpc/sysdev/fsl_pci.c b/arch/powerpc/sysdev/fsl_pci.c index ebc1f412..b8607f6 100644 --- a/arch/powerpc/sysdev/fsl_pci.c +++ b/arch/powerpc/sysdev/fsl_pci.c @@ -193,6 +193,19 @@ static void setup_pci_atmu(struct pci_controller *hose) const u64 *reg; int len; + if (of_device_is_compatible(hose->dn, "fsl,bsc9132-pcie")) { + /* + * BSC9132 Rev1.0 has an issue where all the PEX inbound + * windows have implemented the default target value as 0xf + * for CCSR space.In all Freescale legacy devices the target + * of 0xf is reserved for local memory space. 9132 Rev1.0 + * now has local mempry space mapped to target 0x0 instead of + * 0xf. Hence adding a workaround to remove the target 0xf + * defined for memory space from Inbound window attributes. + */ + piwar &= ~PIWAR_TGI_LOCAL; + } + if (early_find_capability(hose, 0, 0, PCI_CAP_ID_EXP)) { if (in_be32(&pci->block_rev1) >= PCIE_IP_REV_2_2) { win_idx = 2; -- 2.1.0.27.g96db324 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds 2015-10-13 11:29 [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Zhiqiang Hou 2015-10-13 11:29 ` [PATCH 2/2] powerpc/85xx: Add PCIe controller support for bsc9132qds Zhiqiang Hou @ 2015-10-14 20:36 ` Scott Wood 2015-10-15 2:52 ` Hou Zhiqiang 1 sibling, 1 reply; 4+ messages in thread From: Scott Wood @ 2015-10-14 20:36 UTC (permalink / raw) To: Zhiqiang Hou Cc: linuxppc-dev, galak, benh, paulus, mpe, devicetree, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, harninder.rai, Minghuan.Lian, Mingkai.Hu On Tue, 2015-10-13 at 19:29 +0800, Zhiqiang Hou wrote: > From: Harninder Rai <harninder.rai@freescale.com> > > Signed-off-by: Harninder Rai <harninder.rai@freescale.com> > Signed-off-by: Minghuan Lian <Minghuan.Lian@freescale.com> > Change-Id: I4355add4a92d1fcf514843aea5ecadd2e2517969 > Reviewed-on: http://git.am.freescale.net:8181/2454 > Reviewed-by: Zang Tiefei-R61911 <tie-fei.zang@freescale.com> > Reviewed-by: Kushwaha Prabhakar-B32579 <prabhakar@freescale.com> > Reviewed-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> > Tested-by: Fleming Andrew-AFLEMING <AFLEMING@freescale.com> Get rid of the gerrit stuff. And where is your signoff? > diff --git a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi > b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi > index c723071..78c8f1c 100644 > --- a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi > +++ b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi > @@ -40,6 +40,35 @@ > interrupts = <16 2 0 0 20 2 0 0>; > }; > > +/* controller at 0xa000 */ > +&pci0 { > + compatible = "fsl,bsc9132-pcie", "fsl,qoriq-pcie-v2.2"; > + device_type = "pci"; > + #size-cells = <2>; > + #address-cells = <3>; > + bus-range = <0 255>; > + clock-frequency = <33333333>; > + interrupts = <16 2 0 0>; This clock-frequency is not correct for PCIe. Just remove it. -Scott ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds 2015-10-14 20:36 ` [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Scott Wood @ 2015-10-15 2:52 ` Hou Zhiqiang 0 siblings, 0 replies; 4+ messages in thread From: Hou Zhiqiang @ 2015-10-15 2:52 UTC (permalink / raw) To: Scott Wood Cc: linuxppc-dev@lists.ozlabs.org, galak@kernel.crashing.org, benh@kernel.crashing.org, paulus@samba.org, mpe@ellerman.id.au, devicetree@vger.kernel.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, Harninder Rai, Lian M.H., Hu Vincent DQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogV29vZCBTY290dC1CMDc0 MjENCj4gU2VudDogMjAxNeW5tDEw5pyIMTXml6UgNDozNg0KPiBUbzogSG91IFpoaXFpYW5nLUI0 ODI4Ng0KPiBDYzogbGludXhwcGMtZGV2QGxpc3RzLm96bGFicy5vcmc7IGdhbGFrQGtlcm5lbC5j cmFzaGluZy5vcmc7DQo+IGJlbmhAa2VybmVsLmNyYXNoaW5nLm9yZzsgcGF1bHVzQHNhbWJhLm9y ZzsgbXBlQGVsbGVybWFuLmlkLmF1Ow0KPiBkZXZpY2V0cmVlQHZnZXIua2VybmVsLm9yZzsgcm9i aCtkdEBrZXJuZWwub3JnOyBwYXdlbC5tb2xsQGFybS5jb207DQo+IG1hcmsucnV0bGFuZEBhcm0u Y29tOyBpamMrZGV2aWNldHJlZUBoZWxsaW9uLm9yZy51azsgUmFpIEhhcm5pbmRlci1CMDEwNDQ7 DQo+IExpYW4gTWluZ2h1YW4tQjMxOTM5OyBIdSBNaW5na2FpLUIyMTI4NA0KPiBTdWJqZWN0OiBS ZTogW1BBVENIIDEvMl0gcG93ZXJwYy9mc2w6IEFkZCBQQ0kgbm9kZSBpbiBkZXZpY2UgdHJlZSBv Zg0KPiBic2M5MTMycWRzDQo+IA0KPiBPbiBUdWUsIDIwMTUtMTAtMTMgYXQgMTk6MjkgKzA4MDAs IFpoaXFpYW5nIEhvdSB3cm90ZToNCj4gPiBGcm9tOiBIYXJuaW5kZXIgUmFpIDxoYXJuaW5kZXIu cmFpQGZyZWVzY2FsZS5jb20+DQo+ID4NCj4gPiBTaWduZWQtb2ZmLWJ5OiBIYXJuaW5kZXIgUmFp IDxoYXJuaW5kZXIucmFpQGZyZWVzY2FsZS5jb20+DQo+ID4gU2lnbmVkLW9mZi1ieTogTWluZ2h1 YW4gTGlhbiA8TWluZ2h1YW4uTGlhbkBmcmVlc2NhbGUuY29tPg0KPiA+IENoYW5nZS1JZDogSTQz NTVhZGQ0YTkyZDFmY2Y1MTQ4NDNhZWE1ZWNhZGQyZTI1MTc5NjkNCj4gPiBSZXZpZXdlZC1vbjog aHR0cDovL2dpdC5hbS5mcmVlc2NhbGUubmV0OjgxODEvMjQ1NA0KPiA+IFJldmlld2VkLWJ5OiBa YW5nIFRpZWZlaS1SNjE5MTEgPHRpZS1mZWkuemFuZ0BmcmVlc2NhbGUuY29tPg0KPiA+IFJldmll d2VkLWJ5OiBLdXNod2FoYSBQcmFiaGFrYXItQjMyNTc5IDxwcmFiaGFrYXJAZnJlZXNjYWxlLmNv bT4NCj4gPiBSZXZpZXdlZC1ieTogRmxlbWluZyBBbmRyZXctQUZMRU1JTkcgPEFGTEVNSU5HQGZy ZWVzY2FsZS5jb20+DQo+ID4gVGVzdGVkLWJ5OiBGbGVtaW5nIEFuZHJldy1BRkxFTUlORyA8QUZM RU1JTkdAZnJlZXNjYWxlLmNvbT4NCj4gDQo+IEdldCByaWQgb2YgdGhlIGdlcnJpdCBzdHVmZi4g IEFuZCB3aGVyZSBpcyB5b3VyIHNpZ25vZmY/DQo+IA0KPiA+IGRpZmYgLS1naXQgYS9hcmNoL3Bv d2VycGMvYm9vdC9kdHMvZnNsL2JzYzkxMzJzaS1wb3N0LmR0c2kNCj4gPiBiL2FyY2gvcG93ZXJw Yy9ib290L2R0cy9mc2wvYnNjOTEzMnNpLXBvc3QuZHRzaQ0KPiA+IGluZGV4IGM3MjMwNzEuLjc4 YzhmMWMgMTAwNjQ0DQo+ID4gLS0tIGEvYXJjaC9wb3dlcnBjL2Jvb3QvZHRzL2ZzbC9ic2M5MTMy c2ktcG9zdC5kdHNpDQo+ID4gKysrIGIvYXJjaC9wb3dlcnBjL2Jvb3QvZHRzL2ZzbC9ic2M5MTMy c2ktcG9zdC5kdHNpDQo+ID4gQEAgLTQwLDYgKzQwLDM1IEBADQo+ID4gICAgICAgaW50ZXJydXB0 cyA9IDwxNiAyIDAgMCAyMCAyIDAgMD47ICB9Ow0KPiA+DQo+ID4gKy8qIGNvbnRyb2xsZXIgYXQg MHhhMDAwICovDQo+ID4gKyZwY2kwIHsNCj4gPiArICAgICBjb21wYXRpYmxlID0gImZzbCxic2M5 MTMyLXBjaWUiLCAiZnNsLHFvcmlxLXBjaWUtdjIuMiI7DQo+ID4gKyAgICAgZGV2aWNlX3R5cGUg PSAicGNpIjsNCj4gPiArICAgICAjc2l6ZS1jZWxscyA9IDwyPjsNCj4gPiArICAgICAjYWRkcmVz cy1jZWxscyA9IDwzPjsNCj4gPiArICAgICBidXMtcmFuZ2UgPSA8MCAyNTU+Ow0KPiA+ICsgICAg IGNsb2NrLWZyZXF1ZW5jeSA9IDwzMzMzMzMzMz47DQo+ID4gKyAgICAgaW50ZXJydXB0cyA9IDwx NiAyIDAgMD47DQo+IA0KPiBUaGlzIGNsb2NrLWZyZXF1ZW5jeSBpcyBub3QgY29ycmVjdCBmb3Ig UENJZS4gIEp1c3QgcmVtb3ZlIGl0Lg0KPiANCg0KT2ssIHRoYW5rcyBmb3IgeW91ciBjb3JyZWN0 aW9uLg0KIA0KDQpUaGFua3MsDQpaaGlxaWFuZw0KDQo= ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-10-15 3:08 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-10-13 11:29 [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Zhiqiang Hou 2015-10-13 11:29 ` [PATCH 2/2] powerpc/85xx: Add PCIe controller support for bsc9132qds Zhiqiang Hou 2015-10-14 20:36 ` [PATCH 1/2] powerpc/fsl: Add PCI node in device tree of bsc9132qds Scott Wood 2015-10-15 2:52 ` Hou Zhiqiang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).