2023-07-17 18:23:34

by Sebastian Reichel

[permalink] [raw]
Subject: [PATCH v2 0/2] RK3588 PCIe3 support

Hi,

This adds PCIe v3 support for RK3588. The series depends on the PCIe
v2 series [0], since the the same binding is used. It has been tested
on Rockchip EVB1 and Radxa Rock 5B.

Note, that the PCIe3 PHY driver is currently missing bifurcation
support for RK3588. Thus after this series only PCIe3x4 is usable
(in aggregated x4 mode) without adding support for the PHY's
"rockchip,pcie30-phymode" DT property, which allows configuring
how the lanes are distributed. Aggregated 3x4 mode seems to be the
most common configuration. Both EVB1 and Rock 5B use it, so I
cannot test anything else anyways.

[0] https://lore.kernel.org/all/[email protected]/

Changes since v1:
* https://lore.kernel.org/all/[email protected]/
* Collected Acked-by from Conor Dooley
* Dropped deprecated and useless num-ib-windows/num-ob-windows/num-viewport
properties from DT (Serge Semin)

-- Sebastian

Sebastian Reichel (2):
dt-bindings: phy: rockchip: add RK3588 PCIe v3 phy
arm64: dts: rockchip: rk3588: add PCIe3 support

.../bindings/phy/rockchip,pcie3-phy.yaml | 33 ++++-
arch/arm64/boot/dts/rockchip/rk3588.dtsi | 120 ++++++++++++++++++
2 files changed, 148 insertions(+), 5 deletions(-)

--
2.40.1



2023-07-18 06:28:54

by Vinod Koul

[permalink] [raw]
Subject: Re: (subset) [PATCH v2 0/2] RK3588 PCIe3 support


On Mon, 17 Jul 2023 19:35:10 +0200, Sebastian Reichel wrote:
> This adds PCIe v3 support for RK3588. The series depends on the PCIe
> v2 series [0], since the the same binding is used. It has been tested
> on Rockchip EVB1 and Radxa Rock 5B.
>
> Note, that the PCIe3 PHY driver is currently missing bifurcation
> support for RK3588. Thus after this series only PCIe3x4 is usable
> (in aggregated x4 mode) without adding support for the PHY's
> "rockchip,pcie30-phymode" DT property, which allows configuring
> how the lanes are distributed. Aggregated 3x4 mode seems to be the
> most common configuration. Both EVB1 and Rock 5B use it, so I
> cannot test anything else anyways.
>
> [...]

Applied, thanks!

[1/2] dt-bindings: phy: rockchip: add RK3588 PCIe v3 phy
commit: f66782cff479807ad7e98f0cf6a0c0babfe0159b

Best regards,
--
~Vinod



2023-07-24 20:15:09

by Heiko Stuebner

[permalink] [raw]
Subject: Re: (subset) [PATCH v2 0/2] RK3588 PCIe3 support

On Mon, 17 Jul 2023 19:35:10 +0200, Sebastian Reichel wrote:
> This adds PCIe v3 support for RK3588. The series depends on the PCIe
> v2 series [0], since the the same binding is used. It has been tested
> on Rockchip EVB1 and Radxa Rock 5B.
>
> Note, that the PCIe3 PHY driver is currently missing bifurcation
> support for RK3588. Thus after this series only PCIe3x4 is usable
> (in aggregated x4 mode) without adding support for the PHY's
> "rockchip,pcie30-phymode" DT property, which allows configuring
> how the lanes are distributed. Aggregated 3x4 mode seems to be the
> most common configuration. Both EVB1 and Rock 5B use it, so I
> cannot test anything else anyways.
>
> [...]

Applied, thanks!

[2/2] arm64: dts: rockchip: rk3588: add PCIe3 support
commit: 0acf4fa7f187cd7e3dad93f1ee14e9509687621e

Best regards,
--
Heiko Stuebner <[email protected]>