From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 532F7A00C3; Thu, 14 May 2020 02:05:39 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C08621D647; Thu, 14 May 2020 02:05:38 +0200 (CEST) Received: from mail-oi1-f195.google.com (mail-oi1-f195.google.com [209.85.167.195]) by dpdk.org (Postfix) with ESMTP id D99531D643 for ; Thu, 14 May 2020 02:05:36 +0200 (CEST) Received: by mail-oi1-f195.google.com with SMTP id a2so22887733oia.11 for ; Wed, 13 May 2020 17:05:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=WqZ2LU2USxKN45YZwgELOildhdItYzvC70QOuZiJ5kk=; b=E1eIgbcmoKpOOXvoVeX9e/1YkdmWWYix3DOlLNbl2tPwcvFEqPiJsIC7Ibbt2ATajx W7aT1a2GGtIo3U1tc9IRzMAA+GUgBitYGVrHURM0/B6/9WxbPUjyVQfdD1JTWVZsWFge 5si58M/gk9M86dvBX4tTsCE4H0hticN87kQRA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=WqZ2LU2USxKN45YZwgELOildhdItYzvC70QOuZiJ5kk=; b=HVCIetaKzBjvmS8w4rGktiiV/GQQa6A6CGSTXDs0wObMuAcfCOxNb7FWNtnxPHvmav 0A2FSfsRkd8Bq1cmfpFxExJbzEDgR9SnRYy3aPjiTR0BCKI8fcSquBXRAGYKSAYQWqYr De5lzWi6SCktt3H80tKnOdKlFYx7CGWffIVQk5usEH10sLnaTrwFIETyEiOzd5zFcXSJ gnekHdnQVfV0vp7W/FxSZfkH9O1OUJBMXmA8dCcCrqbtKZkf2VEYPKMFyIUqgFpCriH2 E+DkAITCg29CsmiwMY2kgAM8DQ1eYoBcZjQZ143YIYeN8O3ZMgvemzTPpirg/erTgue3 Ua4A== X-Gm-Message-State: AGi0PuahADbZUEcRV03sHTyCuzzOetCjfKLvOkelkS3YxfcP43DQ9p4T 5pQLSydPIItUDqXWNiKpP1/GezIPwkYM+wF1LRsVKA== X-Google-Smtp-Source: APiQypIoDfFmS89OQduz7CA0ANDbvUygEtUt9vYUARydOKwjUAKCkTwtlQ5vPOx7/k4dj2JoMcUF0hIF02LY7t8E2Y0= X-Received: by 2002:aca:5583:: with SMTP id j125mr29064187oib.179.1589414735929; Wed, 13 May 2020 17:05:35 -0700 (PDT) MIME-Version: 1.0 References: <20200513045125.27694-1-kalesh-anakkur.purayil@broadcom.com> In-Reply-To: <20200513045125.27694-1-kalesh-anakkur.purayil@broadcom.com> From: Ajit Khaparde Date: Wed, 13 May 2020 17:05:19 -0700 Message-ID: To: Kalesh A P Cc: dpdk-dev , Ferruh Yigit Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH] net/bnxt: add support for 200G link speed X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, May 12, 2020 at 9:35 PM Kalesh A P < kalesh-anakkur.purayil@broadcom.com> wrote: > From: Kalesh AP > > When the driver is loaded on a 200G NIC, the port speed is not > displayed correctly. Parse the 200G speed before displaying it. > > Signed-off-by: Kalesh AP > Reviewed-by: Somnath Kotur > Reviewed-by: Ajit Khaparde > Applied to dpdk-next-net-brcm. Thanks > --- > drivers/net/bnxt/bnxt_ethdev.c | 2 ++ > drivers/net/bnxt/bnxt_hwrm.c | 12 +++++++++++- > 2 files changed, 13 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c > b/drivers/net/bnxt/bnxt_ethdev.c > index ee0550b..2ba9d6b 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -693,6 +693,8 @@ static uint32_t bnxt_get_speed_capabilities(struct > bnxt *bp) > speed_capa |= ETH_LINK_SPEED_50G; > if (link_speed & HWRM_PORT_PHY_QCFG_OUTPUT_SUPPORT_SPEEDS_100GB) > speed_capa |= ETH_LINK_SPEED_100G; > + if (link_speed & HWRM_PORT_PHY_QCFG_OUTPUT_SUPPORT_SPEEDS_200GB) > + speed_capa |= ETH_LINK_SPEED_200G; > > if (bp->link_info.auto_mode == > HWRM_PORT_PHY_QCFG_OUTPUT_AUTO_MODE_NONE) > speed_capa |= ETH_LINK_SPEED_FIXED; > diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c > index ef65107..77a9110 100644 > --- a/drivers/net/bnxt/bnxt_hwrm.c > +++ b/drivers/net/bnxt/bnxt_hwrm.c > @@ -2762,6 +2762,10 @@ static uint16_t bnxt_parse_eth_link_speed(uint32_t > conf_link_speed) > eth_link_speed = > HWRM_PORT_PHY_CFG_INPUT_FORCE_LINK_SPEED_100GB; > break; > + case ETH_LINK_SPEED_200G: > + eth_link_speed = > + HWRM_PORT_PHY_CFG_INPUT_FORCE_LINK_SPEED_200GB; > + break; > default: > PMD_DRV_LOG(ERR, > "Unsupported link speed %d; default to AUTO\n", > @@ -2774,7 +2778,8 @@ static uint16_t bnxt_parse_eth_link_speed(uint32_t > conf_link_speed) > #define BNXT_SUPPORTED_SPEEDS (ETH_LINK_SPEED_100M | > ETH_LINK_SPEED_100M_HD | \ > ETH_LINK_SPEED_1G | ETH_LINK_SPEED_2_5G | \ > ETH_LINK_SPEED_10G | ETH_LINK_SPEED_20G | > ETH_LINK_SPEED_25G | \ > - ETH_LINK_SPEED_40G | ETH_LINK_SPEED_50G | > ETH_LINK_SPEED_100G) > + ETH_LINK_SPEED_40G | ETH_LINK_SPEED_50G | \ > + ETH_LINK_SPEED_100G | ETH_LINK_SPEED_200G) > > static int bnxt_valid_link_speed(uint32_t link_speed, uint16_t port_id) > { > @@ -2840,6 +2845,8 @@ bnxt_parse_eth_link_speed_mask(struct bnxt *bp, > uint32_t link_speed) > ret |= HWRM_PORT_PHY_CFG_INPUT_AUTO_LINK_SPEED_MASK_50GB; > if (link_speed & ETH_LINK_SPEED_100G) > ret |= HWRM_PORT_PHY_CFG_INPUT_AUTO_LINK_SPEED_MASK_100GB; > + if (link_speed & ETH_LINK_SPEED_200G) > + ret |= HWRM_PORT_PHY_CFG_INPUT_AUTO_LINK_SPEED_MASK_200GB; > return ret; > } > > @@ -2875,6 +2882,9 @@ static uint32_t bnxt_parse_hw_link_speed(uint16_t > hw_link_speed) > case HWRM_PORT_PHY_QCFG_OUTPUT_LINK_SPEED_100GB: > eth_link_speed = ETH_SPEED_NUM_100G; > break; > + case HWRM_PORT_PHY_QCFG_OUTPUT_LINK_SPEED_200GB: > + eth_link_speed = ETH_SPEED_NUM_200G; > + break; > case HWRM_PORT_PHY_QCFG_OUTPUT_LINK_SPEED_2GB: > default: > PMD_DRV_LOG(ERR, "HWRM link speed %d not defined\n", > -- > 2.10.1 > >