diff mbox series

[4.14,44/47] bnxt_en: Remove the setting of dev_port.

Message ID	20210608175931.936056894@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.14 44/47] bnxt_en: Remove the setting of dev_port. Date: Tue, 8 Jun 2021 20:27:27 +0200 Message-Id: <20210608175931.936056894@linuxfoundation.org> In-Reply-To: <20210608175930.477274100@linuxfoundation.org> References: <20210608175930.477274100@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.14,02/47] efi: Allow EFI_MEMORY_XP and EFI_MEMORY_RO both to be cleared [4.14,03/47] efi: cper: fix snprintf() use in cper_dimm_err_location() [4.14,04/47] vfio/pci: Fix error return code in vfio_ecap_init() [4.14,05/47] vfio/pci: zap_vma_ptes() needs MMU [4.14,06/47] vfio/platform: fix module_put call in error flow [4.14,07/47] ipvs: ignore IP_VS_SVC_F_HASHED flag when adding service [4.14,08/47] HID: pidff: fix error return code in hid_pidff_init() [4.14,09/47] HID: i2c-hid: fix format string mismatch [4.14,10/47] netfilter: nfnetlink_cthelper: hit EBUSY on updates if size mismatches [4.14,11/47] ieee802154: fix error return code in ieee802154_add_iface() [4.14,12/47] ieee802154: fix error return code in ieee802154_llsec_getparams() [4.14,13/47] Bluetooth: fix the erroneous flush_work() order [4.14,14/47] Bluetooth: use correct lock to prevent UAF of hdev object [4.14,15/47] net: caif: added cfserl_release function [4.14,16/47] net: caif: add proper error handling [4.14,17/47] net: caif: fix memory leak in caif_device_notify [4.14,18/47] net: caif: fix memory leak in cfusbl_device_notify [4.14,19/47] ALSA: timer: Fix master timer notification [4.14,20/47] ext4: fix bug on in ext4_es_cache_extent as ext4_split_extent_at failed [4.14,21/47] pid: take a reference when initializing `cad_pid` [4.14,22/47] ocfs2: fix data corruption by fallocate [4.14,23/47] nfc: fix NULL ptr dereference in llcp_sock_getname() after failed connect [4.14,24/47] btrfs: fix error handling in btrfs_del_csums [4.14,25/47] btrfs: fixup error handling in fixup_inode_link_counts [4.14,26/47] mm, hugetlb: fix simple resv_huge_pages underflow on UFFDIO_COPY [4.14,27/47] bpf, selftests: Fix up some test_verifier cases for unprivileged [4.14,28/47] bpf: Move off_reg into sanitize_ptr_alu [4.14,29/47] bpf: Ensure off_reg has no mixed signed bounds for all types [4.14,30/47] bpf: Rework ptr_limit into alu_limit and add common error path [4.14,31/47] bpf: Improve verifier error messages for users [4.14,32/47] bpf: Refactor and streamline bounds check into helper [4.14,33/47] bpf: Move sanitize_val_alu out of op switch [4.14,34/47] bpf: Tighten speculative pointer arithmetic mask [4.14,35/47] bpf: Update selftests to reflect new error states [4.14,36/47] bpf: do not allow root to mangle valid pointers [4.14,37/47] bpf/verifier: disallow pointer subtraction [4.14,38/47] selftests/bpf: fix test_align [4.14,39/47] selftests/bpf: make dubious pointer arithmetic test useful [4.14,40/47] bpf: Fix leakage of uninitialized bpf stack under speculation [4.14,41/47] bpf: Wrap aux data inside bpf_sanitize_info container [4.14,42/47] bpf: Fix mask direction swap upon off reg sign change [4.14,43/47] bpf: No need to simulate speculative domain for immediates [4.14,44/47] bnxt_en: Remove the setting of dev_port. [4.14,45/47] KVM: SVM: Truncate GPR value for DR and CR accesses in !64-bit mode [4.14,46/47] sched/fair: Optimize select_idle_cpu [4.14,47/47] 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
@@ -4791,7 +4791,6 @@  static int bnxt_hwrm_func_qcaps(struct b
 
 		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);
 		pf->max_rsscos_ctxs = le16_to_cpu(resp->max_rsscos_ctx);
 		pf->max_cp_rings = le16_to_cpu(resp->max_cmpl_rings);