| Message ID | 5739775a-a5d7-cd72-469c-d1ded3347a77@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 36f9b47457f0cad290502c56664df76a2859c9b4 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net-next] r8169: remove useless PCI region size check | expand |
Hello: This patch was applied to netdev/net-next.git (master) by David S. Miller <davem@davemloft.net>: On Fri, 2 Sep 2022 23:16:52 +0200 you wrote: > Let's trust the hardware here and remove this useless check. > > Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> > --- > drivers/net/ethernet/realtek/r8169_main.c | 6 ------ > 1 file changed, 6 deletions(-) Here is the summary with links: - [net-next] r8169: remove useless PCI region size check https://git.kernel.org/netdev/net-next/c/36f9b47457f0 You are awesome, thank you!
diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c index e6fb6f223..1d571af4d 100644 --- a/drivers/net/ethernet/realtek/r8169_main.c +++ b/drivers/net/ethernet/realtek/r8169_main.c @@ -5206,12 +5206,6 @@ static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) return -ENODEV; } - /* check for weird/broken PCI region reporting */ - if (pci_resource_len(pdev, region) < R8169_REGS_SIZE) { - dev_err(&pdev->dev, "Invalid PCI region size(s), aborting\n"); - return -ENODEV; - } - rc = pcim_iomap_regions(pdev, BIT(region), KBUILD_MODNAME); if (rc < 0) { dev_err(&pdev->dev, "cannot remap MMIO, aborting\n");
Let's trust the hardware here and remove this useless check. Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com> --- drivers/net/ethernet/realtek/r8169_main.c | 6 ------ 1 file changed, 6 deletions(-)