On Sun, Aug 21, 2016 at 04:44:04PM -0700, Vidya Sagar Ravipati wrote:
> From: Vidya Sagar Ravipati <vi...@cumulusnetworks.com>
> 
> Enhancing link mode bits to support 25G/50G/100G
> for supported and advertised speed mode bits
> 
> Signed-off-by: Vidya Sagar Ravipati <vi...@cumulusnetworks.com>
> Acked-By: David Decotigny <de...@googlers.com>
> ---
>  ethtool.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/ethtool.c b/ethtool.c
> index 1d6564e..5c3c765 100644
> --- a/ethtool.c
> +++ b/ethtool.c
> @@ -512,6 +512,15 @@ static void init_global_link_mode_masks(void)
>               ETHTOOL_LINK_MODE_56000baseCR4_Full_BIT,
>               ETHTOOL_LINK_MODE_56000baseSR4_Full_BIT,
>               ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT,
> +             ETHTOOL_LINK_MODE_25000baseCR_Full_BIT,
> +             ETHTOOL_LINK_MODE_25000baseKR_Full_BIT,
> +             ETHTOOL_LINK_MODE_25000baseSR_Full_BIT,
> +             ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT,
> +             ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT,
> +             ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT,
> +             ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT,
> +             ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
> +             ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
>       };
>       static const enum ethtool_link_mode_bit_indices
>               additional_advertised_flags_bits[] = {
> @@ -632,6 +641,24 @@ static void dump_link_caps(const char *prefix, const 
> char *an_prefix,
>                 "56000baseSR4/Full" },
>               { 0, ETHTOOL_LINK_MODE_56000baseLR4_Full_BIT,
>                 "56000baseLR4/Full" },
> +             { 0, ETHTOOL_LINK_MODE_25000baseCR_Full_BIT,
> +               "25000baseCR/Full" },
> +             { 0, ETHTOOL_LINK_MODE_25000baseKR_Full_BIT,
> +               "25000baseKR/Full" },
> +             { 0, ETHTOOL_LINK_MODE_25000baseSR_Full_BIT,
> +               "25000baseSR/Full" },
> +             { 0, ETHTOOL_LINK_MODE_50000baseCR2_Full_BIT,
> +               "50000baseCR2/Full" },
> +             { 0, ETHTOOL_LINK_MODE_50000baseKR2_Full_BIT,
> +               "50000baseKR2/Full" },
> +             { 0, ETHTOOL_LINK_MODE_100000baseKR4_Full_BIT,
> +               "100000baseKR4/Full" },
> +             { 0, ETHTOOL_LINK_MODE_100000baseSR4_Full_BIT,
> +               "100000baseSR4/Full" },
> +             { 0, ETHTOOL_LINK_MODE_100000baseCR4_Full_BIT,
> +               "100000baseCR4/Full" },
> +             { 0, ETHTOOL_LINK_MODE_100000baseLR4_ER4_Full_BIT,
> +               "100000baseLR4_ER4/Full" },
>       };
>       int indent;
>       int did1, new_line_pend, i;

If my other comment wasn't clear, this is the patch I
to which I was suggesting that you could add support for
ETHTOOL_LINK_MODE_50000baseSR2_Full_BIT.

John
-- 
John W. Linville                Someday the world will need a hero, and you
linvi...@tuxdriver.com                  might be all we have.  Be ready.

Reply via email to