Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 2 Dec 2002 09:58:30 +0000
From:      Doug Rabson <dfr@nlsystems.com>
To:        Marcel Moolenaar <marcel@xcllnt.net>
Cc:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   Re: PERFORCE change 21719 for review
Message-ID:  <200212020958.30876.dfr@nlsystems.com>
In-Reply-To: <20021201185017.GA4331@dhcp01.pn.xcllnt.net>
References:  <200211302059.gAUKxkZZ077084@repoman.freebsd.org> <200212011011.33999.dfr@nlsystems.com> <20021201185017.GA4331@dhcp01.pn.xcllnt.net>

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

--=20
Doug Rabson=09=09=09=09Mail:  dfr@nlsystems.com
=09=09=09=09=09Phone: +44 20 8348 6160



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe p4-projects" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200212020958.30876.dfr>