Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 7 Sep 1999 14:32:49 -0400 (EDT)
From:      "Matthew N. Dodd" <winter@jurai.net>
To:        Bruce Evans <bde@FreeBSD.org>
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
Message-ID:  <Pine.BSF.4.10.9909071428150.14497-100000@sasami.jurai.net>
In-Reply-To: <19990907182328.2644614C44@hub.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.10.9909071428150.14497-100000>