cvs commit: src/sys/dev/uart uart_cpu_pc98.c

M. Warner Losh imp at bsdimp.com
Thu Sep 11 09:41:54 PDT 2003


In message: <20030911.204333.41730710.nyan at jp.FreeBSD.org>
            Takahashi Yoshihiro <nyan at jp.FreeBSD.org> writes:
: In article <200309110414.h8B4ERi2062520 at repoman.freebsd.org>
: Warner Losh <imp at FreeBSD.org> writes:
: 
: > imp         2003/09/10 21:14:27 PDT
: > 
: >   FreeBSD src repository
: > 
: >   Modified files:
: >     sys/dev/uart         uart_cpu_pc98.c 
: >   Log:
: >   Fix compile on pc98.  Maybe this is correct.
: 
: It should not call i386_bus_space_handle_alloc() directly in device
: driver.  I think that we need to implement bus_space_map() function.

The functions in uart_cpu_* are used for the console port support
before the device system is up and running.  It is currently abusing
bus space handles a little bit here.  Since we're only planning on
supporting console ports on the first two serial ports, the number of
different devices we need to support is sufficiently small that I
thought this abuse was OK.

I mostly wanted to get pc98 compiling again with my commit, so if
there's a more appropriate way to do so, please feel free to instruct
me.

Warner


More information about the cvs-src mailing list