[PATCH] arm64: dts: rockchip: Fix iomux for PCIe on rk3399

Robin Murphy robin.murphy at arm.com
Fri Dec 4 07:01:10 EST 2020


On 2020-12-01 15:05, Vicente Bergas wrote:
> Fix PCIe pins according to the documentation
> http://opensource.rock-chips.com/images/e/ee/Rockchip_RK3399TRM_V1.4_Part1-20170408.pdf
> in page 301:
> GRF_GPIO2D_IOMUX[5:4] gpio2d2_sel
>    0: gpio
>    1: sdio_detectn
>    2: pcie_clkreqn
> and page 313:
> GRF_GPIO4D_IOMUX[1:0] gpio4d0_sel
>    0: gpio
>    1: pcie_clkreqnb

What exactly is this fixing? Commit 461a00bb9d53 explicitly removed pin 
configurations for these functions on the grounds that they apparently 
don't work. FWIW it looks like they might also be related to some magic 
in GRF_SOC_CON7 which nobody's touching either.

Robin.

> Signed-off-by: Vicente Bergas <vicencb at gmail.com>
> ---
>   arch/arm64/boot/dts/rockchip/rk3399.dtsi | 6 ++----
>   1 file changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> index ada724b12..b37dac094 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi
> @@ -2666,13 +2666,11 @@ hdmi_cec: hdmi-cec {
>   
>   		pcie {
>   			pcie_clkreqn_cpm: pci-clkreqn-cpm {
> -				rockchip,pins =
> -					<2 RK_PD2 RK_FUNC_GPIO &pcfg_pull_none>;
> +				rockchip,pins = <2 RK_PD2 2 &pcfg_pull_none>;
>   			};
>   
>   			pcie_clkreqnb_cpm: pci-clkreqnb-cpm {
> -				rockchip,pins =
> -					<4 RK_PD0 RK_FUNC_GPIO &pcfg_pull_none>;
> +				rockchip,pins = <4 RK_PD0 1 &pcfg_pull_none>;
>   			};
>   		};
>   
> 



More information about the Linux-rockchip mailing list