Skip to contentSkip to conversation

properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER

PATCH In review Steven Walter 2 participants 3 messages Apr 24, 2001

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER

This patch allows the serial driver to properly detect and set up the ActionTec PCI modem. This modem has a PCI class of COMMUNICATION_OTHER, which is why this modem is not otherwise detected.

Any suggestions on the patch are welcome. Thanks

Signature · 16 lines
-Steven
In a time of universal deceit, telling the truth is a revolutionary act.
			-- George Orwell

--- clean-2.4.3/drivers/char/serial.c	Fri Mar 30 23:15:33 2001
+++ linux/drivers/char/serial.c	Tue Apr 24 15:40:50 2001
@@ -4706,6 +4728,9 @@
 
 
 static struct pci_device_id serial_pci_tbl[] __devinitdata = {
+       { PCI_VENDOR_ID_ATT, PCI_DEVICE_ID_ATT_VENUS_MODEM,
+         PCI_ANY_ID, PCI_ANY_ID, PCI_CLASS_COMMUNICATION_OTHER << 8,
+         0xffff00, },
        { PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
 	 PCI_CLASS_COMMUNICATION_SERIAL << 8, 0xffff00, },
        { 0, }
#1 lore raw
Quoted 6 lines · Steven Walter

This patch allows the serial driver to properly detect and set up the ActionTec PCI modem. This modem has a PCI class of COMMUNICATION_OTHER, which is why this modem is not otherwise detected.

Any suggestions on the patch are welcome. Thanks

A small suggestion: Vendor/device id are sufficient to identify the device. You can change PCI_CLASS_COMMUNICATION_OTHER << 8 to 0.

--

Brian Gerst

#2 lore raw
In reply to Steven Walter
Quoted 10 lines · Brian Gerst
Quoted 6 lines · Steven Walter

This patch allows the serial driver to properly detect and set up the ActionTec PCI modem. This modem has a PCI class of COMMUNICATION_OTHER, which is why this modem is not otherwise detected.

Any suggestions on the patch are welcome. Thanks

A small suggestion: Vendor/device id are sufficient to identify the device. You can change PCI_CLASS_COMMUNICATION_OTHER << 8 to 0.

Excellent suggestion. Follows is the amended patch. Compiled and tested to work. BTW: patch is against 2.4.3.

Signature · 15 lines
-Steven
In a time of universal deceit, telling the truth is a revolutionary act.
			-- George Orwell

--- clean-2.4.3/drivers/char/serial.c	Fri Mar 30 23:15:33 2001
+++ linux/drivers/char/serial.c	Tue Apr 24 16:32:02 2001
@@ -4706,6 +4728,8 @@
 
 
 static struct pci_device_id serial_pci_tbl[] __devinitdata = {
+       { PCI_VENDOR_ID_ATT, PCI_DEVICE_ID_ATT_VENUS_MODEM,
+         PCI_ANY_ID, PCI_ANY_ID, },
        { PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
 	 PCI_CLASS_COMMUNICATION_SERIAL << 8, 0xffff00, },
        { 0, }