diff mbox

[net-next] r8169:Avoid to use I/O Space access on new design and others

Message ID 1440073712-26826-1-git-send-email-corcodel.marian@gmail.com
State Rejected, archived
Delegated to: David Miller
Headers show

Commit Message

Corcodel Marian Aug. 20, 2015, 12:28 p.m. UTC
Avoid to use on pci express interface I/O Space access   
 and do not enable Bus Master is set on BIOS.   Warning: Do not apply this
 patch must   apply previously patch  first.
To: netdev@vger.kernel.org

Signed-off-by: Corcodel Marian <corcodel.marian@gmail.com>

Comments

Eric Dumazet Aug. 20, 2015, 1:40 p.m. UTC | #1
On Thu, 2015-08-20 at 15:28 +0300, Corcodel Marian wrote:
> Avoid to use on pci express interface I/O Space access   
>  and do not enable Bus Master is set on BIOS.   Warning: Do not apply this
>  patch must   apply previously patch  first.
> To: netdev@vger.kernel.org

It is time you carefully read Documentation/SubmittingPatches

Seriously.


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c
index b1fb54f..6cd7226 100644
--- a/drivers/net/ethernet/realtek/r8169.c
+++ b/drivers/net/ethernet/realtek/r8169.c
@@ -8171,6 +8171,9 @@  static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 	if (pci_is_pcie(pdev))
 		pci_intx(pdev, 0);
 
+	if (pci_is_pcie(pdev))
+		pci_write_config_word(pdev, PCI_COMMAND, ~PCI_COMMAND_IO);
+
 	/* Identify chip attached to board */
 	rtl8169_get_mac_version(tp, dev, cfg->default_ver);
 
@@ -8183,8 +8186,8 @@  static int rtl_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 	rtl_hw_reset(tp);
 
 	rtl_ack_events(tp, 0xffff);
-
-	pci_set_master(pdev);
+	if (!pci_is_pcie(pdev))
+	 	pci_set_master(pdev);
 
 	rtl_init_mdio_ops(tp);
 	rtl_init_pll_power_ops(tp);