arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 1 - drivers/pci/controller/pcie-rockchip-host.c | 5 +++++ drivers/pci/controller/pcie-rockchip.c | 8 ++++++-- 3 files changed, 11 insertions(+), 3 deletions(-)
In recent interactions with Shawn Lin from Rockchip it came to my attention there's an unknown errata regarding 5.0 GT/s operational speed of their PCIe core. According to Shawn there's grave danger even if the odds are low. To contain any damage, let's cover the remaining corner-cases where the default would lead to 5.0 GT/s operation as well as add a comment to Root Complex driver core, documenting this danger. Geraldo Nascimento (3): PCI: rockchip: warn of danger of 5.0 GT/s speeds PCI: rockchip-host: comment danger of 5.0 GT/s speed arm64: dts: rockchip: drop max-link-speed = <2> in helios64 PCIe arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 1 - drivers/pci/controller/pcie-rockchip-host.c | 5 +++++ drivers/pci/controller/pcie-rockchip.c | 8 ++++++-- 3 files changed, 11 insertions(+), 3 deletions(-) -- 2.49.0
Hi Geraldo, 在 2025/11/15 星期六 17:10, Geraldo Nascimento 写道: > In recent interactions with Shawn Lin from Rockchip it came to my > attention there's an unknown errata regarding 5.0 GT/s operational > speed of their PCIe core. According to Shawn there's grave danger > even if the odds are low. To contain any damage, let's cover the > remaining corner-cases where the default would lead to 5.0 GT/s > operation as well as add a comment to Root Complex driver core, > documenting this danger. > I'm not sure just adding a warn would be a good choice. Could we totally force to use gen1 and add a warn if trying to use Gen2. Meanwhile amend the commit message to add a reference of RK3399 official datesheet[1] which says PCIe on RK3399 should only support 2.5GT/s? [1]https://opensource.rock-chips.com/images/d/d7/Rockchip_RK3399_Datasheet_V2.1-20200323.pdf > Geraldo Nascimento (3): > PCI: rockchip: warn of danger of 5.0 GT/s speeds > PCI: rockchip-host: comment danger of 5.0 GT/s speed > arm64: dts: rockchip: drop max-link-speed = <2> in helios64 PCIe > > arch/arm64/boot/dts/rockchip/rk3399-kobol-helios64.dts | 1 - > drivers/pci/controller/pcie-rockchip-host.c | 5 +++++ > drivers/pci/controller/pcie-rockchip.c | 8 ++++++-- > 3 files changed, 11 insertions(+), 3 deletions(-) >
Hello Shawn and Geraldo, On Monday, November 17, 2025 04:42 CET, Shawn Lin <shawn.lin@rock-chips.com> wrote: > 在 2025/11/15 星期六 17:10, Geraldo Nascimento 写道: > > In recent interactions with Shawn Lin from Rockchip it came to my > > attention there's an unknown errata regarding 5.0 GT/s operational > > speed of their PCIe core. According to Shawn there's grave danger > > even if the odds are low. To contain any damage, let's cover the > > remaining corner-cases where the default would lead to 5.0 GT/s > > operation as well as add a comment to Root Complex driver core, > > documenting this danger. > > I'm not sure just adding a warn would be a good choice. Could we totally > force to use gen1 and add a warn if trying to use Gen2. I think that forcing 2.5 GT/s with an appropriate warning message is a good idea. That would be like some quirk that gets applied automatically, to prevent data corruption, while warning people who attempt to "overclock" the PCIe interface. > Meanwhile amend the commit message to add a reference > of RK3399 official datesheet[1] which says PCIe on RK3399 should only > support 2.5GT/s? > > [1]https://opensource.rock-chips.com/images/d/d7/Rockchip_RK3399_Datasheet_V2.1-20200323.pdf Also, rewording the patch summary as follows below may be good, because that would provide more details: PCI: rockchip: Warn about Gen2 5.0 GT/s on RK3399 being unsafe Or, if we'll go with the automatic downgrading, like this: PCI: rockchip: Limit RK3399 to Gen1 2.5 GT/s to prevent breakage
On Mon, Nov 17, 2025 at 04:57:11AM +0100, Dragan Simic wrote: > Hello Shawn and Geraldo, > > On Monday, November 17, 2025 04:42 CET, Shawn Lin <shawn.lin@rock-chips.com> wrote: > > 在 2025/11/15 星期六 17:10, Geraldo Nascimento 写道: > > > In recent interactions with Shawn Lin from Rockchip it came to my > > > attention there's an unknown errata regarding 5.0 GT/s operational > > > speed of their PCIe core. According to Shawn there's grave danger > > > even if the odds are low. To contain any damage, let's cover the > > > remaining corner-cases where the default would lead to 5.0 GT/s > > > operation as well as add a comment to Root Complex driver core, > > > documenting this danger. > > > > I'm not sure just adding a warn would be a good choice. Could we totally > > force to use gen1 and add a warn if trying to use Gen2. > > I think that forcing 2.5 GT/s with an appropriate warning message > is a good idea. That would be like some quirk that gets applied > automatically, to prevent data corruption, while warning people > who attempt to "overclock" the PCIe interface. Hi Shawn and Dragan, Alright, I'll send v2 with this suggestion in mind. So that driving the core at 5.0 GT/s will require patching and compiling own kernel. > > > Meanwhile amend the commit message to add a reference > > of RK3399 official datesheet[1] which says PCIe on RK3399 should only > > support 2.5GT/s? > > > > [1]https://opensource.rock-chips.com/images/d/d7/Rockchip_RK3399_Datasheet_V2.1-20200323.pdf > Shawn, URLs have the bad habit of changing or simply disappearing, so I don't think it's a good idea to put URL in the commit message. Also, the datasheet just mentions that RK3399 supports only 2.5 GT/s, it does not mention possible damage from driving the core at 5.0 GT/s. > Also, rewording the patch summary as follows below may be good, > because that would provide more details: > > PCI: rockchip: Warn about Gen2 5.0 GT/s on RK3399 being unsafe > > Or, if we'll go with the automatic downgrading, like this: > > PCI: rockchip: Limit RK3399 to Gen1 2.5 GT/s to prevent breakage > Dragan, these are good ones, thanks. Though I think I'll omit Gen1/Gen2 wording since I know how much Bjorn dislikes those terms. Thanks, Geraldo Nascimento
Hello Geraldo, On Monday, November 17, 2025 08:02 CET, Geraldo Nascimento <geraldogabriel@gmail.com> wrote: > On Mon, Nov 17, 2025 at 04:57:11AM +0100, Dragan Simic wrote: > > On Monday, November 17, 2025 04:42 CET, Shawn Lin <shawn.lin@rock-chips.com> wrote: > > > 在 2025/11/15 星期六 17:10, Geraldo Nascimento 写道: > > > > In recent interactions with Shawn Lin from Rockchip it came to my > > > > attention there's an unknown errata regarding 5.0 GT/s operational > > > > speed of their PCIe core. According to Shawn there's grave danger > > > > even if the odds are low. To contain any damage, let's cover the > > > > remaining corner-cases where the default would lead to 5.0 GT/s > > > > operation as well as add a comment to Root Complex driver core, > > > > documenting this danger. > > > > > > I'm not sure just adding a warn would be a good choice. Could we totally > > > force to use gen1 and add a warn if trying to use Gen2. > > > > I think that forcing 2.5 GT/s with an appropriate warning message > > is a good idea. That would be like some quirk that gets applied > > automatically, to prevent data corruption, while warning people > > who attempt to "overclock" the PCIe interface. > > Alright, I'll send v2 with this suggestion in mind. So that driving the > core at 5.0 GT/s will require patching and compiling own kernel. > > > > Meanwhile amend the commit message to add a reference > > > of RK3399 official datesheet[1] which says PCIe on RK3399 should only > > > support 2.5GT/s? > > > > > > [1]https://opensource.rock-chips.com/images/d/d7/Rockchip_RK3399_Datasheet_V2.1-20200323.pdf > > > > Shawn, URLs have the bad habit of changing or simply disappearing, so I > don't think it's a good idea to put URL in the commit message. Ah, it's actually perfectly fine, there's always Wayback Machibe to rescue references/URLs that may disappear over time. Wikipedia relies heavily on exactly that mechanism, for example. [2] https://web.archive.org/ > Also, the datasheet just mentions that RK3399 supports only 2.5 GT/s, > it does not mention possible damage from driving the core at 5.0 GT/s. True, but having an additional reference doesn't hurt, especially because the revision history mentions, albeit vaguely, an update to the PCIe specification in version 1.1.of the datasheet. Though, if we had the version 1.0 publicly available for comparison, that would've been much better. :) > > Also, rewording the patch summary as follows below may be good, > > because that would provide more details: > > > > PCI: rockchip: Warn about Gen2 5.0 GT/s on RK3399 being unsafe > > > > Or, if we'll go with the automatic downgrading, like this: > > > > PCI: rockchip: Limit RK3399 to Gen1 2.5 GT/s to prevent breakage > > Dragan, these are good ones, thanks. Though I think I'll omit Gen1/Gen2 > wording since I know how much Bjorn dislikes those terms. I'm glad that you like those, and I also thought about not including the GenX parts, because they're basically a bit redundant. :)
© 2016 - 2026 Red Hat, Inc.