netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bjorn Helgaas <helgaas@kernel.org>
To: "David S . Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	Bjorn Helgaas <bhelgaas@google.com>,
	Abhijit Ayarekar <aayarekar@marvell.com>,
	Ajit Khaparde <ajit.khaparde@broadcom.com>,
	Ariel Elior <aelior@marvell.com>,
	Chris Snook <chris.snook@gmail.com>,
	Dimitris Michailidis <dmichail@fungible.com>,
	Edward Cree <ecree.xilinx@gmail.com>,
	Jesse Brandeburg <jesse.brandeburg@intel.com>,
	Jiawen Wu <jiawenwu@trustnetic.com>,
	Manish Chopra <manishc@marvell.com>,
	Martin Habets <habetsm.xilinx@gmail.com>,
	Mengyuan Lou <mengyuanlou@net-swift.com>,
	Michael Chan <michael.chan@broadcom.com>,
	Rahul Verma <rahulv@marvell.com>,
	Raju Rangoju <rajur@chelsio.com>, Rasesh Mody <rmody@marvell.com>,
	Salil Mehta <salil.mehta@huawei.com>,
	Shahed Shaikh <shshaikh@marvell.com>,
	Somnath Kotur <somnath.kotur@broadcom.com>,
	Sriharsha Basavapatna <sriharsha.basavapatna@broadcom.com>,
	Sudarsana Kalluru <skalluru@marvell.com>,
	Tony Nguyen <anthony.l.nguyen@intel.com>,
	Veerasenareddy Burru <vburru@marvell.com>,
	Yisen Zhuang <yisen.zhuang@huawei.com>,
	GR-Linux-NIC-Dev@marvell.com, intel-wired-lan@lists.osuosl.org
Subject: [PATCH 00/28] PCI/AER: Remove redundant Device Control Error Reporting Enable
Date: Tue,  7 Mar 2023 12:19:11 -0600	[thread overview]
Message-ID: <20230307181940.868828-1-helgaas@kernel.org> (raw)

From: Bjorn Helgaas <bhelgaas@google.com>

Since f26e58bf6f54 ("PCI/AER: Enable error reporting when AER is native"),
which appeared in v6.0, the PCI core has enabled PCIe error reporting for
all devices during enumeration.

Remove driver code to do this and remove unnecessary includes of
<linux/aer.h> from several other drivers.

Intel folks, sorry that I missed removing the <linux/aer.h> includes in the
first series.


Bjorn Helgaas (28):
  alx: Drop redundant pci_enable_pcie_error_reporting()
  be2net: Drop redundant pci_enable_pcie_error_reporting()
  bnx2: Drop redundant pci_enable_pcie_error_reporting()
  bnx2x: Drop redundant pci_enable_pcie_error_reporting()
  bnxt: Drop redundant pci_enable_pcie_error_reporting()
  cxgb4: Drop redundant pci_enable_pcie_error_reporting()
  net/fungible: Drop redundant pci_enable_pcie_error_reporting()
  net: hns3: remove unnecessary aer.h include
  netxen_nic: Drop redundant pci_enable_pcie_error_reporting()
  octeon_ep: Drop redundant pci_enable_pcie_error_reporting()
  qed: Drop redundant pci_enable_pcie_error_reporting()
  net: qede: Remove unnecessary aer.h include
  qlcnic: Drop redundant pci_enable_pcie_error_reporting()
  qlcnic: Remove unnecessary aer.h include
  sfc: Drop redundant pci_enable_pcie_error_reporting()
  sfc: falcon: Drop redundant pci_enable_pcie_error_reporting()
  sfc/siena: Drop redundant pci_enable_pcie_error_reporting()
  sfc_ef100: Drop redundant pci_disable_pcie_error_reporting()
  net: ngbe: Drop redundant pci_enable_pcie_error_reporting()
  net: txgbe: Drop redundant pci_enable_pcie_error_reporting()
  e1000e: Remove unnecessary aer.h include
  fm10k: Remove unnecessary aer.h include
  i40e: Remove unnecessary aer.h include
  iavf: Remove unnecessary aer.h include
  ice: Remove unnecessary aer.h include
  igb: Remove unnecessary aer.h include
  igc: Remove unnecessary aer.h include
  ixgbe: Remove unnecessary aer.h include

 drivers/net/ethernet/atheros/alx/main.c       |  4 ----
 drivers/net/ethernet/broadcom/bnx2.c          | 21 -------------------
 drivers/net/ethernet/broadcom/bnx2.h          |  1 -
 drivers/net/ethernet/broadcom/bnx2x/bnx2x.h   |  1 -
 .../net/ethernet/broadcom/bnx2x/bnx2x_main.c  | 19 -----------------
 drivers/net/ethernet/broadcom/bnxt/bnxt.c     |  4 ----
 .../net/ethernet/chelsio/cxgb4/cxgb4_main.c   |  4 ----
 drivers/net/ethernet/emulex/benet/be_main.c   |  8 -------
 .../net/ethernet/fungible/funcore/fun_dev.c   |  5 -----
 .../net/ethernet/hisilicon/hns3/hns3_enet.c   |  1 -
 drivers/net/ethernet/intel/e1000e/netdev.c    |  1 -
 drivers/net/ethernet/intel/fm10k/fm10k_pci.c  |  1 -
 drivers/net/ethernet/intel/i40e/i40e.h        |  1 -
 drivers/net/ethernet/intel/iavf/iavf.h        |  1 -
 drivers/net/ethernet/intel/ice/ice.h          |  1 -
 drivers/net/ethernet/intel/igb/igb_main.c     |  1 -
 drivers/net/ethernet/intel/igc/igc_main.c     |  1 -
 drivers/net/ethernet/intel/ixgbe/ixgbe.h      |  1 -
 .../ethernet/marvell/octeon_ep/octep_main.c   |  4 ----
 .../ethernet/qlogic/netxen/netxen_nic_main.c  | 10 +--------
 drivers/net/ethernet/qlogic/qed/qed_main.c    |  9 --------
 drivers/net/ethernet/qlogic/qede/qede_main.c  |  1 -
 .../ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c   |  1 -
 .../net/ethernet/qlogic/qlcnic/qlcnic_main.c  |  4 ----
 .../net/ethernet/qlogic/qlcnic/qlcnic_sysfs.c |  1 -
 drivers/net/ethernet/sfc/ef100.c              |  3 ---
 drivers/net/ethernet/sfc/efx.c                |  5 -----
 drivers/net/ethernet/sfc/falcon/efx.c         |  9 --------
 drivers/net/ethernet/sfc/siena/efx.c          |  5 -----
 drivers/net/ethernet/wangxun/ngbe/ngbe_main.c |  4 ----
 .../net/ethernet/wangxun/txgbe/txgbe_main.c   |  5 -----
 31 files changed, 1 insertion(+), 136 deletions(-)

