Possible subject: dt-bindings: Add gen5, gen6 max-link-speed values On Wed, May 31, 2023 at 10:21:21AM +0100, Ben Dooks wrote: > Add updated max-link-speed values for newer generation PCIe link > speeds. > > Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > --- > Documentation/devicetree/bindings/pci/pci.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pci/pci.txt b/Documentation/devicetree/bindings/pci/pci.txt > index 6a8f2874a24d..56391e193fc4 100644 > --- a/Documentation/devicetree/bindings/pci/pci.txt > +++ b/Documentation/devicetree/bindings/pci/pci.txt > @@ -22,8 +22,9 @@ driver implementation may support the following properties: > If present this property specifies PCI gen for link capability. Host > drivers could add this as a strategy to avoid unnecessary operation for > unsupported link speed, for instance, trying to do training for > - unsupported link speed, etc. Must be '4' for gen4, '3' for gen3, '2' > - for gen2, and '1' for gen1. Any other values are invalid. > + unsupported link speed, etc. Must be '6' for gen6, '5' for gen5, > + '4' for gen4, '3' for gen3, '2' for gen2, and '1' for gen1. > + Any other values are invalid. I really wish we'd used values with some connection to the actual speed, e.g., "16" for 16 GT/s. These "gen X" values are a real hassle to convert back to the speed. But I guess that's water under the bridge. Maybe we should annotate the documentation here, though, e.g., '6' for gen6 (64.0 GT/s), '5' for gen5 (32.0 GT/s), ... Do I have that right? I don't see "gen5" etc in the specs themselves, so this is just based on Google. Bjorn
