Big round of minor updates:

- Use pci_get_powerstate()/pci_set_powerstate() in all the other drivers
  that need them so we don't have to fiddle with the PCI power management
  registers directly.
- Use pci_enable_busmaster()/pci_enable_io() to turn on busmastering and
  PIO/memory mapped accesses.
- Add support to the RealTek driver for the D-Link DFE-530TX+ which has
  a RealTek 8139 with its own PCI ID. (Submitted by Jason Wright)
- Have the SiS 900/National DP83815 driver be sure to disable PME
  mode in sis_reset(). This apparently fixes a problem on some
  motherboards where the DP83815 chip fails to receive packets.
  (Submitted by Chuck McCrobie <mccrobie@cablespeed.com>)

1 files changed, 21 insertions, 26 deletions

diff --git a/sys/pci/if_sk.c b/sys/pci/if_sk.c
index 960b98b..4b5e7f6 100644
--- a/sys/pci/if_sk.c
+++ b/sys/pci/if_sk.c
@@ -1192,37 +1192,32 @@ static int sk_attach(dev)
 	/*
 	 * Handle power management nonsense.
 	 */
-	command = pci_read_config(dev, SK_PCI_CAPID, 4) & 0x000000FF;
-	if (command == 0x01) {
-
-		command = pci_read_config(dev, SK_PCI_PWRMGMTCTRL, 4);
-		if (command & SK_PSTATE_MASK) {
-			u_int32_t		iobase, membase, irq;
-
-			/* Save important PCI config data. */
-			iobase = pci_read_config(dev, SK_PCI_LOIO, 4);
-			membase = pci_read_config(dev, SK_PCI_LOMEM, 4);
-			irq = pci_read_config(dev, SK_PCI_INTLINE, 4);
-
-			/* Reset the power state. */
-			printf("skc%d: chip is in D%d power mode "
-			"-- setting to D0\n", unit, command & SK_PSTATE_MASK);
-			command &= 0xFFFFFFFC;
-			pci_write_config(dev, SK_PCI_PWRMGMTCTRL, command, 4);
-
-			/* Restore PCI config data. */
-			pci_write_config(dev, SK_PCI_LOIO, iobase, 4);
-			pci_write_config(dev, SK_PCI_LOMEM, membase, 4);
-			pci_write_config(dev, SK_PCI_INTLINE, irq, 4);
-		}
+	if (pci_get_powerstate(dev) != PCI_POWERSTATE_D0) {
+		u_int32_t		iobase, membase, irq;
+
+		/* Save important PCI config data. */
+		iobase = pci_read_config(dev, SK_PCI_LOIO, 4);
+		membase = pci_read_config(dev, SK_PCI_LOMEM, 4);
+		irq = pci_read_config(dev, SK_PCI_INTLINE, 4);
+
+		/* Reset the power state. */
+		printf("skc%d: chip is in D%d power mode "
+		    "-- setting to D0\n", unit,
+		    pci_get_powerstate(dev));
+		pci_set_powerstate(dev, PCI_POWERSTATE_D0);
+
+		/* Restore PCI config data. */
+		pci_write_config(dev, SK_PCI_LOIO, iobase, 4);
+		pci_write_config(dev, SK_PCI_LOMEM, membase, 4);
+		pci_write_config(dev, SK_PCI_INTLINE, irq, 4);
 	}
 
 	/*
 	 * Map control/status registers.
 	 */
-	command = pci_read_config(dev, PCIR_COMMAND, 4);
-	command |= (PCIM_CMD_PORTEN|PCIM_CMD_MEMEN|PCIM_CMD_BUSMASTEREN);
-	pci_write_config(dev, PCIR_COMMAND, command, 4);
+	pci_enable_busmaster(dev);
+	pci_enable_io(dev, PCIM_CMD_PORTEN);
+	pci_enable_io(dev, PCIM_CMD_MEMEN);
 	command = pci_read_config(dev, PCIR_COMMAND, 4);
 
 #ifdef SK_USEIOSPACE