| Message ID | 20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Enable IPQ5018 PCI support | expand |
On Fri, 21 Mar 2025 16:14:38 +0400, George Moussalem wrote: > This patch series adds the relevant phy and controller > DT configurations for enabling PCI gen2 support > on IPQ5018. IPQ5018 has two phys and two controllers, > one dual-lane and one single-lane. > > Last patch series (v3) submitted dates back to August 30, 2024. > As I've worked to add IPQ5018 platform support in OpenWrt, I'm > continuing the efforts to add Linux kernel support. > > Signed-off-by: George Moussalem <george.moussalem@outlook.com> > --- > Changes in v6: > - Fixed issues reported by 'make dt_bindings_check' as per Rob's bot > - Removed Krzysztof's Ack-tag on > - Link to v5: https://lore.kernel.org/r/20250321-ipq5018-pcie-v5-0-aae2caa1f418@outlook.com > > Changes in v5: > - Re-ordered reg and reg-names in dt-bindings and dts to align with > other IPQ SoCs > - Corrected nr of interrupts in dt-bindings: phy: qcom: Add IPQ5018 SoC > - Corrected ranges property of pcie controller nodes > - Removed newlines between cells properties in pcie phy nodes > - Modified dt bindings to add descriptions and separate conditions for > ipq5018 and ipq5332 as they have different nr of clocks and resets > As such, also removed Krzysztof's RB tag for validation > - Ran dtbs_check and fixed: > interrupt-map property in pcie nodes: > /soc@0/pcie@80000000:interrupt-map: Cell 13 is not a phandle(0) > /soc@0/pcie@a0000000:interrupt-map: Cell 13 is not a phandle(0) > - Added missing gpio header file to ipq5018-rdp432-c2.dts > - Added MHI register requirement to bindings and to PCIe nodes as per: > Depends-on: <20250317100029.881286-2-quic_varada@quicinc.com> > - Link to v4: https://lore.kernel.org/all/DS7PR19MB8883F2538AA7D047E13C102B9DD22@DS7PR19MB8883.namprd19.prod.outlook.com/ > > Changes in v4: > - removed dependency as the following have been applied: > dt-bindings: phy: qcom,uniphy-pcie: Document PCIe uniphy > phy: qcom: Introduce PCIe UNIPHY 28LP driver > dt-bindings: PCI: qcom: Document the IPQ5332 PCIe controller > Link: https://lore.kernel.org/all/20250313080600.1719505-1-quic_varada@quicinc.com/ > - added Mani's RB tag to: PCI: qcom: Add support for IPQ5018 > - Removed power-domains property requirement in dt-bindings for IPQ5018 > and removed Krzysztof's RB tag from: > dt-bindings: PCI: qcom: Add IPQ5018 SoC > - fixed author chain and retained Sricharan Ramabadhran in SoB tags and > kept Nitheesh Sekar as the original author > - Removed comments as per Konrad's comment in: > arm64: dts: qcom: ipq5018: Add PCIe related nodes > - Link to v3 submitted by Sricharan Ramabadhran: > Link: https://lore.kernel.org/all/20240830081132.4016860-1-quic_srichara@quicinc.com/ > - Link to v3, incorrectly versioned: > Link: https://lore.kernel.org/all/DS7PR19MB8883BC190797BECAA78EC50F9DCB2@DS7PR19MB8883.namprd19.prod.outlook.com/ > > Changes in v3 (incorrectly versioned): > - Depends on > Link: https://patchwork.kernel.org/project/linux-arm-msm/cover/20250220094251.230936-1-quic_varada@quicinc.com/ > - Added 8 MSI SPI and 1 global interrupts (Thanks Mani for confirming) > - Added hw revision (internal/synopsys) and nr of lanes in patch 4 > commit msg > - Sorted reg addresses and moved PCIe nodes accordingly > - Moved to GIC based interrupts > - Added rootport node in controller nodes > - Tested on Linksys devices (MX5500/SPNMX56) > - Link to v2: https://lore.kernel.org/all/20240827045757.1101194-1-quic_srichara com/ > > Changes in v3: > - Added Reviewed-by tag for patch#1. > - Fixed dev_err_probe usage in patch#3. > - Added pinctrl/wak pins for pcie1 in patch#6. > > Changes in v2: > - Fixed all review comments from Krzysztof, Robert Marko, > Dmitry Baryshkov, Manivannan Sadhasivam, Konrad Dybcio. > - Updated the respective patches for their changes. > - Link to v1: https://lore.kernel.org/lkml/32389b66-48f3-8ee8-e2f1-1613feed3cc7@gmail.com/T/ > > --- > Nitheesh Sekar (6): > dt-bindings: phy: qcom: uniphy-pcie: Add ipq5018 compatible > phy: qualcomm: qcom-uniphy-pcie 28LP add support for IPQ5018 > dt-bindings: PCI: qcom: Add IPQ5018 SoC > PCI: qcom: Add support for IPQ5018 > arm64: dts: qcom: ipq5018: Add PCIe related nodes > arm64: dts: qcom: ipq5018: Enable PCIe > > .../devicetree/bindings/pci/qcom,pcie.yaml | 50 +++++ > .../bindings/phy/qcom,ipq5332-uniphy-pcie-phy.yaml | 49 ++++- > arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dts | 40 ++++ > arch/arm64/boot/dts/qcom/ipq5018.dtsi | 234 ++++++++++++++++++++- > drivers/pci/controller/dwc/pcie-qcom.c | 1 + > drivers/phy/qualcomm/phy-qcom-uniphy-pcie-28lp.c | 45 ++++ > 6 files changed, 409 insertions(+), 10 deletions(-) > --- > base-commit: 5744a64fddfc33629f3bcc9a06a646f7443077a7 > change-id: 20250321-ipq5018-pcie-1d44abf0e2f5 > > Best regards, > -- > George Moussalem <george.moussalem@outlook.com> > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/qcom/' for 20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com: arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dtb: pcie@80000000: reg: [[2147483648, 3869], [2147487520, 168], [2147487744, 4096], [491520, 12288], [2148532224, 4096], [503808, 4096]] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dtb: pcie@80000000: reg-names: ['dbi', 'elbi', 'atu', 'parf', 'config', 'mhi'] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dtb: pcie@a0000000: reg: [[2684354560, 3869], [2684358432, 168], [2684358656, 4096], [524288, 12288], [2685403136, 4096], [536576, 4096]] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dtb: pcie@a0000000: reg-names: ['dbi', 'elbi', 'atu', 'parf', 'config', 'mhi'] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-tplink-archer-ax55-v1.dtb: pcie@80000000: reg: [[2147483648, 3869], [2147487520, 168], [2147487744, 4096], [491520, 12288], [2148532224, 4096], [503808, 4096]] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-tplink-archer-ax55-v1.dtb: pcie@80000000: reg-names: ['dbi', 'elbi', 'atu', 'parf', 'config', 'mhi'] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-tplink-archer-ax55-v1.dtb: pcie@a0000000: reg: [[2684354560, 3869], [2684358432, 168], [2684358656, 4096], [524288, 12288], [2685403136, 4096], [536576, 4096]] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml# arch/arm64/boot/dts/qcom/ipq5018-tplink-archer-ax55-v1.dtb: pcie@a0000000: reg-names: ['dbi', 'elbi', 'atu', 'parf', 'config', 'mhi'] is too long from schema $id: http://devicetree.org/schemas/pci/qcom,pcie.yaml#
On Fri, Mar 21, 2025 at 04:14:38PM +0400, George Moussalem wrote: > This patch series adds the relevant phy and controller > DT configurations for enabling PCI gen2 support > on IPQ5018. IPQ5018 has two phys and two controllers, > one dual-lane and one single-lane. > > Last patch series (v3) submitted dates back to August 30, 2024. > As I've worked to add IPQ5018 platform support in OpenWrt, I'm > continuing the efforts to add Linux kernel support. > > Signed-off-by: George Moussalem <george.moussalem@outlook.com> > --- > Changes in v6: > - Fixed issues reported by 'make dt_bindings_check' as per Rob's bot > - Removed Krzysztof's Ack-tag on Why? Again, I cannot compare this serie: b4 diff '20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com' Grabbing thread from lore.kernel.org/all/20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com/t.mbox.gz Checking for older revisions Grabbing search results from lore.kernel.org Nothing matching that query. --- Analyzing 12 messages in the thread Could not find lower series to compare against.
On 3/24/25 11:47, Krzysztof Kozlowski wrote: > On Fri, Mar 21, 2025 at 04:14:38PM +0400, George Moussalem wrote: >> This patch series adds the relevant phy and controller >> DT configurations for enabling PCI gen2 support >> on IPQ5018. IPQ5018 has two phys and two controllers, >> one dual-lane and one single-lane. >> >> Last patch series (v3) submitted dates back to August 30, 2024. >> As I've worked to add IPQ5018 platform support in OpenWrt, I'm >> continuing the efforts to add Linux kernel support. >> >> Signed-off-by: George Moussalem <george.moussalem@outlook.com> >> --- >> Changes in v6: >> - Fixed issues reported by 'make dt_bindings_check' as per Rob's bot >> - Removed Krzysztof's Ack-tag on > > Why? In v5, I modified dt bindings (patch 1) to add descriptions and separate conditions for ipq5018 and ipq5332 as they have different nr of clocks and resets but left your tag in there. I removed it in v6 to run the changes past you again. I intend to send the next version with comments addressed in patch 5 (to add a comment above the max-link-speed property). Anything else you would like to see addressed? > > Again, I cannot compare this serie: > > b4 diff '20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com' > Grabbing thread from lore.kernel.org/all/20250321-ipq5018-pcie-v6-0-b7d659a76205@outlook.com/t.mbox.gz > Checking for older revisions > Grabbing search results from lore.kernel.org > Nothing matching that query. > --- > Analyzing 12 messages in the thread > Could not find lower series to compare against. > I tried 'b4 prep -n <topical branch> -F <message-ID>' to try and preseve the history but that didn't work, unfortunately. Instead I used -f and forked it off the master branch of linux-next. > > Best regards, George
This patch series adds the relevant phy and controller DT configurations for enabling PCI gen2 support on IPQ5018. IPQ5018 has two phys and two controllers, one dual-lane and one single-lane. Last patch series (v3) submitted dates back to August 30, 2024. As I've worked to add IPQ5018 platform support in OpenWrt, I'm continuing the efforts to add Linux kernel support. Signed-off-by: George Moussalem <george.moussalem@outlook.com> --- Changes in v6: - Fixed issues reported by 'make dt_bindings_check' as per Rob's bot - Removed Krzysztof's Ack-tag on - Link to v5: https://lore.kernel.org/r/20250321-ipq5018-pcie-v5-0-aae2caa1f418@outlook.com Changes in v5: - Re-ordered reg and reg-names in dt-bindings and dts to align with other IPQ SoCs - Corrected nr of interrupts in dt-bindings: phy: qcom: Add IPQ5018 SoC - Corrected ranges property of pcie controller nodes - Removed newlines between cells properties in pcie phy nodes - Modified dt bindings to add descriptions and separate conditions for ipq5018 and ipq5332 as they have different nr of clocks and resets As such, also removed Krzysztof's RB tag for validation - Ran dtbs_check and fixed: interrupt-map property in pcie nodes: /soc@0/pcie@80000000:interrupt-map: Cell 13 is not a phandle(0) /soc@0/pcie@a0000000:interrupt-map: Cell 13 is not a phandle(0) - Added missing gpio header file to ipq5018-rdp432-c2.dts - Added MHI register requirement to bindings and to PCIe nodes as per: Depends-on: <20250317100029.881286-2-quic_varada@quicinc.com> - Link to v4: https://lore.kernel.org/all/DS7PR19MB8883F2538AA7D047E13C102B9DD22@DS7PR19MB8883.namprd19.prod.outlook.com/ Changes in v4: - removed dependency as the following have been applied: dt-bindings: phy: qcom,uniphy-pcie: Document PCIe uniphy phy: qcom: Introduce PCIe UNIPHY 28LP driver dt-bindings: PCI: qcom: Document the IPQ5332 PCIe controller Link: https://lore.kernel.org/all/20250313080600.1719505-1-quic_varada@quicinc.com/ - added Mani's RB tag to: PCI: qcom: Add support for IPQ5018 - Removed power-domains property requirement in dt-bindings for IPQ5018 and removed Krzysztof's RB tag from: dt-bindings: PCI: qcom: Add IPQ5018 SoC - fixed author chain and retained Sricharan Ramabadhran in SoB tags and kept Nitheesh Sekar as the original author - Removed comments as per Konrad's comment in: arm64: dts: qcom: ipq5018: Add PCIe related nodes - Link to v3 submitted by Sricharan Ramabadhran: Link: https://lore.kernel.org/all/20240830081132.4016860-1-quic_srichara@quicinc.com/ - Link to v3, incorrectly versioned: Link: https://lore.kernel.org/all/DS7PR19MB8883BC190797BECAA78EC50F9DCB2@DS7PR19MB8883.namprd19.prod.outlook.com/ Changes in v3 (incorrectly versioned): - Depends on Link: https://patchwork.kernel.org/project/linux-arm-msm/cover/20250220094251.230936-1-quic_varada@quicinc.com/ - Added 8 MSI SPI and 1 global interrupts (Thanks Mani for confirming) - Added hw revision (internal/synopsys) and nr of lanes in patch 4 commit msg - Sorted reg addresses and moved PCIe nodes accordingly - Moved to GIC based interrupts - Added rootport node in controller nodes - Tested on Linksys devices (MX5500/SPNMX56) - Link to v2: https://lore.kernel.org/all/20240827045757.1101194-1-quic_srichara com/ Changes in v3: - Added Reviewed-by tag for patch#1. - Fixed dev_err_probe usage in patch#3. - Added pinctrl/wak pins for pcie1 in patch#6. Changes in v2: - Fixed all review comments from Krzysztof, Robert Marko, Dmitry Baryshkov, Manivannan Sadhasivam, Konrad Dybcio. - Updated the respective patches for their changes. - Link to v1: https://lore.kernel.org/lkml/32389b66-48f3-8ee8-e2f1-1613feed3cc7@gmail.com/T/ --- Nitheesh Sekar (6): dt-bindings: phy: qcom: uniphy-pcie: Add ipq5018 compatible phy: qualcomm: qcom-uniphy-pcie 28LP add support for IPQ5018 dt-bindings: PCI: qcom: Add IPQ5018 SoC PCI: qcom: Add support for IPQ5018 arm64: dts: qcom: ipq5018: Add PCIe related nodes arm64: dts: qcom: ipq5018: Enable PCIe .../devicetree/bindings/pci/qcom,pcie.yaml | 50 +++++ .../bindings/phy/qcom,ipq5332-uniphy-pcie-phy.yaml | 49 ++++- arch/arm64/boot/dts/qcom/ipq5018-rdp432-c2.dts | 40 ++++ arch/arm64/boot/dts/qcom/ipq5018.dtsi | 234 ++++++++++++++++++++- drivers/pci/controller/dwc/pcie-qcom.c | 1 + drivers/phy/qualcomm/phy-qcom-uniphy-pcie-28lp.c | 45 ++++ 6 files changed, 409 insertions(+), 10 deletions(-) --- base-commit: 5744a64fddfc33629f3bcc9a06a646f7443077a7 change-id: 20250321-ipq5018-pcie-1d44abf0e2f5 Best regards,