From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-oi0-f65.google.com (mail-oi0-f65.google.com [209.85.218.65]) by dpdk.org (Postfix) with ESMTP id ED619CF68 for ; Tue, 17 Apr 2018 20:25:41 +0200 (CEST) Received: by mail-oi0-f65.google.com with SMTP id p62-v6so2562505oie.10 for ; Tue, 17 Apr 2018 11:25:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=gBw1+6dimzQ6mvkYJ9u2nVkbHO/E/x0j55d1i82fZUk=; b=VIrWYfRC7Ta8TkVN3ZCLw9N3A3UmgDhepCJuYfZdEeCrPVssHvQXxq58bkexFa6Y94 RCg2J0KCKGSx1AJ8FAJvBv96S/dau1vuRdCNLBUyheFPrtbK3E3eG9MV08WS71q6kGVP M+eCEQe0Tt9j03BAwwob86oUtkxeuKr7wadis= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=gBw1+6dimzQ6mvkYJ9u2nVkbHO/E/x0j55d1i82fZUk=; b=QkkwDCczoX64GJW61uWlLDAjjaV9V6MHbyTIYZypH1iLnaCqxI+agW63cnmbHYn9yo rhq/mEWfoUJwGSJ/LH1lR+ZpJHKogXNqGptBwSMkOVLl7Lb+6xBUSoz0827DL/wwJd6U 1Ov2OspKlIMmAPqY6VS/nH0gSC25AsoLWoybo9Kbvl12Rw6fJjuVs3rr5QCWrNaDJV7z 7jBJrHNbK3qou54vF0iq9LEEX+pRTPePwOKlePC4/uu/lYDsoEkAys6zCcFO5lgtNwcb wGzh6oEDAm4hJtnZLQF76o291SDWuuhMxnE9GZTQNWDHF2SisK+NXTN23xyzRALfSYAn o2CQ== X-Gm-Message-State: ALQs6tBzRRsGrZ3shpgxDwMark8lIFoxyGVZx2Zt8//pVFUqlp0POcCo Td1Z4O1iXa9mxiQKSMSgKQNjlMvXRyrctFUqy+kgtw== X-Google-Smtp-Source: AIpwx4+oGCwjLckyDGNH05bP9ZCF/VKyBXJMGP9CBLx3pkiquQz2nYY81ZAjqHuy/78jR1gf0X04yKpbv8E0bvsJTDs= X-Received: by 2002:aca:1c9:: with SMTP id 192-v6mr1938000oib.175.1523989541287; Tue, 17 Apr 2018 11:25:41 -0700 (PDT) MIME-Version: 1.0 Received: by 10.201.48.41 with HTTP; Tue, 17 Apr 2018 11:25:00 -0700 (PDT) In-Reply-To: <8c22cafe-6095-8e64-c5cd-21a9135dc8d3@intel.com> References: <20180417011126.12622-1-ajit.khaparde@broadcom.com> <20180417011126.12622-15-ajit.khaparde@broadcom.com> <8c22cafe-6095-8e64-c5cd-21a9135dc8d3@intel.com> From: Ajit Khaparde Date: Tue, 17 Apr 2018 11:25:00 -0700 Message-ID: To: Ferruh Yigit Cc: dev@dpdk.org, Somnath Kotur Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH 14/14] bnxt: add device ID for Stratus VF 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: , X-List-Received-Date: Tue, 17 Apr 2018 18:25:42 -0000 =E2=80=8BNo Ferruh. It is not a new product family=E2=80=8B. Just adding a device ID (for a VF) for a skew of already existing product. So a documentation update is not required. Thanks Ajit On Tue, Apr 17, 2018 at 11:20 AM, Ferruh Yigit wrote: > On 4/17/2018 2:11 AM, Ajit Khaparde wrote: > > From: Somnath Kotur > > > > This is adding new device id to driver. > Does it require any change in documentation? > Do you want to announce new hw support in release notes? > > > Fixes: 1cd45aeb3270 ("net/bnxt: support Stratus VF device") > > Cc: ajit.khaparde@broadcom.com > > Signed-off-by: Somnath Kotur > > --- > > drivers/net/bnxt/bnxt_ethdev.c | 10 +++++++--- > > 1 file changed, 7 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ > ethdev.c > > index 3cf845089..7632c326b 100644 > > --- a/drivers/net/bnxt/bnxt_ethdev.c > > +++ b/drivers/net/bnxt/bnxt_ethdev.c > > @@ -34,7 +34,8 @@ int bnxt_logtype_driver; > > > > #define PCI_VENDOR_ID_BROADCOM 0x14E4 > > > > -#define BROADCOM_DEV_ID_STRATUS_NIC_VF 0x1609 > > +#define BROADCOM_DEV_ID_STRATUS_NIC_VF1 0x1606 > > +#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 > > @@ -75,7 +76,9 @@ int bnxt_logtype_driver; > > > > static const struct rte_pci_id bnxt_pci_id_map[] =3D { > > { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, > > - BROADCOM_DEV_ID_STRATUS_NIC_VF) }, > > + BROADCOM_DEV_ID_STRATUS_NIC_VF1) }, > > + { RTE_PCI_DEVICE(PCI_VENDOR_ID_BROADCOM, > > + 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) }= , > > @@ -3063,7 +3066,8 @@ static bool bnxt_vf_pciid(uint16_t id) > > id =3D=3D BROADCOM_DEV_ID_5731X_VF || > > id =3D=3D BROADCOM_DEV_ID_5741X_VF || > > id =3D=3D BROADCOM_DEV_ID_57414_VF || > > - id =3D=3D BROADCOM_DEV_ID_STRATUS_NIC_VF) > > + id =3D=3D BROADCOM_DEV_ID_STRATUS_NIC_VF1 || > > + id =3D=3D BROADCOM_DEV_ID_STRATUS_NIC_VF2) > > return true; > > return false; > > } > > > >