mbox series

[0/8] net: Phase out hybrid PCI devres API

Message ID 20250416164407.127261-2-phasta@kernel.org (mailing list archive)
Headers show
Series net: Phase out hybrid PCI devres API | expand

Message

Philipp Stanner April 16, 2025, 4:44 p.m. UTC 
Fixes a number of minor issues with the usage of the PCI API in net.
Notbaly, it replaces calls to the sometimes-managed
pci_request_regions() to the always-managed pcim_request_all_regions(),
enabling us to remove that hybrid functionality from PCI.

Philipp Stanner (8):
  net: prestera: Use pure PCI devres API
  net: octeontx2: Use pure PCI devres API
  net: tulip: Use pure PCI devres API
  net: ethernet: natsemi: Use pure PCI devres API
  net: ethernet: sis900: Use pure PCI devres API
  net: mdio: thunder: Use pure PCI devres API
  net: thunder_bgx: Use pure PCI devres API
  net: thunder_bgx: Don't disable PCI device manually

 drivers/net/ethernet/cavium/thunder/thunder_bgx.c  | 13 ++++---------
 drivers/net/ethernet/dec/tulip/tulip_core.c        |  2 +-
 drivers/net/ethernet/dec/tulip/winbond-840.c       |  2 +-
 .../net/ethernet/marvell/octeontx2/nic/otx2_pf.c   | 14 ++++----------
 .../net/ethernet/marvell/octeontx2/nic/otx2_vf.c   | 14 ++++----------
 drivers/net/ethernet/marvell/octeontx2/nic/rep.c   |  4 +---
 .../net/ethernet/marvell/prestera/prestera_pci.c   |  6 ++----
 drivers/net/ethernet/natsemi/natsemi.c             |  2 +-
 drivers/net/ethernet/sis/sis900.c                  |  2 +-
 drivers/net/mdio/mdio-thunder.c                    | 10 +++-------
 10 files changed, 22 insertions(+), 47 deletions(-)

Comments

Jacob Keller April 16, 2025, 5:10 p.m. UTC | #1 
> -----Original Message-----
> From: Philipp Stanner <phasta@kernel.org>
> Sent: Wednesday, April 16, 2025 9:44 AM
> To: Sunil Goutham <sgoutham@marvell.com>; Andrew Lunn
> <andrew+netdev@lunn.ch>; David S. Miller <davem@davemloft.net>; Dumazet,
> Eric <edumazet@google.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni
> <pabeni@redhat.com>; Geetha sowjanya <gakula@marvell.com>; Subbaraya
> Sundeep <sbhatta@marvell.com>; hariprasad <hkelam@marvell.com>; Bharat
> Bhushan <bbhushan2@marvell.com>; Taras Chornyi
> <taras.chornyi@plvision.eu>; Daniele Venzano <venza@brownhat.org>; Heiner
> Kallweit <hkallweit1@gmail.com>; Russell King <linux@armlinux.org.uk>; Thomas
> Gleixner <tglx@linutronix.de>; Philipp Stanner <phasta@kernel.org>; Helge Deller
> <deller@gmx.de>; Ingo Molnar <mingo@kernel.org>; Simon Horman
> <horms@kernel.org>; Al Viro <viro@zeniv.linux.org.uk>; Sabrina Dubroca
> <sd@queasysnail.net>; Keller, Jacob E <jacob.e.keller@intel.com>
> Cc: linux-arm-kernel@lists.infradead.org; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org; linux-parisc@vger.kernel.org
> Subject: [PATCH 0/8] net: Phase out hybrid PCI devres API
> 
> Fixes a number of minor issues with the usage of the PCI API in net.
> Notbaly, it replaces calls to the sometimes-managed
> pci_request_regions() to the always-managed pcim_request_all_regions(),
> enabling us to remove that hybrid functionality from PCI.
> 

Removing the problematic "sometimes" behavior is good.

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>