* [PATCH v3 0/7] PCI: xilinx-nwl: Add phy support @ 2024-05-20 14:53 Sean Anderson 2024-05-20 14:53 ` [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys Sean Anderson 0 siblings, 1 reply; 5+ messages in thread From: Sean Anderson @ 2024-05-20 14:53 UTC (permalink / raw) To: Lorenzo Pieralisi, Krzysztof Wilczyński, Rob Herring, linux-pci Cc: Michal Simek, Thippeswamy Havalige, linux-arm-kernel, Bjorn Helgaas, linux-kernel, Sean Anderson, Bharat Kumar Gogada, Bharat Kumar Gogada, Bjorn Helgaas, Conor Dooley, Krzysztof Kozlowski, Lorenzo Pieralisi, Michal Simek, devicetree Add phy subsystem support for the xilinx-nwl PCIe controller. This series also includes several small fixes and improvements. Changes in v3: - Document phys property - Expand off-by-one commit message Changes in v2: - Remove phy-names - Add an example - Get phys by index and not by name Sean Anderson (7): dt-bindings: pci: xilinx-nwl: Add phys PCI: xilinx-nwl: Fix off-by-one in IRQ handler PCI: xilinx-nwl: Fix register misspelling PCI: xilinx-nwl: Rate-limit misc interrupt messages PCI: xilinx-nwl: Clean up clock on probe failure/removal PCI: xilinx-nwl: Add phy support arm64: zynqmp: Add PCIe phys .../bindings/pci/xlnx,nwl-pcie.yaml | 7 + .../boot/dts/xilinx/zynqmp-zcu102-revA.dts | 1 + drivers/pci/controller/pcie-xilinx-nwl.c | 122 ++++++++++++++---- 3 files changed, 107 insertions(+), 23 deletions(-) -- 2.35.1.1320.gc452695387.dirty ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys 2024-05-20 14:53 [PATCH v3 0/7] PCI: xilinx-nwl: Add phy support Sean Anderson @ 2024-05-20 14:53 ` Sean Anderson 2024-05-22 14:46 ` Rob Herring (Arm) 2024-05-22 22:28 ` Bjorn Helgaas 0 siblings, 2 replies; 5+ messages in thread From: Sean Anderson @ 2024-05-20 14:53 UTC (permalink / raw) To: Lorenzo Pieralisi, Krzysztof Wilczyński, Rob Herring, linux-pci Cc: Michal Simek, Thippeswamy Havalige, linux-arm-kernel, Bjorn Helgaas, linux-kernel, Sean Anderson, Conor Dooley, Krzysztof Kozlowski, devicetree Add phys properties so Linux can power-on/configure the GTR transcievers. Signed-off-by: Sean Anderson <sean.anderson@linux.dev> --- Changes in v3: - Document phys property Changes in v2: - Remove phy-names - Add an example Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml index 426f90a47f35..cc50795d170b 100644 --- a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml +++ b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml @@ -61,6 +61,11 @@ properties: interrupt-map: maxItems: 4 + phys: + minItems: 1 + maxItems: 4 + description: One phy per logical lane, in order + power-domains: maxItems: 1 @@ -110,6 +115,7 @@ examples: - | #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/interrupt-controller/irq.h> + #include <dt-bindings/phy/phy.h> #include <dt-bindings/power/xlnx-zynqmp-power.h> soc { #address-cells = <2>; @@ -138,6 +144,7 @@ examples: <0x0 0x0 0x0 0x3 &pcie_intc 0x3>, <0x0 0x0 0x0 0x4 &pcie_intc 0x4>; msi-parent = <&nwl_pcie>; + phys = <&psgtr 0 PHY_TYPE_PCIE 0 0>; power-domains = <&zynqmp_firmware PD_PCIE>; iommus = <&smmu 0x4d0>; pcie_intc: legacy-interrupt-controller { -- 2.35.1.1320.gc452695387.dirty ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys 2024-05-20 14:53 ` [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys Sean Anderson @ 2024-05-22 14:46 ` Rob Herring (Arm) 2024-05-22 22:28 ` Bjorn Helgaas 1 sibling, 0 replies; 5+ messages in thread From: Rob Herring (Arm) @ 2024-05-22 14:46 UTC (permalink / raw) To: Sean Anderson Cc: Krzysztof Kozlowski, Lorenzo Pieralisi, Krzysztof Wilczyński, linux-arm-kernel, Michal Simek, Bjorn Helgaas, Conor Dooley, Thippeswamy Havalige, linux-pci, linux-kernel, devicetree On Mon, 20 May 2024 10:53:56 -0400, Sean Anderson wrote: > Add phys properties so Linux can power-on/configure the GTR > transcievers. > > Signed-off-by: Sean Anderson <sean.anderson@linux.dev> > --- > > Changes in v3: > - Document phys property > > Changes in v2: > - Remove phy-names > - Add an example > > Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > Reviewed-by: Rob Herring (Arm) <robh@kernel.org> ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys 2024-05-20 14:53 ` [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys Sean Anderson 2024-05-22 14:46 ` Rob Herring (Arm) @ 2024-05-22 22:28 ` Bjorn Helgaas 2024-05-23 15:19 ` Sean Anderson 1 sibling, 1 reply; 5+ messages in thread From: Bjorn Helgaas @ 2024-05-22 22:28 UTC (permalink / raw) To: Sean Anderson, g Cc: Lorenzo Pieralisi, Krzysztof Wilczyński, Rob Herring, linux-pci, Michal Simek, Thippeswamy Havalige, linux-arm-kernel, Bjorn Helgaas, linux-kernel, Conor Dooley, Krzysztof Kozlowski, devicetree On Mon, May 20, 2024 at 10:53:56AM -0400, Sean Anderson wrote: > Add phys properties so Linux can power-on/configure the GTR > transcievers. s/transcievers/transceivers/ Possibly s/phys/PHYs/ in subject, commit log, DT description to avoid confusion with "phys" (short for generic "physical"). Or maybe even just "PHY properties"? What does "GTR" mean? Possibly expand that? > Signed-off-by: Sean Anderson <sean.anderson@linux.dev> > --- > > Changes in v3: > - Document phys property > > Changes in v2: > - Remove phy-names > - Add an example > > Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml > index 426f90a47f35..cc50795d170b 100644 > --- a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml > +++ b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml > @@ -61,6 +61,11 @@ properties: > interrupt-map: > maxItems: 4 > > + phys: > + minItems: 1 > + maxItems: 4 > + description: One phy per logical lane, in order > + > power-domains: > maxItems: 1 > > @@ -110,6 +115,7 @@ examples: > - | > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/interrupt-controller/irq.h> > + #include <dt-bindings/phy/phy.h> > #include <dt-bindings/power/xlnx-zynqmp-power.h> > soc { > #address-cells = <2>; > @@ -138,6 +144,7 @@ examples: > <0x0 0x0 0x0 0x3 &pcie_intc 0x3>, > <0x0 0x0 0x0 0x4 &pcie_intc 0x4>; > msi-parent = <&nwl_pcie>; > + phys = <&psgtr 0 PHY_TYPE_PCIE 0 0>; > power-domains = <&zynqmp_firmware PD_PCIE>; > iommus = <&smmu 0x4d0>; > pcie_intc: legacy-interrupt-controller { > -- > 2.35.1.1320.gc452695387.dirty > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys 2024-05-22 22:28 ` Bjorn Helgaas @ 2024-05-23 15:19 ` Sean Anderson 0 siblings, 0 replies; 5+ messages in thread From: Sean Anderson @ 2024-05-23 15:19 UTC (permalink / raw) To: Bjorn Helgaas, g Cc: Lorenzo Pieralisi, Krzysztof Wilczyński, Rob Herring, linux-pci, Michal Simek, Thippeswamy Havalige, linux-arm-kernel, Bjorn Helgaas, linux-kernel, Conor Dooley, Krzysztof Kozlowski, devicetree On 5/22/24 18:28, Bjorn Helgaas wrote: > On Mon, May 20, 2024 at 10:53:56AM -0400, Sean Anderson wrote: >> Add phys properties so Linux can power-on/configure the GTR >> transcievers. > > s/transcievers/transceivers/ I always forget the spelling is backwards on this one > Possibly s/phys/PHYs/ in subject, commit log, DT description to avoid > confusion with "phys" (short for generic "physical"). Or maybe even > just "PHY properties"? Well, this is the name for the property... > What does "GTR" mean? Possibly expand that? It's "xlnx,zynqmp-psgtr-v1.1". These are the available transceivers on the ZynqMP, which is the only SoC this device is present on. I had hoped this would be clear by calling them "GTR transcievers"... --Sean >> Signed-off-by: Sean Anderson <sean.anderson@linux.dev> >> --- >> >> Changes in v3: >> - Document phys property >> >> Changes in v2: >> - Remove phy-names >> - Add an example >> >> Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml >> index 426f90a47f35..cc50795d170b 100644 >> --- a/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml >> +++ b/Documentation/devicetree/bindings/pci/xlnx,nwl-pcie.yaml >> @@ -61,6 +61,11 @@ properties: >> interrupt-map: >> maxItems: 4 >> >> + phys: >> + minItems: 1 >> + maxItems: 4 >> + description: One phy per logical lane, in order >> + >> power-domains: >> maxItems: 1 >> >> @@ -110,6 +115,7 @@ examples: >> - | >> #include <dt-bindings/interrupt-controller/arm-gic.h> >> #include <dt-bindings/interrupt-controller/irq.h> >> + #include <dt-bindings/phy/phy.h> >> #include <dt-bindings/power/xlnx-zynqmp-power.h> >> soc { >> #address-cells = <2>; >> @@ -138,6 +144,7 @@ examples: >> <0x0 0x0 0x0 0x3 &pcie_intc 0x3>, >> <0x0 0x0 0x0 0x4 &pcie_intc 0x4>; >> msi-parent = <&nwl_pcie>; >> + phys = <&psgtr 0 PHY_TYPE_PCIE 0 0>; >> power-domains = <&zynqmp_firmware PD_PCIE>; >> iommus = <&smmu 0x4d0>; >> pcie_intc: legacy-interrupt-controller { >> -- >> 2.35.1.1320.gc452695387.dirty >> ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-05-23 15:19 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-05-20 14:53 [PATCH v3 0/7] PCI: xilinx-nwl: Add phy support Sean Anderson 2024-05-20 14:53 ` [PATCH v3 1/7] dt-bindings: pci: xilinx-nwl: Add phys Sean Anderson 2024-05-22 14:46 ` Rob Herring (Arm) 2024-05-22 22:28 ` Bjorn Helgaas 2024-05-23 15:19 ` Sean Anderson
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).