#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
+#include <linux/pci_ids.h>
 #include "ixgb_hw.h"
 #include "ixgb_ids.h"
 
        ASSERT(!(ctrl_reg & IXGB_CTRL0_RST));
 #endif
 
-       if (hw->subsystem_vendor_id == SUN_SUBVENDOR_ID) {
+       if (hw->subsystem_vendor_id == PCI_VENDOR_ID_SUN) {
                ctrl_reg =  /* Enable interrupt from XFP and SerDes */
                           IXGB_CTRL1_GPI0_EN |
                           IXGB_CTRL1_SDP6_DIR |
        }
 
        /* update phy type for sun specific board */
-       if (hw->subsystem_vendor_id == SUN_SUBVENDOR_ID)
+       if (hw->subsystem_vendor_id == PCI_VENDOR_ID_SUN)
                phy_type = ixgb_phy_type_bcm;
 
        return phy_type;
 
 ** The Device and Vendor IDs for 10 Gigabit MACs
 **********************************************************************/
 
-#define INTEL_VENDOR_ID             0x8086
-#define INTEL_SUBVENDOR_ID          0x8086
-#define SUN_VENDOR_ID               0x108E
-#define SUN_SUBVENDOR_ID            0x108E
-
 #define IXGB_DEVICE_ID_82597EX      0x1048
 #define IXGB_DEVICE_ID_82597EX_SR   0x1A48
 #define IXGB_DEVICE_ID_82597EX_LR   0x1B48
 
  *   Class, Class Mask, private data (not used) }
  */
 static DEFINE_PCI_DEVICE_TABLE(ixgb_pci_tbl) = {
-       {INTEL_VENDOR_ID, IXGB_DEVICE_ID_82597EX,
+       {PCI_VENDOR_ID_INTEL, IXGB_DEVICE_ID_82597EX,
         PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
-       {INTEL_VENDOR_ID, IXGB_DEVICE_ID_82597EX_CX4,
+       {PCI_VENDOR_ID_INTEL, IXGB_DEVICE_ID_82597EX_CX4,
         PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
-       {INTEL_VENDOR_ID, IXGB_DEVICE_ID_82597EX_SR,
+       {PCI_VENDOR_ID_INTEL, IXGB_DEVICE_ID_82597EX_SR,
         PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
-       {INTEL_VENDOR_ID, IXGB_DEVICE_ID_82597EX_LR,
+       {PCI_VENDOR_ID_INTEL, IXGB_DEVICE_ID_82597EX_LR,
         PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0},
 
        /* required last entry */
 {
        u32 val = IXGB_INT_RXT0 | IXGB_INT_RXDMT0 |
                  IXGB_INT_TXDW | IXGB_INT_LSC;
-       if (adapter->hw.subsystem_vendor_id == SUN_SUBVENDOR_ID)
+       if (adapter->hw.subsystem_vendor_id == PCI_VENDOR_ID_SUN)
                val |= IXGB_INT_GPI0;
        IXGB_WRITE_REG(&adapter->hw, IMS, val);
        IXGB_WRITE_FLUSH(&adapter->hw);