-- 
2.25.1


             reply	other threads:[~2023-03-07 18:25 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-07 18:19 Bjorn Helgaas [this message]
2023-03-07 18:19 ` [PATCH 01/28] alx: Drop redundant pci_enable_pcie_error_reporting() Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 02/28] be2net: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 03/28] bnx2: " Bjorn Helgaas
2023-03-08  0:21   ` Michael Chan
2023-03-07 18:19 ` [PATCH 04/28] bnx2x: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 05/28] bnxt: " Bjorn Helgaas
2023-03-08  0:23   ` Michael Chan
2023-03-07 18:19 ` [PATCH 06/28] cxgb4: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 07/28] net/fungible: " Bjorn Helgaas
2023-03-07 19:17   ` Dimitris Michailidis
2023-03-07 18:19 ` [PATCH 08/28] net: hns3: remove unnecessary aer.h include Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 09/28] netxen_nic: Drop redundant pci_enable_pcie_error_reporting() Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 10/28] octeon_ep: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 11/28] qed: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 12/28] net: qede: Remove unnecessary aer.h include Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 13/28] qlcnic: Drop redundant pci_enable_pcie_error_reporting() Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 14/28] qlcnic: Remove unnecessary aer.h include Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 15/28] sfc: Drop redundant pci_enable_pcie_error_reporting() Bjorn Helgaas
2023-03-08 16:11   ` Edward Cree
2023-03-07 18:19 ` [PATCH 16/28] sfc: falcon: " Bjorn Helgaas
2023-03-08 16:12   ` Edward Cree
2023-03-07 18:19 ` [PATCH 17/28] sfc/siena: " Bjorn Helgaas
2023-03-08 16:12   ` Edward Cree
2023-03-07 18:19 ` [PATCH 18/28] sfc_ef100: Drop redundant pci_disable_pcie_error_reporting() Bjorn Helgaas
2023-03-08 16:16   ` Edward Cree
2023-03-07 18:19 ` [PATCH 19/28] net: ngbe: Drop redundant pci_enable_pcie_error_reporting() Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 20/28] net: txgbe: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 21/28] e1000e: Remove unnecessary aer.h include Bjorn Helgaas
2023-03-15 12:00   ` [Intel-wired-lan] " naamax.meir
2023-03-07 18:19 ` [PATCH 22/28] fm10k: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 23/28] i40e: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 24/28] iavf: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 25/28] ice: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 26/28] igb: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 27/28] igc: " Bjorn Helgaas
2023-03-07 18:19 ` [PATCH 28/28] ixgbe: " Bjorn Helgaas
2023-03-07 19:28 ` [PATCH 00/28] PCI/AER: Remove redundant Device Control Error Reporting Enable Jesse Brandeburg
2023-03-09  7:40 ` patchwork-bot+netdevbpf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230307181940.868828-1-helgaas@kernel.org \
    --to=helgaas@kernel.org \
    --cc=GR-Linux-NIC-Dev@marvell.com \
    --cc=aayarekar@marvell.com \
    --cc=aelior@marvell.com \
    --cc=ajit.khaparde@broadcom.com \
    --cc=anthony.l.nguyen@intel.com \
    --cc=bhelgaas@google.com \
    --cc=chris.snook@gmail.com \
    --cc=davem@davemloft.net \
    --cc=dmichail@fungible.com \
    --cc=ecree.xilinx@gmail.com \
    --cc=edumazet@google.com \
    --cc=habetsm.xilinx@gmail.com \
    --cc=intel-wired-lan@lists.osuosl.org \
    --cc=jesse.brandeburg@intel.com \
    --cc=jiawenwu@trustnetic.com \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=manishc@marvell.com \
    --cc=mengyuanlou@net-swift.com \
    --cc=michael.chan@broadcom.com \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=rahulv@marvell.com \
    --cc=rajur@chelsio.com \
    --cc=rmody@marvell.com \
    --cc=salil.mehta@huawei.com \
    --cc=shshaikh@marvell.com \
    --cc=skalluru@marvell.com \
    --cc=somnath.kotur@broadcom.com \
    --cc=sriharsha.basavapatna@broadcom.com \
    --cc=vburru@marvell.com \
    --cc=yisen.zhuang@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).