diff options
author | Jean Delvare <khali@linux-fr.org> | 2011-05-15 12:13:46 -0400 |
---|---|---|
committer | Jesse Barnes <jbarnes@virtuousgeek.org> | 2011-05-21 15:17:15 -0400 |
commit | 9251bac97d47fdaea406ea0595c2d0aa50022f12 (patch) | |
tree | a312c2f1810b9d8f8e0164113bd81a2f39227c74 | |
parent | cbfddd20937ed890ce7027fa08a7c84e977128cf (diff) |
PCI: Don't use dmi_name_in_vendors in quirk
Don't use the costly dmi_name_in_vendors() when we know the string we
are looking for can only be in the DMI board name field. This is more
robust and, more importantly, much faster.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
-rw-r--r-- | drivers/pci/quirks.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 4fc1b0daedaf..e8a140669f90 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c | |||
@@ -2349,8 +2349,11 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_8132_BRIDGE, | |||
2349 | */ | 2349 | */ |
2350 | static void __devinit nvenet_msi_disable(struct pci_dev *dev) | 2350 | static void __devinit nvenet_msi_disable(struct pci_dev *dev) |
2351 | { | 2351 | { |
2352 | if (dmi_name_in_vendors("P5N32-SLI PREMIUM") || | 2352 | const char *board_name = dmi_get_system_info(DMI_BOARD_NAME); |
2353 | dmi_name_in_vendors("P5N32-E SLI")) { | 2353 | |
2354 | if (board_name && | ||
2355 | (strstr(board_name, "P5N32-SLI PREMIUM") || | ||
2356 | strstr(board_name, "P5N32-E SLI"))) { | ||
2354 | dev_info(&dev->dev, | 2357 | dev_info(&dev->dev, |
2355 | "Disabling msi for MCP55 NIC on P5N32-SLI\n"); | 2358 | "Disabling msi for MCP55 NIC on P5N32-SLI\n"); |
2356 | dev->no_msi = 1; | 2359 | dev->no_msi = 1; |