[v6,0/5] Add PCIe EP support for SDX65

Message

Rohit Agarwal May 18, 2023, 5:47 p.m. UTC

Hi,

Changes in v6:
 - Rebased on top of 6.4-rc2.

Changes in v5:
 - Addressed some minor comments from Konrad
 - Rebased on top of 6.3-rc5.

Changes in v4:
 - Addressed comment from Dmitry to move the gpios to the board file.

Changes in v3:
 - Removing the applied patch.
 - Addressing some of the compile time issues missed in v2.

Changes in v2:
 - Addressing comments from Konrad and Dmitry.
 - Rebased on top of 6.3-rc1.

This series adds the devicetree support for PCIe PHY and PCIe EP on SDX65.
The PCIe EP is enabled on SDX65 MTP board.

Thanks,
Rohit.

Rohit Agarwal (5):
  dt-bindings: PCI: qcom: Add SDX65 SoC
  ARM: dts: qcom: sdx65: Add support for PCIe PHY
  ARM: dts: qcom: sdx65: Add support for PCIe EP
  ARM: dts: qcom: sdx65-mtp: Enable PCIe PHY
  ARM: dts: qcom: sdx65-mtp: Enable PCIe EP

 .../devicetree/bindings/pci/qcom,pcie-ep.yaml      |  2 +
 arch/arm/boot/dts/qcom-sdx65-mtp.dts               | 42 +++++++++++
 arch/arm/boot/dts/qcom-sdx65.dtsi                  | 87 ++++++++++++++++++++++
 3 files changed, 131 insertions(+)

Comments

Bjorn Andersson May 25, 2023, 4:54 a.m. UTC | #1

On Thu, 18 May 2023 23:17:48 +0530, Rohit Agarwal wrote:
> Changes in v6:
>  - Rebased on top of 6.4-rc2.
> 
> Changes in v5:
>  - Addressed some minor comments from Konrad
>  - Rebased on top of 6.3-rc5.
> 
> [...]

Applied, thanks!

[1/5] dt-bindings: PCI: qcom: Add SDX65 SoC
      (no commit info)
[2/5] ARM: dts: qcom: sdx65: Add support for PCIe PHY
      commit: 92543a1ef22d0270425a4dfe8efe4ab30c4a8a5e
[3/5] ARM: dts: qcom: sdx65: Add support for PCIe EP
      commit: 91dfb64ba70bab4d3517f4e7cb2e4cc8f5a8f81a
[4/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe PHY
      commit: 07bb20f207cb5868a47217681e4843f566843d29
[5/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe EP
      commit: e110dea61ff3f35e6d15e8c5009fb0e876a7d8ae

Best regards,

Bjorn Helgaas May 25, 2023, 10:13 p.m. UTC | #2

On Wed, May 24, 2023 at 09:54:38PM -0700, Bjorn Andersson wrote:
> On Thu, 18 May 2023 23:17:48 +0530, Rohit Agarwal wrote:
> > Changes in v6:
> >  - Rebased on top of 6.4-rc2.
> > 
> > Changes in v5:
> >  - Addressed some minor comments from Konrad
> >  - Rebased on top of 6.3-rc5.
> > 
> > [...]
> 
> Applied, thanks!
> 
> [1/5] dt-bindings: PCI: qcom: Add SDX65 SoC
>       (no commit info)

For clarification, I guess this means you did *not* apply [1/5], and
you'd like Lorenzo to apply it?

> [2/5] ARM: dts: qcom: sdx65: Add support for PCIe PHY
>       commit: 92543a1ef22d0270425a4dfe8efe4ab30c4a8a5e
> [3/5] ARM: dts: qcom: sdx65: Add support for PCIe EP
>       commit: 91dfb64ba70bab4d3517f4e7cb2e4cc8f5a8f81a
> [4/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe PHY
>       commit: 07bb20f207cb5868a47217681e4843f566843d29
> [5/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe EP
>       commit: e110dea61ff3f35e6d15e8c5009fb0e876a7d8ae

Bjorn Andersson May 25, 2023, 11:02 p.m. UTC | #3

On Thu, May 25, 2023 at 05:13:27PM -0500, Bjorn Helgaas wrote:
> On Wed, May 24, 2023 at 09:54:38PM -0700, Bjorn Andersson wrote:
> > On Thu, 18 May 2023 23:17:48 +0530, Rohit Agarwal wrote:
> > > Changes in v6:
> > >  - Rebased on top of 6.4-rc2.
> > > 
> > > Changes in v5:
> > >  - Addressed some minor comments from Konrad
> > >  - Rebased on top of 6.3-rc5.
> > > 
> > > [...]
> > 
> > Applied, thanks!
> > 
> > [1/5] dt-bindings: PCI: qcom: Add SDX65 SoC
> >       (no commit info)
> 
> For clarification, I guess this means you did *not* apply [1/5], and
> you'd like Lorenzo to apply it?
> 

That is correct, sorry for the confusion.

Thanks,
Bjorn

> > [2/5] ARM: dts: qcom: sdx65: Add support for PCIe PHY
> >       commit: 92543a1ef22d0270425a4dfe8efe4ab30c4a8a5e
> > [3/5] ARM: dts: qcom: sdx65: Add support for PCIe EP
> >       commit: 91dfb64ba70bab4d3517f4e7cb2e4cc8f5a8f81a
> > [4/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe PHY
> >       commit: 07bb20f207cb5868a47217681e4843f566843d29
> > [5/5] ARM: dts: qcom: sdx65-mtp: Enable PCIe EP
> >       commit: e110dea61ff3f35e6d15e8c5009fb0e876a7d8ae