* [PATCH] properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER
@ 2001-04-24 21:03 Steven Walter
2001-04-24 21:18 ` Brian Gerst
0 siblings, 1 reply; 3+ messages in thread
From: Steven Walter @ 2001-04-24 21:03 UTC (permalink / raw)
To: linux-kernel; +Cc: jgarzik, tytso
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
--
-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, }
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER
2001-04-24 21:03 [PATCH] properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER Steven Walter
@ 2001-04-24 21:18 ` Brian Gerst
2001-04-24 22:19 ` Steven Walter
0 siblings, 1 reply; 3+ messages in thread
From: Brian Gerst @ 2001-04-24 21:18 UTC (permalink / raw)
To: Steven Walter; +Cc: linux-kernel
Steven Walter wrote:
>
> 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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER
2001-04-24 21:18 ` Brian Gerst
@ 2001-04-24 22:19 ` Steven Walter
0 siblings, 0 replies; 3+ messages in thread
From: Steven Walter @ 2001-04-24 22:19 UTC (permalink / raw)
To: Brian Gerst; +Cc: jgarzik, tytso, linux-kernel
On Tue, Apr 24, 2001 at 05:18:36PM -0400, Brian Gerst wrote:
> Steven Walter wrote:
> >
> > 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.
--
-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, }
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2001-04-24 22:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-04-24 21:03 [PATCH] properly detect ActionTec modem of PCI_CLASS_COMMUNICATION_OTHER Steven Walter
2001-04-24 21:18 ` Brian Gerst
2001-04-24 22:19 ` Steven Walter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox