Module Name: src Committed By: mrg Date: Sat Oct 2 20:41:47 UTC 2021
Modified Files: src/sys/arch/arm/dts: rk3399-pinebook-pro.dts rk3399-rockpro64.dts src/sys/arch/arm/rockchip: rk3399_pcie.c Log Message: default RK3399 PCIe to 1.x. sometime in mid-2019, rockchip disabled PCIe 2.x referrencing a non-public errata in linux and the dts, both of which were changed to default to only PCIe 1.x (in code, and in DT.) see: https://lore.kernel.org/all/1481881357-1793-1-git-send-email-shawn....@rock-chips.com/ we haven't observed any specific problems, but we've also tested far fewer cards than linux. there are other issues such as what happens for a PCI abort depending on which CPU triggers it we do not currently handle either. tested on rockpro64 with nvme & ahcisata, and pinebook pro nvme. still allows DT to set back to '2' if the user so desires. To generate a diff of this commit: cvs rdiff -u -r1.3 -r1.4 src/sys/arch/arm/dts/rk3399-pinebook-pro.dts cvs rdiff -u -r1.12 -r1.13 src/sys/arch/arm/dts/rk3399-rockpro64.dts cvs rdiff -u -r1.17 -r1.18 src/sys/arch/arm/rockchip/rk3399_pcie.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/arch/arm/dts/rk3399-pinebook-pro.dts diff -u src/sys/arch/arm/dts/rk3399-pinebook-pro.dts:1.3 src/sys/arch/arm/dts/rk3399-pinebook-pro.dts:1.4 --- src/sys/arch/arm/dts/rk3399-pinebook-pro.dts:1.3 Sun May 17 19:57:25 2020 +++ src/sys/arch/arm/dts/rk3399-pinebook-pro.dts Sat Oct 2 20:41:47 2021 @@ -766,7 +766,6 @@ &pcie0 { ep-gpios = <&gpio2 RK_PD4 GPIO_ACTIVE_HIGH>; num-lanes = <4>; - max-link-speed = <2>; pinctrl-names = "default"; pinctrl-0 = <&pcie_clkreqn_cpm>; vpcie3v3-supply = <&vcc3v3_pcie>; Index: src/sys/arch/arm/dts/rk3399-rockpro64.dts diff -u src/sys/arch/arm/dts/rk3399-rockpro64.dts:1.12 src/sys/arch/arm/dts/rk3399-rockpro64.dts:1.13 --- src/sys/arch/arm/dts/rk3399-rockpro64.dts:1.12 Fri Sep 3 01:21:48 2021 +++ src/sys/arch/arm/dts/rk3399-rockpro64.dts Sat Oct 2 20:41:47 2021 @@ -1,4 +1,4 @@ -/* $NetBSD: rk3399-rockpro64.dts,v 1.12 2021/09/03 01:21:48 mrg Exp $ */ +/* $NetBSD: rk3399-rockpro64.dts,v 1.13 2021/10/02 20:41:47 mrg Exp $ */ /*- * Copyright (c) 2019 Jared McNeill <jmcne...@invisible.ca> @@ -181,7 +181,6 @@ &pcie0 { ep-gpios = <&gpio2 RK_PD4 GPIO_ACTIVE_HIGH>; num-lanes = <4>; - max-link-speed = <2>; pinctrl-names = "default"; pinctrl-0 = <&pcie_clkreqn>; vpcie3v3-supply = <&vcc3v3_pcie>; Index: src/sys/arch/arm/rockchip/rk3399_pcie.c diff -u src/sys/arch/arm/rockchip/rk3399_pcie.c:1.17 src/sys/arch/arm/rockchip/rk3399_pcie.c:1.18 --- src/sys/arch/arm/rockchip/rk3399_pcie.c:1.17 Mon Sep 6 14:03:17 2021 +++ src/sys/arch/arm/rockchip/rk3399_pcie.c Sat Oct 2 20:41:47 2021 @@ -1,4 +1,4 @@ -/* $NetBSD: rk3399_pcie.c,v 1.17 2021/09/06 14:03:17 jmcneill Exp $ */ +/* $NetBSD: rk3399_pcie.c,v 1.18 2021/10/02 20:41:47 mrg Exp $ */ /* * Copyright (c) 2018 Mark Kettenis <kette...@openbsd.org> * @@ -17,7 +17,7 @@ #include <sys/cdefs.h> -__KERNEL_RCSID(1, "$NetBSD: rk3399_pcie.c,v 1.17 2021/09/06 14:03:17 jmcneill Exp $"); +__KERNEL_RCSID(1, "$NetBSD: rk3399_pcie.c,v 1.18 2021/10/02 20:41:47 mrg Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -255,8 +255,17 @@ rkpcie_attach(device_t parent, device_t ep_gpio = fdtbus_gpio_acquire(phandle, "ep-gpios", GPIO_PIN_OUTPUT); + /* + * Let board-specific properties override the default, which is set + * to PCIe 1.x, due to errata in the RK3399 CPU. We don't know exactly + * what these errata involved (not public), but posts from the + * @rock-chips.com domain to u-boot and linux-kernel lists indicate + * that there is a errata related to this, and indeed, the Datasheet + * since at least Rev 1.6 and inluding the latest Rev 1.8 say that the + * PCIe can handle 2.5GT/s (ie, PCIe 1.x). + */ if (of_getprop_uint32(phandle, "max-link-speed", &max_link_speed) != 0) - max_link_speed = 2; + max_link_speed = 1; if (of_getprop_uint32(phandle, "num-lanes", &num_lanes) != 0) num_lanes = 1;