From owner-p4-projects Mon Dec 2 15:40:38 2002 Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 76E6D37B404; Mon, 2 Dec 2002 15:40:36 -0800 (PST) Delivered-To: perforce@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 2515B37B401 for ; Mon, 2 Dec 2002 15:40:36 -0800 (PST) Received: from canning.wemm.org (canning.wemm.org [192.203.228.65]) by mx1.FreeBSD.org (Postfix) with ESMTP id C385F43EA9 for ; Mon, 2 Dec 2002 15:40:35 -0800 (PST) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by canning.wemm.org (Postfix) with ESMTP id 6BB862A8A8; Mon, 2 Dec 2002 15:40:35 -0800 (PST) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Doug Rabson Cc: Marcel Moolenaar , Perforce Change Reviews Subject: Re: PERFORCE change 21719 for review In-Reply-To: <200212020958.30876.dfr@nlsystems.com> Date: Mon, 02 Dec 2002 15:40:35 -0800 From: Peter Wemm Message-Id: <20021202234035.6BB862A8A8@canning.wemm.org> Sender: owner-p4-projects@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Doug Rabson wrote: > On Sunday 01 December 2002 6:50 pm, Marcel Moolenaar wrote: > > On Sun, Dec 01, 2002 at 10:11:33AM +0000, Doug Rabson wrote: > > > On Saturday 30 November 2002 8:59 pm, Marcel Moolenaar wrote: > > > > http://perforce.freebsd.org/chv.cgi?CH=3D21719 > > > > > > > > Change 21719 by marcel@marcel_nfs on 2002/11/30 12:58:56 > > > > > > > > =09Remove isa and BOOTP_*. > > > > =09Comment out sio. > > > > > > > > =09The UART hardware is not supported by the sio driver. It is > > > > =09probed with some hackery, but the sio driver is in essense > > > > =09too ISA/i386 oriented (pretty much like fb/vga/sc). > > > > > > In what way? > > > > It assumes the UART uses I/O. The isa_irq_pending() function is also > > an example of an ISA dependency. The latter does not prevent the sio > > driver from working, but it does cause an annoying message at boot. > > > > > The sio driver itself just assumes that it can use > > > bus_space to access a standard 16550 uart or similar. The bus > > > attachment code (sio_isa, sio_pccard, sio_ebus etc.) contains any > > > code which is bus-related. This driver works quite well on five > > > different busses - what does the HP machine do that is different? > > > > For one, it hasn't got any ISA busses. Secondly, the Diva comm board > > is memory mapped. In sioprobe() we assume I/O. It's also a multiport > > board and we also don't seem to have the framework yet to just tell > > it the characteristics of this board as the Linux driver has. > > I'm sorry - I thought that the thing was converted to bus_space when it=20 > grew all the non-isa bus attachments. This really needs to happen to=20 > make the driver portable. I'm not sure what to do with the=20 > isa_irq_pending call - probably migrate it to the isa attachment. The division of labor is pretty broken the last time I checked. isa_irq_pending() is how bde checks that the interrupts are actually working on the isa bus. This stuff should be in sio_isa.c only, not in the common probe/attach routines. sio_pci.c should be able to say "trust me, it is irq N". Cheers, -Peter -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "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 p4-projects" in the body of the message