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 2CA22A09FD; Sun, 20 Dec 2020 19:43:06 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 02B90CB6B; Sun, 20 Dec 2020 19:43:04 +0100 (CET) Received: from mail-ej1-f53.google.com (mail-ej1-f53.google.com [209.85.218.53]) by dpdk.org (Postfix) with ESMTP id 735B8CB2F for ; Sun, 20 Dec 2020 19:43:02 +0100 (CET) Received: by mail-ej1-f53.google.com with SMTP id n26so10476426eju.6 for ; Sun, 20 Dec 2020 10:43:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=gq+tKWVCtQJUxwZAQVM+xHz9Vm4uDu9g5Ln9Ft2GAts=; b=KFfnBuvJ2nzAP77faveugKNqsm5qQZfR6sOKXX5bHkxTAVjw3w1k3VZo3y6DOi2psv UrprbHdwaGwQmVMYKvCf4wfdPDu7h61RfaMd3qWD02kxnsWpRYxbiB4cctG1PdwoXDmh MsFkZq2LdvGY2UpgrAS0WQIfafZOkHLJFYnxBB73HTCYyOJNMcobHFJ/XPIwpNyV0CA9 mIRWHuADSI+dwZEP+tUjODD+VjA1mKsKZYD9USvsLt2d20cexS1mVPG1tyuCkAMxIE+K mh6toyXV8BHwj/BLBkz/2cViSSEuwgtVXXIoVGAAbAMfCs3ZA1b+AO4f0bMzMQf2iDk7 Ovqg== 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:content-transfer-encoding; bh=gq+tKWVCtQJUxwZAQVM+xHz9Vm4uDu9g5Ln9Ft2GAts=; b=p+YQ4NOWivwTrjJgAm2hGT2nvBIOW1UoWCXMEiMAOW4rwTaaNersJD41nQ1JdIEnDH kY8OCI9CbrPGBO43EEWlzj5CYYCVDkNPWgA/KydfQyvqgFLsLwJ3xFmYDmjyLXg+ym9J iN2Vix3IjKv1bgzBRoYvhIyOrUjoEhK7tmmVS0zE3lDXHH39F9ycMjZHD9Cpt3kAvqkB jRUApOC1ECxdanqy5BIGpNCkFiHPA3cjrxBzy576f1cHYT3HFxKsKmUQlVVn5JhpPJHc ObOx+YZhzijRd1iWWwIzECkH46t8GXQbEFmAkdXgRBQNR4FbmtMXztynOUWpED1PFNp4 Eppg== X-Gm-Message-State: AOAM530rQOUxzm0GXXBceWuWqRR/lfXiBTwfOxsC1E+pAmN/V+Hyzwzy Yo6xkH/pF1+/HV7wg8g1/ny4WD2fYnTu7PUqtzGPkHG1wmNZ2g== X-Google-Smtp-Source: ABdhPJzufdnv+Zmx1XsbzcN6AAny4OsEpYbBY7AyqN9wPuPd9AX0DTbIR+PPJKDfJP/34qBIkWltcjsbJ+rrSKD5L7Y= X-Received: by 2002:a17:906:d28f:: with SMTP id ay15mr12232662ejb.327.1608489780870; Sun, 20 Dec 2020 10:43:00 -0800 (PST) MIME-Version: 1.0 References: <20201218190738.12669-1-ajit.khaparde@broadcom.com> In-Reply-To: <20201218190738.12669-1-ajit.khaparde@broadcom.com> From: Ajit Khaparde Date: Sun, 20 Dec 2020 10:42:49 -0800 Message-ID: To: dev@dpdk.org Cc: Lance Richardson Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH] net/bnxt: remove support for some PCI IDs 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 Fri, Dec 18, 2020 at 11:07 AM Ajit Khaparde wro= te: > > As announced the deprecation notice during the 20.11 release, > remove support for NetXtreme devices belonging to BCM573xx and > BCM5740x families. Specifically the support for the following Broadcom > PCI device IDs: 0x16c8, 0x16c9, 0x16ca, 0x16ce, 0x16cf, 0x16df, 0x16d0, > 0x16d1, 0x16d2, 0x16d4, 0x16d5, 0x16e7, 0x16e8, 0x16e9 has been removed. > > Deprecation notice has been removed and release notes for 21.02 has > been updated accordingly. > > Signed-off-by: Ajit Khaparde > Reviewed-by: Lance Richardson Patch applied to dpdk-next-net-brcm. > > --- > doc/guides/nics/bnxt.rst | 2 +- > doc/guides/rel_notes/deprecation.rst | 6 ------ > doc/guides/rel_notes/release_21_02.rst | 4 ++++ > drivers/net/bnxt/bnxt.h | 14 -------------- > drivers/net/bnxt/bnxt_ethdev.c | 14 -------------- > 5 files changed, 5 insertions(+), 35 deletions(-) > > diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst > index 4cd8475e8..801ec3ca5 100644 > --- a/doc/guides/nics/bnxt.rst > +++ b/doc/guides/nics/bnxt.rst > @@ -6,7 +6,7 @@ BNXT Poll Mode Driver > > The Broadcom BNXT PMD (**librte_net_bnxt**) implements support for adapt= ers > based on Ethernet controllers and SoCs belonging to the Broadcom > -BCM574XX/BCM575XX NetXtreme-E=C2=AE Family of Ethernet Network Controlle= rs, > +BCM5741X/BCM575XX NetXtreme-E=C2=AE Family of Ethernet Network Controlle= rs, > the Broadcom BCM588XX Stingray Family of Smart NIC Adapters, and the Bro= adcom > StrataGX=C2=AE BCM5873X Series of Communications Processors. > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/= deprecation.rst > index 2f498a0be..0fdaf37ed 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -121,12 +121,6 @@ Deprecation Notices > will be limited to maximum 256 queues. > Also compile time flag ``RTE_ETHDEV_QUEUE_STAT_CNTRS`` will be removed= . > > -* Broadcom bnxt PMD: NetXtreme devices belonging to the ``BCM573xx and > - BCM5740x`` families will no longer be supported as of DPDK 21.02. > - Specifically the support for the following Broadcom PCI IDs will be re= moved > - from the release: ``0x16c8, 0x16c9, 0x16ca, 0x16ce, 0x16cf, 0x16df,`` > - ``0x16d0, 0x16d1, 0x16d2, 0x16d4, 0x16d5, 0x16e7, 0x16e8, 0x16e9``. > - > * sched: To allow more traffic classes, flexible mapping of pipe queues = to > traffic classes, and subport level configuration of pipes and queues > changes will be made to macros, data structures and API functions defi= ned > diff --git a/doc/guides/rel_notes/release_21_02.rst b/doc/guides/rel_note= s/release_21_02.rst > index 638f98168..706cbf8f0 100644 > --- a/doc/guides/rel_notes/release_21_02.rst > +++ b/doc/guides/rel_notes/release_21_02.rst > @@ -68,6 +68,10 @@ Removed Items > Also, make sure to start the actual text at the margin. > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D > > +* Removed support for NetXtreme devices belonging to ``BCM573xx and > + BCM5740x`` families. Specifically the support for the following Broadc= om > + PCI device IDs ``0x16c8, 0x16c9, 0x16ca, 0x16ce, 0x16cf, 0x16df, 0x16d= 0,`` > + ``0x16d1, 0x16d2, 0x16d4, 0x16d5, 0x16e7, 0x16e8, 0x16e9`` has been re= moved. > > API Changes > ----------- > diff --git a/drivers/net/bnxt/bnxt.h b/drivers/net/bnxt/bnxt.h > index ad7d59458..8374e9fad 100644 > --- a/drivers/net/bnxt/bnxt.h > +++ b/drivers/net/bnxt/bnxt.h > @@ -33,35 +33,21 @@ > #define BROADCOM_DEV_ID_STRATUS_NIC_VF2 0x1609 > #define BROADCOM_DEV_ID_STRATUS_NIC 0x1614 > #define BROADCOM_DEV_ID_57414_VF 0x16c1 > -#define BROADCOM_DEV_ID_57301 0x16c8 > -#define BROADCOM_DEV_ID_57302 0x16c9 > -#define BROADCOM_DEV_ID_57304_PF 0x16ca > #define BROADCOM_DEV_ID_57304_VF 0x16cb > #define BROADCOM_DEV_ID_57417_MF 0x16cc > #define BROADCOM_DEV_ID_NS2 0x16cd > -#define BROADCOM_DEV_ID_57311 0x16ce > -#define BROADCOM_DEV_ID_57312 0x16cf > -#define BROADCOM_DEV_ID_57402 0x16d0 > -#define BROADCOM_DEV_ID_57404 0x16d1 > -#define BROADCOM_DEV_ID_57406_PF 0x16d2 > #define BROADCOM_DEV_ID_57406_VF 0x16d3 > -#define BROADCOM_DEV_ID_57402_MF 0x16d4 > -#define BROADCOM_DEV_ID_57407_RJ45 0x16d5 > #define BROADCOM_DEV_ID_57412 0x16d6 > #define BROADCOM_DEV_ID_57414 0x16d7 > #define BROADCOM_DEV_ID_57416_RJ45 0x16d8 > #define BROADCOM_DEV_ID_57417_RJ45 0x16d9 > #define BROADCOM_DEV_ID_5741X_VF 0x16dc > #define BROADCOM_DEV_ID_57412_MF 0x16de > -#define BROADCOM_DEV_ID_57314 0x16df > #define BROADCOM_DEV_ID_57317_RJ45 0x16e0 > #define BROADCOM_DEV_ID_5731X_VF 0x16e1 > #define BROADCOM_DEV_ID_57417_SFP 0x16e2 > #define BROADCOM_DEV_ID_57416_SFP 0x16e3 > #define BROADCOM_DEV_ID_57317_SFP 0x16e4 > -#define BROADCOM_DEV_ID_57404_MF 0x16e7 > -#define BROADCOM_DEV_ID_57406_MF 0x16e8 > -#define BROADCOM_DEV_ID_57407_SFP 0x16e9 > #define BROADCOM_DEV_ID_57407_MF 0x16ea > #define BROADCOM_DEV_ID_57414_MF 0x16ec > #define BROADCOM_DEV_ID_57416_MF 0x16ee > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethde= v.c > index a3035def5..0788d263d 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -47,27 +47,13 @@ static const struct rte_pci_id bnxt_pci_id_map[] =3D = { > BROADCOM_DEV_ID_STRATUS_NIC_VF2) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_STRATUS_= NIC) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57414_VF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57301) }= , > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57302) }= , > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57304_PF= ) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57304_VF= ) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_NS2) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57402) }= , > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57404) }= , > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57406_PF= ) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57406_VF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57402_MF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57407_RJ= 45) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57404_MF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57406_MF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57407_SF= P) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57407_MF= ) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_5741X_VF= ) }, > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_5731X_VF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57314) }= , > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57417_MF= ) }, > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57311) }= , > - { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57312) }= , > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57412) }= , > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57414) }= , > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, BROADCOM_DEV_ID_57416_RJ= 45) }, > -- > 2.21.1 (Apple Git-122.3) > --=20 Ajit Khaparde