[PATCH] arm64: dts: qcom: qcs6490-rubikpi3: Move PCIe GPIOs to root ports

[PATCH] arm64: dts: qcom: qcs6490-rubikpi3: Move PCIe GPIOs to root ports

[Hongyang Zhao]

The Qualcomm PCIe binding deprecates perst-gpios on the host
bridge and expects endpoint reset GPIOs to be described on the root
port as reset-gpios.

Move the PCIe0 and PCIe1 reset and wake GPIOs to their root port
nodes. This keeps the GPIO ownership with the device below the root
port and matches the PCIe binding.

Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
---
Fix the PCIe reset and wake GPIO description for the Thundercomm
RubikPi3 board.

The board currently describes PERST# and wake GPIOs on the Qualcomm
PCIe host bridge nodes. The Qualcomm PCIe binding deprecates this
and expects endpoint reset GPIOs on the root port nodes as
reset-gpios.

Move the PCIe0 and PCIe1 GPIOs to the corresponding root port
nodes.
---
 .../arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts b/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
index f47efca42d48..5c08ab53cdbd 100644
--- a/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
+++ b/arch/arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts
@@ -812,9 +812,6 @@ &mdss_dsi_phy {
 };
 
 &pcie0 {
-	perst-gpios = <&tlmm 87 GPIO_ACTIVE_LOW>;
-	wake-gpios = <&tlmm 89 GPIO_ACTIVE_HIGH>;
-
 	pinctrl-0 = <&pcie0_clkreq_n>,
 		    <&pcie0_reset_n>,
 		    <&pcie0_wake_n>;
@@ -830,10 +827,12 @@ &pcie0_phy {
 	status = "okay";
 };
 
-&pcie1 {
-	perst-gpios = <&tlmm 2 GPIO_ACTIVE_LOW>;
-	wake-gpios = <&tlmm 3 GPIO_ACTIVE_LOW>;
+&pcie0_port {
+	reset-gpios = <&tlmm 87 GPIO_ACTIVE_LOW>;
+	wake-gpios = <&tlmm 89 GPIO_ACTIVE_HIGH>;
+};
 
+&pcie1 {
 	pinctrl-0 = <&pcie1_clkreq_n>,
 		    <&pcie1_reset_n>,
 		    <&pcie1_wake_n>;
@@ -849,6 +848,11 @@ &pcie1_phy {
 	status = "okay";
 };
 
+&pcie1_port0 {
+	reset-gpios = <&tlmm 2 GPIO_ACTIVE_LOW>;
+	wake-gpios = <&tlmm 3 GPIO_ACTIVE_LOW>;
+};
+
 &pm7325_gpios {
 	kypd_vol_up_n: kypd-vol-up-n-state {
 		pins = "gpio6";

---
base-commit: 6e845bcb78c95af935094040bd4edc3c2b6dd784
change-id: 20260607-rubikpi-bugfix-next-20260605-3c82cae6a7f1

Best regards,
--  
Hongyang Zhao <hongyang.zhao@thundersoft.com>

Re: [PATCH] arm64: dts: qcom: qcs6490-rubikpi3: Move PCIe GPIOs to root ports

[Dmitry Baryshkov]

On Sun, Jun 07, 2026 at 03:13:49AM +0800, Hongyang Zhao wrote:
> The Qualcomm PCIe binding deprecates perst-gpios on the host
> bridge and expects endpoint reset GPIOs to be described on the root
> port as reset-gpios.
> 
> Move the PCIe0 and PCIe1 reset and wake GPIOs to their root port
> nodes. This keeps the GPIO ownership with the device below the root
> port and matches the PCIe binding.
> 
> Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> ---
> Fix the PCIe reset and wake GPIO description for the Thundercomm
> RubikPi3 board.
> 
> The board currently describes PERST# and wake GPIOs on the Qualcomm
> PCIe host bridge nodes. The Qualcomm PCIe binding deprecates this
> and expects endpoint reset GPIOs on the root port nodes as
> reset-gpios.
> 
> Move the PCIe0 and PCIe1 GPIOs to the corresponding root port
> nodes.
> ---
>  .../arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts | 16 ++++++++++------
>  1 file changed, 10 insertions(+), 6 deletions(-)

Could you please refresh all kodiak DTs at once (and also move PHYs to
the the port node).


-- 
With best wishes
Dmitry

Re: [PATCH] arm64: dts: qcom: qcs6490-rubikpi3: Move PCIe GPIOs to root ports

[Hongyang Zhao]

> On Sun, Jun 07, 2026 at 03:13:49AM +0800, Hongyang Zhao wrote:
> > The Qualcomm PCIe binding deprecates perst-gpios on the host
> > bridge and expects endpoint reset GPIOs to be described on the root
> > port as reset-gpios.
> > 
> > Move the PCIe0 and PCIe1 reset and wake GPIOs to their root port
> > nodes. This keeps the GPIO ownership with the device below the root
> > port and matches the PCIe binding.
> > 
> > Signed-off-by: Hongyang Zhao <hongyang.zhao@thundersoft.com>
> > ---
> > Fix the PCIe reset and wake GPIO description for the Thundercomm
> > RubikPi3 board.
> > 
> > The board currently describes PERST# and wake GPIOs on the Qualcomm
> > PCIe host bridge nodes. The Qualcomm PCIe binding deprecates this
> > and expects endpoint reset GPIOs on the root port nodes as
> > reset-gpios.
> > 
> > Move the PCIe0 and PCIe1 GPIOs to the corresponding root port
> > nodes.
> > ---
> >  .../arm64/boot/dts/qcom/qcs6490-thundercomm-rubikpi3.dts | 16 ++++++++++------
> >  1 file changed, 10 insertions(+), 6 deletions(-)
> 
> Could you please refresh all kodiak DTs at once (and also move PHYs to
> the the port node).

Thanks! The requested changes are clear to me now, so I will send a v2
patch directly.


--
Thanks,
Hongyang