Date: Wed, 24 Jan 2001 23:33:17 -0800 From: Peter Wemm <peter@netplex.com.au> To: John Hay <jhay@icomtek.csir.co.za> Cc: imp@harmony.village.org (Warner Losh), jhay@FreeBSD.org (John Hay), cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/isa if_ar.c if_arregs.h src/sys/pci if_ar_p.c Message-ID: <200101250733.f0P7XH490063@mobile.wemm.org> In-Reply-To: <200101250700.f0P70ka99921@zibbi.icomtek.csir.co.za>
next in thread | previous in thread | raw e-mail | index | archive | help
John Hay wrote: > > : Modified files: > > : sys/i386/isa if_ar.c if_arregs.h > > : sys/pci if_ar_p.c > > : Log: > > : Newbusify ar(4). > > > > The convention that we've used in the past is to move the newbusified > > drivers to sys/dev/ar and call the isa attachment if_ar_isa.c and the > > pci attachment if_ar_pci.c. > > Ok, what should I do with i386/isa/ic/hd64570.h? Do we have a common > place for things like that? It isn't really related to ISA and in > this case it will be used by more than one driver. At least ar(4) and > sr(4). This is bikeshed material, but NetBSD put things like that in dev/ic/*. There are others too, eg: if_the wavelan_ieee.h that I tripped over and broke when moving wi to dev/wi (I broke if_wl.c). Some ideas: dev/ic dev/common dev/include .. actually, I think I like the second most. it means we can put shared header files in sys/dev/common and have that automatically installed with the other includes, and any common C code that is shared between drivers (eg: some 3com stuff) as well. If there was a common components there could be, for example: dev/common/hd64570.h dev/common/hd64570.c Can anybody not live with this? ie: dev/common ? Userland could #include <dev/common/if_wavelan_ieee.h>. Cheers, -Peter -- Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au "All of this is for nothing if we don't go to the stars" - JMS/B5 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?200101250733.f0P7XH490063>