Hi Erwan,

On Tue, 9 Dec 2025 10:43:58 +0100, Erwan Velu wrote:
> >
> >
> > +               case 0xC5: /* EDSFF */
> > +               case 0xC6: /* EDSFF */
> >                         pr_attr("ID", "%u", code1);
> >                         break;
>
> Depending on Slot Length the ports will become E1.S or E1.L for C5 or
> E3.S/E3.L for C6.
> Could it be possible to add this information, as most of products uses this
> definition to describe products/ports.

Thanks for your feedback. The comments above only document the port
families for which printing the port ID as an 8-bit decimal number is
appropriate. When decoding the port type itself, the displayed strings
are more detailed. For 0xC5 and 0xC6, they are respectively:

                "EDSFF E1",
                "EDSFF E3" /* 0xC6 */

The port length is displayed separately (same as other port types), so
for example the output for an E1.S port would look like:

Handle 0x00BD, DMI type 9, 23 bytes
System Slot Information
        Designation: EDSFF port 1
        Type: EDSFF E1
        (...)
        Length: Short
        ID: 1

Is that OK with you, or would you prefer a different presentation?

-- 
Jean Delvare
SUSE L3 Support

Reply via email to