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

[Keller, Jacob E]

> -----Original Message-----
> From: Philipp Stanner <phasta@xxxxxxxxxx>
> Sent: Wednesday, April 16, 2025 9:44 AM
> To: Sunil Goutham <sgoutham@xxxxxxxxxxx>; Andrew Lunn
> <andrew+netdev@xxxxxxx>; David S. Miller <davem@xxxxxxxxxxxxx>; Dumazet,
> Eric <edumazet@xxxxxxxxxx>; Jakub Kicinski <kuba@xxxxxxxxxx>; Paolo Abeni
> <pabeni@xxxxxxxxxx>; Geetha sowjanya <gakula@xxxxxxxxxxx>; Subbaraya
> Sundeep <sbhatta@xxxxxxxxxxx>; hariprasad <hkelam@xxxxxxxxxxx>; Bharat
> Bhushan <bbhushan2@xxxxxxxxxxx>; Taras Chornyi
> <taras.chornyi@xxxxxxxxxxx>; Daniele Venzano <venza@xxxxxxxxxxxx>; Heiner
> Kallweit <hkallweit1@xxxxxxxxx>; Russell King <linux@xxxxxxxxxxxxxxx>; Thomas
> Gleixner <tglx@xxxxxxxxxxxxx>; Philipp Stanner <phasta@xxxxxxxxxx>; Helge Deller
> <deller@xxxxxx>; Ingo Molnar <mingo@xxxxxxxxxx>; Simon Horman
> <horms@xxxxxxxxxx>; Al Viro <viro@xxxxxxxxxxxxxxxxxx>; Sabrina Dubroca
> <sd@xxxxxxxxxxxxxxx>; Keller, Jacob E <jacob.e.keller@xxxxxxxxx>
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; linux-parisc@xxxxxxxxxxxxxxx
> 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@xxxxxxxxx>