> +enum devlink_autoneg_protocol { > + DEVLINK_AUTONEG_PROTOCOL_IEEE8023BY_BAM, > + DEVLINK_AUTONEG_PROTOCOL_IEEE8023BY_CONSORTIUM, > + DEVLINK_AUTONEG_PROTOCOL_IEEE8023BY, > + DEVLINK_AUTONEG_PROTOCOL_BAM, /* Broadcom > Autoneg Mode */ > + DEVLINK_AUTONEG_PROTOCOL_CONSORTIUM, /* > Consortium Autoneg Mode */ > +};
Wouldn't adding BAM as a 'generic' mode of operation be like adding non-consortium speeds to ethtool API? [I profess ignorance in this area; For all I know it can be a widely accepted industry standard]