diff mbox series

[4.9,26/29] bnxt_en: Remove the setting of dev_port.

Message ID	20210608175928.669082541@linuxfoundation.org
State	New
Headers	show Return-Path: <stable-owner@kernel.org> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, stable@vger.kernel.org, Michael Chan <michael.chan@broadcom.com>, "David S. Miller" <davem@davemloft.net>, Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Subject: [PATCH 4.9 26/29] bnxt_en: Remove the setting of dev_port. Date: Tue, 8 Jun 2021 20:27:20 +0200 Message-Id: <20210608175928.669082541@linuxfoundation.org> In-Reply-To: <20210608175927.821075974@linuxfoundation.org> References: <20210608175927.821075974@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	None \| expand [4.9,02/29] efi: Allow EFI_MEMORY_XP and EFI_MEMORY_RO both to be cleared [4.9,03/29] efi: cper: fix snprintf() use in cper_dimm_err_location() [4.9,04/29] vfio/pci: Fix error return code in vfio_ecap_init() [4.9,05/29] vfio/pci: zap_vma_ptes() needs MMU [4.9,06/29] vfio/platform: fix module_put call in error flow [4.9,07/29] ipvs: ignore IP_VS_SVC_F_HASHED flag when adding service [4.9,08/29] HID: pidff: fix error return code in hid_pidff_init() [4.9,09/29] HID: i2c-hid: fix format string mismatch [4.9,10/29] netfilter: nfnetlink_cthelper: hit EBUSY on updates if size mismatches [4.9,11/29] ieee802154: fix error return code in ieee802154_add_iface() [4.9,12/29] ieee802154: fix error return code in ieee802154_llsec_getparams() [4.9,13/29] Bluetooth: fix the erroneous flush_work() order [4.9,14/29] Bluetooth: use correct lock to prevent UAF of hdev object [4.9,15/29] net: caif: added cfserl_release function [4.9,16/29] net: caif: add proper error handling [4.9,17/29] net: caif: fix memory leak in caif_device_notify [4.9,18/29] net: caif: fix memory leak in cfusbl_device_notify [4.9,19/29] ALSA: timer: Fix master timer notification [4.9,20/29] ext4: fix bug on in ext4_es_cache_extent as ext4_split_extent_at failed [4.9,21/29] pid: take a reference when initializing `cad_pid` [4.9,22/29] ocfs2: fix data corruption by fallocate [4.9,23/29] nfc: fix NULL ptr dereference in llcp_sock_getname() after failed connect [4.9,24/29] btrfs: fix error handling in btrfs_del_csums [4.9,25/29] btrfs: fixup error handling in fixup_inode_link_counts [4.9,26/29] bnxt_en: Remove the setting of dev_port. [4.9,27/29] KVM: SVM: Truncate GPR value for DR and CR accesses in !64-bit mode [4.9,28/29] arm64: Remove unimplemented syscall log message [4.9,29/29] xen-pciback: redo VF placement in the virtual topology

Commit Message

Greg KH June 8, 2021, 6:27 p.m. UTC

From: Michael Chan <michael.chan@broadcom.com>

commit 1d86859fdf31a0d50cc82b5d0d6bfb5fe98f6c00 upstream.

The dev_port is meant to distinguish the network ports belonging to
the same PCI function.  Our devices only have one network port
associated with each PCI function and so we should not set it for
correctness.

Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---
 drivers/net/ethernet/broadcom/bnxt/bnxt.c |    1 -
 1 file changed, 1 deletion(-)

diff mbox series

Patch

--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
@@ -4262,7 +4262,6 @@  int bnxt_hwrm_func_qcaps(struct bnxt *bp
 
 		pf->fw_fid = le16_to_cpu(resp->fid);
 		pf->port_id = le16_to_cpu(resp->port_id);
-		bp->dev->dev_port = pf->port_id;
 		memcpy(pf->mac_addr, resp->mac_address, ETH_ALEN);
 		memcpy(bp->dev->dev_addr, pf->mac_addr, ETH_ALEN);
 		pf->max_rsscos_ctxs = le16_to_cpu(resp->max_rsscos_ctx);