From owner-freebsd-current Mon Apr 19 11:53:25 1999 Delivered-To: freebsd-current@freebsd.org Received: from lor.watermarkgroup.com (lor.watermarkgroup.com [207.202.73.33]) by hub.freebsd.org (Postfix) with ESMTP id 1E82114D6B for ; Mon, 19 Apr 1999 11:52:46 -0700 (PDT) (envelope-from luoqi@watermarkgroup.com) Received: (from luoqi@localhost) by lor.watermarkgroup.com (8.8.8/8.8.8) id OAA05777; Mon, 19 Apr 1999 14:21:41 -0400 (EDT) (envelope-from luoqi) Date: Mon, 19 Apr 1999 14:21:41 -0400 (EDT) From: Luoqi Chen Message-Id: <199904191821.OAA05777@lor.watermarkgroup.com> To: jhay@mikom.csir.co.za, peter@netplex.com.au Subject: Re: newbus and isa auto irq Cc: current@FreeBSD.ORG, dfr@nlsystems.com Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > This is not happening for the maddr stuff. > I suspect this would do better: > > if (portsize > 0) > isa_set_portsize(dev, portsize); > if (dvp->id_iobase >= 0) > isa_set_port(dev, dvp->id_iobase); > if (dvp->id_irq != 0) > isa_set_irq(dev, ffs(dvp->id_irq) - 1); > if (dvp->id_drq != -1) > isa_set_drq(dev, dvp->id_drq); > if (dvp->id_maddr != 0) > isa_set_maddr(dev, > (int) dvp->id_maddr - KERNBASE); > if (dvp->id_msize != 0) > isa_set_msize(dev, dvp->id_msize); > I'm not sure about the "nothing" value for id_drq though, it might need to > be 0 - but that's a valid dma channel. > > IMHO, isa_release_resources() clearing of the default values for the probe > hints is a timebomb... I thought about adding a seperate store for the > "hint" values rather than using the id_foo[0] entries, and leaving the > tracked resource entries for alloc/free without risking the hints. > > Cheers, > -Peter > Off the topic, I think we should replace (dvp->id_maddr - KERNBASE) with kvtop(dvp->id_maddr), too much assumption about vm layout... -lq To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message