Il giorno 29/gen/06, alle ore 14:23, Lennert Buytenhek ha scritto:
One more thing:
I'm still checking the other mail, but this is easier, so I'll just reply first...

@@ -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.

+/* PCI Registers */
+enum sis900_pci_registers {
+       CFGPMC = 0x40,
+       CFGPMCSR = 0x44
+};

But this implies that it's a config space register. So which one is it?
It's definitely a PCI config space register. There is reserved space at 0x44 on I/O space. This needs a correction, but it's not a bug that affects functionality. I've filed a bug in bugzilla (5976) to remind me of this, because right now I can't make a patch.

Thanks.
--
Daniele Venzano
http://www.brownhat.org

-
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