[PATCH 1/1] lspci: Add PCIe 6.0 data rate (64 GT/s) also to LnkCap2

[PATCH 1/1] lspci: Add PCIe 6.0 data rate (64 GT/s) also to LnkCap2

[Ilpo Järvinen]

While commit cecfc33d9c8a ("lspci: Add PCIe 6.0 data rate (64 GT/s)
support") added 64 GT/s support to some registers, LnkCap2 Supported
Link Speeds Vector was not included.

Add PCIe 6.0 data rate bit check also into
cap_express_link2_speed_cap().

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
 ls-caps.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ls-caps.c b/ls-caps.c
index fce9502bd29a..2c99812c4ed2 100644
--- a/ls-caps.c
+++ b/ls-caps.c
@@ -1191,8 +1191,10 @@ static const char *cap_express_link2_speed_cap(int vector)
    * permitted to skip support for any data rates between 2.5GT/s and the
    * highest supported rate.
    */
-  if (vector & 0x60)
+  if (vector & 0x40)
     return "RsvdP";
+  if (vector & 0x20)
+    return "2.5-64GT/s";
   if (vector & 0x10)
     return "2.5-32GT/s";
   if (vector & 0x08)
-- 
2.30.2

Re: [PATCH 1/1] lspci: Add PCIe 6.0 data rate (64 GT/s) also to LnkCap2

[Ilpo Järvinen]

[-- Attachment #1: Type: text/plain, Size: 1166 bytes --]

On Fri, 8 Dec 2023, Ilpo Järvinen wrote:

> While commit cecfc33d9c8a ("lspci: Add PCIe 6.0 data rate (64 GT/s)

Please disregard this one! I'll send v2 with the correct hash (I played 
with my .gitconfig to get it always 12 char hashes and while doing so,
I managed to copy the wrong hash here).

-- 
 i.


> support") added 64 GT/s support to some registers, LnkCap2 Supported
> Link Speeds Vector was not included.
> 
> Add PCIe 6.0 data rate bit check also into
> cap_express_link2_speed_cap().
> 
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
> ---
>  ls-caps.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/ls-caps.c b/ls-caps.c
> index fce9502bd29a..2c99812c4ed2 100644
> --- a/ls-caps.c
> +++ b/ls-caps.c
> @@ -1191,8 +1191,10 @@ static const char *cap_express_link2_speed_cap(int vector)
>     * permitted to skip support for any data rates between 2.5GT/s and the
>     * highest supported rate.
>     */
> -  if (vector & 0x60)
> +  if (vector & 0x40)
>      return "RsvdP";
> +  if (vector & 0x20)
> +    return "2.5-64GT/s";
>    if (vector & 0x10)
>      return "2.5-32GT/s";
>    if (vector & 0x08)
>