Subject: Re: CVS commit: syssrc
To: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
From: Matthew Jacob <mjacob@feral.com>
List: source-changes
Date: 02/24/2001 19:16:51
> > mjacob@netbsd.org wrote:
> > 
> > > Modified Files:
> > > 	syssrc/sys/dev/sbus: isp_sbus.c
> > > 
> > > Log Message:
> > > Add missing BUS_DMA_COHERENT to bus_dmamap_load_raw calls for request
> > > and response queues- this makes this work on UltraSparc now.
> > 
> > Can bus_dmamap_load*() functions take BUS_DMA_COHERENT flag?
> > Isn't it valid for only bus_dmamem_[m]map() ?
> > Anyway BUS_DMA_COHERENT flag is just a hint,
> > so it should not be "required", I think.
> > (bus_dmamap_sync() is missing in some place?)

It turns out there are a number of other drivers that are adding
BUS_DMA_COHERENT to the bus_dmamap_load* calls as well. Guess they need fixing
too.

-matt