Subject: Re: CVS commit: src/sys/dev/pci
To: None <>
From: Rui Paulo <>
List: source-changes
Date: 02/23/2006 18:48:46
Quentin Garnier <> writes:

> On Thu, Feb 23, 2006 at 10:38:13AM -0800, Jason Thorpe wrote:
>> On Feb 21, 2006, at 6:57 PM, Tim Rightnour wrote:
>> >
>> >Module Name:	src
>> >Committed By:	garbled
>> >Date:		Wed Feb 22 02:57:26 UTC 2006
>> >
>> >Modified Files:
>> >	src/sys/dev/pci: autri.c if_pcn.c
>> >
>> >Log Message:
>> >IBM makes a pcn card that shows up as:
>> >Trident Microsystems 4DWAVE DX (ethernet network, revision 0x25)
>> >Unfortunately, this is an autri sound card.  Special case the match  
>> >routines
>> >for both drivers so if_pcn properly picks it up, and autri doesn't.
>> The if_pcn.c change is OK, but I don't like the autri.c change.  In  
>> autri.c, rather than checking for one (of many) classes that you  
>> don't want, instead you should check for the class that you DO want,  
>> which in this case is probably PCI_CLASS_MULTIMEDIA (but you'll want  
>> to verify that with someone who actually has the device).
> Why not making pcn_match return a greater value for that board?  I don't
> see why autri should change, somehow.

I suggested that to Tim too, but IMO setting PCI_CLASS_XXX for autri
and pcn seems more cleaner. After all, they are only interested in
multimedia or network devices.

  Rui Paulo			<rpaulo@{NetBSD{,-PT}.org,}>