Skip to contentSkip to conversation

PCI: Disable MSI for MCP55 on P5N32-E SLI

PATCH Applied Ben Hutchings 2 participants 2 messages May 16, 2010 – May 18, 2010

Subsystems (3)
Networking Network drivers PCI
lore
Share & tools
lore

Ben Hutchings

author
#0 lore raw

PATCH 1/1 PCI: Disable MSI for MCP55 on P5N32-E SLI

As reported in <http://bugs.debian.org/552299>, MSI appears to be broken for this on-board device. We already have a quirk for the P5N32-SLI Premium; extend it to cover both variants of the board.

Reported-by Romain DEGEZ <[email protected]>
Signed-off-by Ben Hutchings <[email protected]>
Cc [email protected]
---
 drivers/pci/quirks.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)
drivers/pci/quirks.c +4 -3
index 27c0e6e..4807825 100644
@@ -2218,15 +2218,16 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_SERVERWORKS,
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_8132_BRIDGE,
 			 ht_enable_msi_mapping);
 
-/* The P5N32-SLI Premium motherboard from Asus has a problem with msi
+/* The P5N32-SLI motherboards from Asus have a problem with msi
  * for the MCP55 NIC. It is not yet determined whether the msi problem
  * also affects other devices. As for now, turn off msi for this device.
  */
 static void __devinit nvenet_msi_disable(struct pci_dev *dev)
 {
-	if (dmi_name_in_vendors("P5N32-SLI PREMIUM")) {
+	if (dmi_name_in_vendors("P5N32-SLI PREMIUM") ||
+	    dmi_name_in_vendors("P5N32-E SLI")) {
 		dev_info(&dev->dev,
-			 "Disabling msi for MCP55 NIC on P5N32-SLI Premium\n");
+			 "Disabling msi for MCP55 NIC on P5N32-SLI\n");
 		dev->no_msi = 1;
 	}
 }
Signature · 1 line
1.7.0.3
Applied
#1 lore raw
Quoted 8 lines · On Sun, 16 May 2010 02:28:49 +0100 Ben Hutchings <[email protected]> wrote:

As reported in <http://bugs.debian.org/552299>, MSI appears to be broken for this on-board device. We already have a quirk for the P5N32-SLI Premium; extend it to cover both variants of the board.

Reported-by Romain DEGEZ <[email protected]>
Signed-off-by Ben Hutchings <[email protected]>
Cc [email protected]

---

Applied to linux-next, sorry for missing it earlier.

Signature · 1 line
Jesse Barnes, Intel Open Source Technology Center