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 7E82CA04B7 for ; Tue, 13 Oct 2020 23:37:12 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6F2F51D426; Tue, 13 Oct 2020 23:37:11 +0200 (CEST) Received: from mail-ot1-f68.google.com (mail-ot1-f68.google.com [209.85.210.68]) by dpdk.org (Postfix) with ESMTP id BCBFC1D426 for ; Tue, 13 Oct 2020 23:37:09 +0200 (CEST) Received: by mail-ot1-f68.google.com with SMTP id s66so1526164otb.2 for ; Tue, 13 Oct 2020 14:37:09 -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=n5RZBDYROqdtpfWsUPHQ0tQgz/uprn/2UJxbkaSebgY=; b=ZLJW6zAHIUR+bNNLidmDsKrVxY0o5m/nCaFmAUsomcMJzRRrYiPljoqJ6Yv0qzJttu 6EL5XRiaOqIB56h6pKGkCo09osb453IvOrybIsOeCJCvKxhLUDDdENjk8caRvpfEYu4W NodMb+J+q3RCnBtSe7lLs5KV1uN49KXeE7BRM= 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=n5RZBDYROqdtpfWsUPHQ0tQgz/uprn/2UJxbkaSebgY=; b=SlPRqFKxfEyB7abFt3L7vVMZ39EOtxi9soNfMUUXjOjvyz8FCLsms/kuUnVJrtFSmS 9z9Ld+zDaldZIv5Mp1ARo6exzFfAksL/+rW0Y/Hww8sf71libqBI1N9/vssC6L5C1RIg qsJgwbehr274PJf7I2VMTjArFkEa7/8dyLpbb6zMz4xYCBDQZUwEMmbnwnjpE7ei6X9X BMyOJX9HCdY+IoHCW4aAazQlwRcqpkk/rlKgRFASnxNMbX3I/SAVoBPuPm91J+RX6Arb banbjlW4Qwz2px1y6u/bLSeLrYd8Jqdl04aJ0YdsxZbBaNci9SFeklPzrekt6siIg5Y7 JmEQ== X-Gm-Message-State: AOAM530MTVcQ3bEHhIcbg8q4np8d/99ksAaizG7av2GNHEjYVZ6sqyZr S42U7vl7hD4JNiu+14RrqU94U1VabwTFgyH1F62Q/Q== X-Google-Smtp-Source: ABdhPJyOf1wpUVfs0gCivEGj2a8ioJ+IbmdgBEjfkAPRBGWlgaTn1uZ4fQFuonoi5GQzrbfNrjU8FCeWROpEb+zQ6PI= X-Received: by 2002:a9d:1909:: with SMTP id j9mr1228704ota.283.1602625027832; Tue, 13 Oct 2020 14:37:07 -0700 (PDT) MIME-Version: 1.0 References: <20201012224814.10472-1-rmody@marvell.com> In-Reply-To: From: Ajit Khaparde Date: Tue, 13 Oct 2020 14:36:51 -0700 Message-ID: To: Ferruh Yigit Cc: Jerin Jacob , Rasesh Mody , dpdklab , Jerin Jacob , dpdk-dev , dpdk stable , GR-Everest-DPDK-Dev Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH] net/bnx2x: fix to add QLogic vendor id X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Tue, Oct 13, 2020 at 1:43 PM Ferruh Yigit wrote: > > On 10/13/2020 7:28 PM, Jerin Jacob wrote: > > On Tue, Oct 13, 2020 at 4:18 AM Rasesh Mody wrote: > >> > >> Add QLogic vendor id support for BCM57840 device ids. > >> > >> Fixes: 9fb557035d90 ("bnx2x: enable PMD build") > >> Cc: stable@dpdk.org > >> > >> Reported-by: Souvik Dey > >> Signed-off-by: Rasesh Mody > > > > > > There was an error from CI on Broadcom CI, > > Since this patch is about Broadcom PMD fix, Do we need to worry about this? > > http://mails.dpdk.org/archives/test-report/2020-October/158246.html > > > > Errors shouldn't be related to this patch, so I think error can be ignored for > this patch. +1 > > But added Ajit & lab for the issue. I have seen this behavior earlier. It showed up randomly on some patches which are not even related to Broadcom PMD. I will try to keep an eye on it. > > > > >> --- > >> drivers/net/bnx2x/bnx2x_ethdev.c | 4 ++++ > >> 1 file changed, 4 insertions(+) > >> > >> diff --git a/drivers/net/bnx2x/bnx2x_ethdev.c b/drivers/net/bnx2x/bnx2x_ethdev.c > >> index 40225b2f4..ce9df8748 100644 > >> --- a/drivers/net/bnx2x/bnx2x_ethdev.c > >> +++ b/drivers/net/bnx2x/bnx2x_ethdev.c > >> @@ -17,6 +17,7 @@ > >> * The set of PCI devices this driver supports > >> */ > >> #define BROADCOM_PCI_VENDOR_ID 0x14E4 > >> +#define QLOGIC_PCI_VENDOR_ID 0x1077 > >> static const struct rte_pci_id pci_id_bnx2x_map[] = { > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57800) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57711) }, > >> @@ -24,11 +25,13 @@ static const struct rte_pci_id pci_id_bnx2x_map[] = { > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57811) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57840_OBS) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57840_4_10) }, > >> + { RTE_PCI_DEVICE(QLOGIC_PCI_VENDOR_ID, CHIP_NUM_57840_4_10) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57840_2_20) }, > >> #ifdef RTE_LIBRTE_BNX2X_MF_SUPPORT > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57810_MF) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57811_MF) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57840_MF) }, > >> + { RTE_PCI_DEVICE(QLOGIC_PCI_VENDOR_ID, CHIP_NUM_57840_MF) }, > >> #endif > >> { .vendor_id = 0, } > >> }; > >> @@ -38,6 +41,7 @@ static const struct rte_pci_id pci_id_bnx2xvf_map[] = { > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57810_VF) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57811_VF) }, > >> { RTE_PCI_DEVICE(BROADCOM_PCI_VENDOR_ID, CHIP_NUM_57840_VF) }, > >> + { RTE_PCI_DEVICE(QLOGIC_PCI_VENDOR_ID, CHIP_NUM_57840_VF) }, > >> { .vendor_id = 0, } > >> }; > >> > >> -- > >> 2.18.0 > >> >