Date: Fri, 19 Jun 1998 00:51:57 +1000 From: Bruce Evans <bde@zeta.org.au> To: bde@FreeBSD.ORG, dfr@nlsystems.com Cc: cvs-all@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, cvs-sys@FreeBSD.ORG Subject: Re: cvs commit: src/sys/i386/i386 machdep.c src/sys/i386/include conf.h src/sys/i386/isa if_wl.c isa_device.h src/sys/i386/isa/sound ad1848.c src/sys/pci ide_pci.c src/sys/sys conf.h Message-ID: <199806181451.AAA00492@godzilla.zeta.org.au>
next in thread | raw e-mail | index | archive | help
>> Log: >> Don't declare isa device structs or isa interrupt handlers in <sys/conf>, >> and don't depend on them being declared there. This will cause lots of >> warnings for a few minutes until config is updated. Interrupt handlers >> should never have been configured by config, and the machine generated >> declarations get in the way of changing the arg type from int to void *. >... >Does this mean that the individual drivers register their interrupt >handlers during attach? I haven't decided. It should be possible, and I currently call int isa_intr_establish(struct isa_device *idp, intrmask_t *maskp, inthand2_t *handler, void *arg); from the driver attach routines for the few drivers that I've converted, but a default of automatic registration on successful attach based on the contents of *idp has some advantages (such as not affecting pccard and pnp configuration immediately). Bruce 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?199806181451.AAA00492>