From owner-cvs-all@FreeBSD.ORG Wed Jun 16 02:41:23 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6025116A4CE; Wed, 16 Jun 2004 02:41:23 +0000 (GMT) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id ED10943D2D; Wed, 16 Jun 2004 02:41:22 +0000 (GMT) (envelope-from scottl@freebsd.org) Received: from pooker.samsco.org (scottl@localhost [127.0.0.1]) by pooker.samsco.org (8.12.11/8.12.10) with ESMTP id i5G2hO0m013325; Tue, 15 Jun 2004 20:43:24 -0600 (MDT) (envelope-from scottl@freebsd.org) Received: from localhost (scottl@localhost)i5G2hOZw013322; Tue, 15 Jun 2004 20:43:24 -0600 (MDT) (envelope-from scottl@freebsd.org) X-Authentication-Warning: pooker.samsco.org: scottl owned process doing -bs Date: Tue, 15 Jun 2004 20:43:24 -0600 (MDT) From: Scott Long Sender: scottl@pooker.samsco.org To: "David O'Brien" In-Reply-To: <20040615231328.GB13516@dragon.nuxi.com> Message-ID: <20040615203430.X13271@pooker.samsco.org> References: <200406132212.i5DMC3b9075832@repoman.freebsd.org> <20040615231328.GB13516@dragon.nuxi.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Spam-Status: No, hits=0.0 required=3.8 tests=none autolearn=no version=2.63 X-Spam-Checker-Version: SpamAssassin 2.63 (2004-01-11) on pooker.samsco.org cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org cc: Joseph Dunn Subject: Re: cvs commit: src/sys/dev/sound/pci emu10k1.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Jun 2004 02:41:23 -0000 On Tue, 15 Jun 2004, David O'Brien wrote: > On Mon, Jun 14, 2004 at 01:10:12AM +0200, Marius Strobl wrote: > > On Sun, Jun 13, 2004 at 10:12:02PM +0000, David E. O'Brien wrote: > > > obrien 2004-06-13 22:12:02 UTC > > > > > > FreeBSD src repository > > > > > > Modified files: > > > sys/dev/sound/pci emu10k1.c > > > Log: > > > Add PCI identifier for Dell modified SBLive! card > > > > > > Submitted by: Joseph Dunn > > > > > > > Wasn't there consensus that simply adding its PCI ID isn't enough to > > get this card working? > > No, not with the current emu10k1 driver. I think my commit gets us > closer to working with the Dell emu10k1 hardware. I really don't think > some dell-emu10k1 driver will be written vs. adding extra bits to this > emu10k1 driver if needed. So getting the proper driver to at least probe > and try to work with the hardware are steps closer than not even trying > to work with the Dell emu10k1 hardware. > > -- > -- David (obrien@FreeBSD.org) > No, please don't add ids to drivers without confirmation that the hardware will work with the driver. You are muddying the 'support' issue, and this gets you no more than what 'pciconf' can give you. By your logic, we should just suck the entire pci id database into the kernel, and you know that thatis silly. For someone who is usually so concerned about FreeBSD standards and what goes into the tree, this is very unlike you. Please take out this change until there is confirmation that it will work. If it is te same Dell SbLive card that I've heard about, then the emu10k1 driver won't be able to do a thing with it, period. It may even, as Peter hinted, cause a lockup on attach. Scott