From owner-cvs-all Tue Sep 7 11:33:35 1999 Delivered-To: cvs-all@freebsd.org Received: from sasami.jurai.net (sasami.jurai.net [63.67.141.99]) by hub.freebsd.org (Postfix) with ESMTP id 54A1114E1D; Tue, 7 Sep 1999 11:33:29 -0700 (PDT) (envelope-from winter@jurai.net) Received: from localhost (winter@localhost) by sasami.jurai.net (8.8.8/8.8.7) with ESMTP id OAA03193; Tue, 7 Sep 1999 14:32:50 -0400 (EDT) Date: Tue, 7 Sep 1999 14:32:49 -0400 (EDT) From: "Matthew N. Dodd" To: Bruce Evans Cc: dfr@nlsystems.com, peter@netplex.com.au, bde@zeta.org.au, cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/alpha/conf files.alpha src/sys/i386/conf files.i386 src/sys/conf files src/sys/dev/sio sio.c src/sys/isa sio.c sioreg.h In-Reply-To: <19990907182328.2644614C44@hub.freebsd.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk On Tue, 7 Sep 1999, Bruce Evans wrote: > Right now I'm only objecting to the misorganisation in /dev. I have > objected to it before :-). I don't want directories with 1 or 2 files > in them. The bus-independent part of the sio driver probably needs 2 > files, sio.c and siovar.h (not the existing sioreg.h). Right now sys/dev/sio has 1 or 2 files. In the future it will have as many as 10. sio_isa.c sio_pci.c sio_mca.c sio_eisa.c etc... I like this better than having sys/dev/{isa,pci,mca,eisa,tc,foo}/sio.c Granted, it might be nice to have some reasonable hierarchy under sys/dev/ but so far there haven't been any suggestions that anyone has not had a problem with. (This issue was discussed a good bit on #bsdcode with Mike, Peter, Doug and others and we concluded that sys/dev/foo/foo_{isa,eisa,pci,mca,foo}.c was about the cleanest way to go. Keeping all the files for a particular driver in 1 place and not all spread out makes propogating changes to the various files a little easier to manage no? -- | Matthew N. Dodd | '78 Datsun 280Z | '75 Volvo 164E | FreeBSD/NetBSD | | winter@jurai.net | 2 x '84 Volvo 245DL | ix86,sparc,pmax | | http://www.jurai.net/~winter | This Space For Rent | ISO8802.5 4ever | To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message