On Wed, Feb 04, 2026 at 06:03:55PM +0530, Manivannan Sadhasivam wrote:

On Wed, Feb 04, 2026 at 06:44:07AM +0000, Sherry Sun wrote:

quoted

quoted

On Tue, Feb 03, 2026 at 09:56:07AM +0800, Sherry Sun wrote:

quoted

Since describing the PCIe PERST# property under Host Bridge node is
now deprecated, it is recommended to add it to the Root Port node, so
creating the Root Port node and add the reset-gpios property in Root
Port.

Signed-off-by: Sherry Sun <redacted>
---
 arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi |  5 +++++
 arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi         | 11 +++++++++++
 arch/arm/boot/dts/nxp/imx/imx6qp-sabreauto.dts |  5 +++++
 3 files changed, 21 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi

b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi
index ba29720e3f72..fe9046c03ddd 100644

--- a/arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl-sabresd.dtsi

@@ -754,11 +754,16 @@ lvds0_out: endpoint {  &pcie {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_pcie>;
+	/* This property is deprecated, use reset-gpios from the Root Port
+node. */
 	reset-gpio = <&gpio7 12 GPIO_ACTIVE_LOW>;

You should just remove this property.

Hi Manivannan,

Actually I have discussed with Frank about this in V1, Frank suggested add
comments here instead of removing the old property, just in case the dts
used by old kernel to avoid function break. Or maybe we can at least keep it
for some kernel release, then remove it later.

If the new DT breaks the old kernel, then the kernel needs to be upgraded. But
typically, one would update both kernel + DT or just the kernel, not just DT.

But it should not impact bisect. Driver and dts is not in branch. If driver
applied ahead of dts patch, it should be fine to remove it. But default
linux-next can't guaratee the merge order.

To avoid merge complex, keep it for a whole, then clean up it later.

Frank

- Mani

quoted

Best Regards
Sherry

quoted

quoted

 	vpcie-supply = <&reg_pcie>;
 	status = "okay";
 };

+&pcie_port0 {
+	reset-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>; };
+
 &pwm1 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_pwm1>;

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi

b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
index 76e6043e1f91..eeb376193398 100644

--- a/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6qdl.dtsi

@@ -289,6 +289,17 @@ pcie: pcie@1ffc000 {
 				 <&clks IMX6QDL_CLK_PCIE_REF_125M>;
 			clock-names = "pcie", "pcie_bus", "pcie_phy";
 			status = "disabled";
+
+			pcie_port0: pcie@0 {
+				compatible = "pciclass,0604";
+				device_type = "pci";
+				reg = <0x0 0x0 0x0 0x0 0x0>;
+				bus-range = <0x01 0xff>;
+
+				#address-cells = <3>;
+				#size-cells = <2>;
+				ranges;
+			};
 		};

 		aips1: bus@2000000 { /* AIPS1 */

diff --git a/arch/arm/boot/dts/nxp/imx/imx6qp-sabreauto.dts

b/arch/arm/boot/dts/nxp/imx/imx6qp-sabreauto.dts
index c5b220aeaefd..6b12cab7175f 100644

--- a/arch/arm/boot/dts/nxp/imx/imx6qp-sabreauto.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6qp-sabreauto.dts

@@ -45,10 +45,15 @@ MX6QDL_PAD_GPIO_6__ENET_IRQ

	0x000b1

quoted

 };

 &pcie {
+	/* This property is deprecated, use reset-gpios from the Root Port
+node. */
 	reset-gpio = <&max7310_c 5 GPIO_ACTIVE_LOW>;
 	status = "okay";
 };

+&pcie_port0 {
+	reset-gpios = <&max7310_c 5 GPIO_ACTIVE_LOW>; };
+
 &sata {
 	status = "okay";
 };
--
2.37.1

--
மணிவண்ணன் சதாசிவம்

--
மணிவண்ணன் சதாசிவம்