Re: [PATCH] arm64: dts: rockchip: Fix PCIe v3 pinctrl for BPI-R2-Pro
From: Heiko Stuebner <heiko@sntech.de>
Date: 2026-03-02 11:08:19
Also in:
linux-devicetree, linux-rockchip, lkml
Hi David, Am Sonntag, 1. März 2026, 16:54:41 Mitteleuropäische Normalzeit schrieb David Heidelberg via B4 Relay:
quoted hunk ↗ jump to hunk
From: David Heidelberg <david@ixit.cz> Add missing pinctrl for PD4 (PI6C PCIe clock enable, PCIE_CLKEN_H_GPIO0_D4) and set output high. This does fix freeze at initialization due pinctrl being in input state. Reported-by: Martin Filla <redacted> # reported by private message Signed-off-by: David Heidelberg <david@ixit.cz> --- arch/arm64/boot/dts/rockchip/rk3568-bpi-r2-pro.dts | 6 ++++++ 1 file changed, 6 insertions(+)diff --git a/arch/arm64/boot/dts/rockchip/rk3568-bpi-r2-pro.dts b/arch/arm64/boot/dts/rockchip/rk3568-bpi-r2-pro.dts index 4d3ebe50b90ba..b4c8abe37d1f3 100644 --- a/arch/arm64/boot/dts/rockchip/rk3568-bpi-r2-pro.dts +++ b/arch/arm64/boot/dts/rockchip/rk3568-bpi-r2-pro.dts@@ -121,6 +121,8 @@ vcc3v3_pi6c_05: regulator-vcc3v3-pi6c-05 { regulator-max-microvolt = <3300000>; enable-active-high; gpios = <&gpio0 RK_PD4 GPIO_ACTIVE_HIGH>; + pinctrl-0 = <&pi6cpcie_enable_h>; + pinctrl-names = "default";
please model hardware not how you want to use it. The pi6c in the pin-name suggests, that this is a Diodes clock-generator that creates the 100MHz frequency for PCIe [0] . Please see [1] for one possible variant to model this. Heiko [0] https://www.diodes.com/part/view/PI6CG33602C or some variant of it [1] http://lore.kernel.org/all/20260210080303.680403-1-heiko@sntech.de (local)