| Message ID | 20200303105418.2840-1-vidyas@nvidia.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add support for PCIe endpoint mode in Tegra194 | expand |
On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: > Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can operate > either in root port mode or in end point mode but only in one mode at a time. > Platform P2972-0000 supports enabling endpoint mode for C5 controller. This > patch series adds support for PCIe endpoint mode in both the driver as well as > in DT. > This patch series depends on the changes made for Synopsys DesignWare endpoint > mode subsystem that are recently accepted. > @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 > which in turn depends on the patch made by Kishon > @ https://patchwork.kernel.org/patch/10975123/ > which is also under review. > > V4: > * Started using threaded irqs instead of kthreads > > V3: > * Re-ordered patches in the series to make the driver change as the last patch > * Took care of Thierry's review comments > > V2: > * Addressed Thierry & Bjorn's review comments > * Added EP mode specific binding documentation to already existing binding documentation file > * Removed patch that enables GPIO controller nodes explicitly as they are enabled already > > Vidya Sagar (5): > soc/tegra: bpmp: Update ABI header > dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 > arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 > arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 > platform > PCI: tegra: Add support for PCIe endpoint mode in Tegra194 Hi Lorenzo, I've acked patches 1, 2 and 5 of the series. I think you're going to need to apply patch 1 in order to satisfy a build-time dependency from patch 5. I can apply patches 3 and 4 to the Tegra tree since they're only adding device tree content that may conflict with some other patches that I have in the Tegra tree. Does that sound reasonable? Thierry
On Tue, Mar 03, 2020 at 02:40:53PM +0100, Thierry Reding wrote: > On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: > > Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can operate > > either in root port mode or in end point mode but only in one mode at a time. > > Platform P2972-0000 supports enabling endpoint mode for C5 controller. This > > patch series adds support for PCIe endpoint mode in both the driver as well as > > in DT. > > This patch series depends on the changes made for Synopsys DesignWare endpoint > > mode subsystem that are recently accepted. > > @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 > > which in turn depends on the patch made by Kishon > > @ https://patchwork.kernel.org/patch/10975123/ > > which is also under review. > > > > V4: > > * Started using threaded irqs instead of kthreads > > > > V3: > > * Re-ordered patches in the series to make the driver change as the last patch > > * Took care of Thierry's review comments > > > > V2: > > * Addressed Thierry & Bjorn's review comments > > * Added EP mode specific binding documentation to already existing binding documentation file > > * Removed patch that enables GPIO controller nodes explicitly as they are enabled already > > > > Vidya Sagar (5): > > soc/tegra: bpmp: Update ABI header > > dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 > > arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 > > arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 > > platform > > PCI: tegra: Add support for PCIe endpoint mode in Tegra194 > > Hi Lorenzo, > > I've acked patches 1, 2 and 5 of the series. I think you're going to > need to apply patch 1 in order to satisfy a build-time dependency from > patch 5. I can apply patches 3 and 4 to the Tegra tree since they're > only adding device tree content that may conflict with some other > patches that I have in the Tegra tree. > > Does that sound reasonable? Sure, that's absolutely fine by me, I will do. Thanks, Lorenzo
On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: > Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can operate > either in root port mode or in end point mode but only in one mode at a time. > Platform P2972-0000 supports enabling endpoint mode for C5 controller. This > patch series adds support for PCIe endpoint mode in both the driver as well as > in DT. > This patch series depends on the changes made for Synopsys DesignWare endpoint > mode subsystem that are recently accepted. > @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 > which in turn depends on the patch made by Kishon > @ https://patchwork.kernel.org/patch/10975123/ > which is also under review. > > V4: > * Started using threaded irqs instead of kthreads Hi Vidya, sorry for the bother, may I ask you to rebase the series (after answering Thierry's query) on top of my pci/endpoint branch please ? Please resend it and I will merge patches {1,2,5} then. Thanks, Lorenzo > V3: > * Re-ordered patches in the series to make the driver change as the last patch > * Took care of Thierry's review comments > > V2: > * Addressed Thierry & Bjorn's review comments > * Added EP mode specific binding documentation to already existing binding documentation file > * Removed patch that enables GPIO controller nodes explicitly as they are enabled already > > Vidya Sagar (5): > soc/tegra: bpmp: Update ABI header > dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 > arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 > arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 > platform > PCI: tegra: Add support for PCIe endpoint mode in Tegra194 > > .../bindings/pci/nvidia,tegra194-pcie.txt | 125 +++- > .../boot/dts/nvidia/tegra194-p2972-0000.dts | 18 + > arch/arm64/boot/dts/nvidia/tegra194.dtsi | 99 +++ > drivers/pci/controller/dwc/Kconfig | 30 +- > drivers/pci/controller/dwc/pcie-tegra194.c | 681 +++++++++++++++++- > include/soc/tegra/bpmp-abi.h | 10 +- > 6 files changed, 918 insertions(+), 45 deletions(-) > > -- > 2.17.1 >
On 3/3/2020 10:31 PM, Lorenzo Pieralisi wrote: > External email: Use caution opening links or attachments > > > On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: >> Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can operate >> either in root port mode or in end point mode but only in one mode at a time. >> Platform P2972-0000 supports enabling endpoint mode for C5 controller. This >> patch series adds support for PCIe endpoint mode in both the driver as well as >> in DT. >> This patch series depends on the changes made for Synopsys DesignWare endpoint >> mode subsystem that are recently accepted. >> @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 >> which in turn depends on the patch made by Kishon >> @ https://patchwork.kernel.org/patch/10975123/ >> which is also under review. >> >> V4: >> * Started using threaded irqs instead of kthreads > > Hi Vidya, > > sorry for the bother, may I ask you to rebase the series (after > answering Thierry's query) on top of my pci/endpoint branch please ? > > Please resend it and I will merge patches {1,2,5} then. Sure. I just sent V5 series rebasing patches on top of your pci/endpoint branch. Thanks, Vidya Sagar > > Thanks, > Lorenzo > >> V3: >> * Re-ordered patches in the series to make the driver change as the last patch >> * Took care of Thierry's review comments >> >> V2: >> * Addressed Thierry & Bjorn's review comments >> * Added EP mode specific binding documentation to already existing binding documentation file >> * Removed patch that enables GPIO controller nodes explicitly as they are enabled already >> >> Vidya Sagar (5): >> soc/tegra: bpmp: Update ABI header >> dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 >> arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 >> arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 >> platform >> PCI: tegra: Add support for PCIe endpoint mode in Tegra194 >> >> .../bindings/pci/nvidia,tegra194-pcie.txt | 125 +++- >> .../boot/dts/nvidia/tegra194-p2972-0000.dts | 18 + >> arch/arm64/boot/dts/nvidia/tegra194.dtsi | 99 +++ >> drivers/pci/controller/dwc/Kconfig | 30 +- >> drivers/pci/controller/dwc/pcie-tegra194.c | 681 +++++++++++++++++- >> include/soc/tegra/bpmp-abi.h | 10 +- >> 6 files changed, 918 insertions(+), 45 deletions(-) >> >> -- >> 2.17.1 >>
On 3/3/2020 11:43 PM, Vidya Sagar wrote: > > > On 3/3/2020 10:31 PM, Lorenzo Pieralisi wrote: >> External email: Use caution opening links or attachments >> >> >> On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: >>> Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can >>> operate >>> either in root port mode or in end point mode but only in one mode at >>> a time. >>> Platform P2972-0000 supports enabling endpoint mode for C5 >>> controller. This >>> patch series adds support for PCIe endpoint mode in both the driver >>> as well as >>> in DT. >>> This patch series depends on the changes made for Synopsys DesignWare >>> endpoint >>> mode subsystem that are recently accepted. >>> @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 >>> which in turn depends on the patch made by Kishon >>> @ https://patchwork.kernel.org/patch/10975123/ >>> which is also under review. >>> >>> V4: >>> * Started using threaded irqs instead of kthreads >> >> Hi Vidya, >> >> sorry for the bother, may I ask you to rebase the series (after >> answering Thierry's query) on top of my pci/endpoint branch please ? >> >> Please resend it and I will merge patches {1,2,5} then. > Sure. > I just sent V5 series rebasing patches on top of your pci/endpoint branch. Hi, Sorry to bother you again. Could you please take a look at V5 series? Thanks, Vidya Sagar > > Thanks, > Vidya Sagar >> >> Thanks, >> Lorenzo >> >>> V3: >>> * Re-ordered patches in the series to make the driver change as the >>> last patch >>> * Took care of Thierry's review comments >>> >>> V2: >>> * Addressed Thierry & Bjorn's review comments >>> * Added EP mode specific binding documentation to already existing >>> binding documentation file >>> * Removed patch that enables GPIO controller nodes explicitly as they >>> are enabled already >>> >>> Vidya Sagar (5): >>> soc/tegra: bpmp: Update ABI header >>> dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 >>> arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 >>> arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 >>> platform >>> PCI: tegra: Add support for PCIe endpoint mode in Tegra194 >>> >>> .../bindings/pci/nvidia,tegra194-pcie.txt | 125 +++- >>> .../boot/dts/nvidia/tegra194-p2972-0000.dts | 18 + >>> arch/arm64/boot/dts/nvidia/tegra194.dtsi | 99 +++ >>> drivers/pci/controller/dwc/Kconfig | 30 +- >>> drivers/pci/controller/dwc/pcie-tegra194.c | 681 +++++++++++++++++- >>> include/soc/tegra/bpmp-abi.h | 10 +- >>> 6 files changed, 918 insertions(+), 45 deletions(-) >>> >>> -- >>> 2.17.1 >>>
On Tue, Mar 10, 2020 at 11:12:35PM +0530, Vidya Sagar wrote: > > > On 3/3/2020 11:43 PM, Vidya Sagar wrote: > > > > > > On 3/3/2020 10:31 PM, Lorenzo Pieralisi wrote: > > > External email: Use caution opening links or attachments > > > > > > > > > On Tue, Mar 03, 2020 at 04:24:13PM +0530, Vidya Sagar wrote: > > > > Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that > > > > can operate > > > > either in root port mode or in end point mode but only in one > > > > mode at a time. > > > > Platform P2972-0000 supports enabling endpoint mode for C5 > > > > controller. This > > > > patch series adds support for PCIe endpoint mode in both the > > > > driver as well as > > > > in DT. > > > > This patch series depends on the changes made for Synopsys > > > > DesignWare endpoint > > > > mode subsystem that are recently accepted. > > > > @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 > > > > which in turn depends on the patch made by Kishon > > > > @ https://patchwork.kernel.org/patch/10975123/ > > > > which is also under review. > > > > > > > > V4: > > > > * Started using threaded irqs instead of kthreads > > > > > > Hi Vidya, > > > > > > sorry for the bother, may I ask you to rebase the series (after > > > answering Thierry's query) on top of my pci/endpoint branch please ? > > > > > > Please resend it and I will merge patches {1,2,5} then. > > Sure. > > I just sent V5 series rebasing patches on top of your pci/endpoint branch. > Hi, > Sorry to bother you again. > Could you please take a look at V5 series? I will merge it tomorrow, apologies. Thanks, Lorenzo
Tegra194 has three (C0, C4 & C5) dual mode PCIe controllers that can operate either in root port mode or in end point mode but only in one mode at a time. Platform P2972-0000 supports enabling endpoint mode for C5 controller. This patch series adds support for PCIe endpoint mode in both the driver as well as in DT. This patch series depends on the changes made for Synopsys DesignWare endpoint mode subsystem that are recently accepted. @ https://patchwork.kernel.org/project/linux-pci/list/?series=202211 which in turn depends on the patch made by Kishon @ https://patchwork.kernel.org/patch/10975123/ which is also under review. V4: * Started using threaded irqs instead of kthreads V3: * Re-ordered patches in the series to make the driver change as the last patch * Took care of Thierry's review comments V2: * Addressed Thierry & Bjorn's review comments * Added EP mode specific binding documentation to already existing binding documentation file * Removed patch that enables GPIO controller nodes explicitly as they are enabled already Vidya Sagar (5): soc/tegra: bpmp: Update ABI header dt-bindings: PCI: tegra: Add DT support for PCIe EP nodes in Tegra194 arm64: tegra: Add PCIe endpoint controllers nodes for Tegra194 arm64: tegra: Add support for PCIe endpoint mode in P2972-0000 platform PCI: tegra: Add support for PCIe endpoint mode in Tegra194 .../bindings/pci/nvidia,tegra194-pcie.txt | 125 +++- .../boot/dts/nvidia/tegra194-p2972-0000.dts | 18 + arch/arm64/boot/dts/nvidia/tegra194.dtsi | 99 +++ drivers/pci/controller/dwc/Kconfig | 30 +- drivers/pci/controller/dwc/pcie-tegra194.c | 681 +++++++++++++++++- include/soc/tegra/bpmp-abi.h | 10 +- 6 files changed, 918 insertions(+), 45 deletions(-)