Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 6 Oct 2001 13:52:20 -0700
From:      Marcel Moolenaar <marcel@xcllnt.net>
To:        Warner Losh <imp@harmony.village.org>
Cc:        Bruce Evans <bde@zeta.org.au>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/ia64/include cpufunc.h
Message-ID:  <20011006135220.A1606@athlon.pn.xcllnt.net>
In-Reply-To: <200110061947.f96JlU747798@harmony.village.org>
References:  <20011006121654.B68233@kayak.xcllnt.net> <200110060931.f969ViC77564@freefall.freebsd.org> <20011006224619.X1107-100000@delplex.bde.org> <200110061947.f96JlU747798@harmony.village.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Oct 06, 2001 at 01:47:30PM -0600, Warner Losh wrote:
> In message <20011006121654.B68233@kayak.xcllnt.net> Marcel Moolenaar writes:
> : I thought so. What is the right place, then. It seems that bus_memio.h
> : is a good match, but that file is empty for all ports and thus is
> : making me nervous. The other logical place is bus.h. On alpha the
> : functions are there, but that's mostly because of the implementation.
> 
> bus_memio.h means "I'll use memory operations for this driver" and if
> bus_pio.h is also included, that means "I'll use I/o operations for
> this driver."  this allows bus_space stuff to be highly optimized for
> those drivers that cannot possibly talk to memory or pio hardware,
> because such hardware doesn't exist.

Ah, it basicly serves the same purpose as something like:

#define BUS_MEMIO	1
#include <machine/bus.h>

> Don't put anything else in them.  And complain to Justin Gibbs about
> the API, not me, if you don't like it because he invented it :-)

It doesn't really matter what I like. It's just that I'm covering
new areas in the kernel (read: I'm way over my head here :-)
and the purpose is not trivially clear. A note in the header files
would help for example.

-- 
 Marcel Moolenaar	  USPA: A-39004		 marcel@xcllnt.net

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?20011006135220.A1606>