On Sun, Jan 29, 2006 at 02:16:00PM +0100, Lennert Buytenhek wrote:

> I found some fishy-looking things while backporting sis900 wake-on-lan
> support.

One more thing:

@@ -33,6 +33,7 @@ enum sis900_registers {
        rxcfg=0x34, //Receive Configuration Register
        flctrl=0x38, //Flow Control Register
        rxlen=0x3c, //Receive Packet Length Register
+       cfgpmcsr=0x44, //Configuration Power Management Control/Status Register
        rfcr=0x48, //Receive Filter Control Register
        rfdr=0x4C, //Receive Filter Data Register
        pmctrl=0xB0, //Power Management Control Register

This implies that CFGPCMSR is an I/O space register.


@@ -140,6 +141,50 @@ enum sis96x_eeprom_command {
        EEREQ = 0x00000400, EEDONE = 0x00000200, EEGNT = 0x00000100
 };
 
+/* PCI Registers */
+enum sis900_pci_registers {
+       CFGPMC = 0x40,
+       CFGPMCSR = 0x44
+};

And in sis900_set_wol:

+               pci_read_config_dword(sis_priv->pci_dev, CFGPMCSR, &cfgpmcsr);
+               cfgpmcsr |= ~PME_EN;
+               pci_write_config_dword(sis_priv->pci_dev, CFGPMCSR, cfgpmcsr);

But this implies that it's a config space register.  So which one is it?


--L
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to