From owner-cvs-src@FreeBSD.ORG Wed Oct 18 14:57:40 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.ORG Delivered-To: cvs-src@FreeBSD.ORG Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 654CE16A407; Wed, 18 Oct 2006 14:57:40 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (vc4-2-0-87.dsl.netrack.net [199.45.160.85]) by mx1.FreeBSD.org (Postfix) with ESMTP id E775343D45; Wed, 18 Oct 2006 14:57:39 +0000 (GMT) (envelope-from imp@bsdimp.com) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.13.4/8.13.4) with ESMTP id k9IEvRdp074565; Wed, 18 Oct 2006 08:57:28 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Wed, 18 Oct 2006 08:56:42 -0600 (MDT) Message-Id: <20061018.085642.652860238.imp@bsdimp.com> To: Alexander@Leidinger.net From: "M. Warner Losh" In-Reply-To: <20061018100033.pz0u24u2gw8owc8s@webmail.leidinger.net> References: <200610171808.k9HI84bw054602@repoman.freebsd.org> <20061018100033.pz0u24u2gw8owc8s@webmail.leidinger.net> X-Mailer: Mew version 4.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-2.0 (harmony.bsdimp.com [127.0.0.1]); Wed, 18 Oct 2006 08:57:28 -0600 (MDT) Cc: cvs-src@FreeBSD.ORG, src-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/conf files X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Oct 2006 14:57:40 -0000 In message: <20061018100033.pz0u24u2gw8owc8s@webmail.leidinger.net> Alexander Leidinger writes: : Quoting Warner Losh (from Tue, 17 Oct 2006 18:08:04 : +0000 (UTC)): : : > imp 2006-10-17 18:08:04 UTC : > : > FreeBSD src repository : > : > Modified files: : > sys/conf files : > Log: : > Collapse the two identical emu10k1-alsa%diked.h creation rules for : > snd_emu10k1 and snd_emu10kx into one line. The 'pci' dependency here : > adds no value, so I eliminted it (we don't have a snd.all file that : > might make it mildly useful, and even then it wouldn't be that : > useful). With the pci optional component eliminated, I could use the : > '|' operator. I could have also include pci on both sides of the | : > operator, but since it isn't a value add at all, it was better to : > eliminate it. : : Nice. Is this stuff documented somewhere? No. A lot of this is black magic voodoo :-(. Would make a good project to document config. However, I fear that once documented, the mythical replacement would actually arrive... : How do I determine if 'pci' : adds a value or not? Is it of no value because the header lists : emu10k[1x] as a depedency and the emu10k[1x].c lists the 'pci' : dependency? It is of no value because it is only a pci device. If you have snd_emu10k1 in the kernel config file w/o pci, then nothing is included. It should, instead, give an error at config or compile time since this violates pola. It wouldn't be useless in, say, the ep case. In that case we only compile the bus front ends that are in the kernel. This means you don't get EISA frontend if there's no 'device eisa' in your kernel. Warner