bus_get_dma_tag now necessary for drivers using bus_dma
Hans Petter Selasky
hselasky at c2i.net
Tue Sep 5 19:11:43 UTC 2006
On Tuesday 05 September 2006 20:31, Scott Long wrote:
> Poul-Henning Kamp wrote:
> > In message <20060905173334.GH9421 at funkthat.com>, John-Mark Gurney writes:
> >>This means that for each call to bus_dma_tag_create, instead of passing
> >>a NULL pointer, you should call bus_get_dma_tag(yourdev) and use that
> >>as the parent tag. I committed some example code to various drivers,
> >>such as ahc, ata, em, and ohci.
> >
> > If there are never any exeptions to this requirement, why not
> > pass "yourdev" and have the magic hidden, rather than add 17
> > ritual characters to the API ?
>
> Drivers can have their own multi-level tag heirarchy. You'd have to
> create a new bus_dma_tag_create() variant that expected a device_t
> instead of a bus_dma_tag_t. This might be a good idea.
>
I think it is easier to share code with NetBSD if we use
"bus_get_dma_tag(yourdev)".
For example I use the following prototype on NetBSD and FreeBSD:
void *
usbd_mem_alloc(bus_dma_tag_t parent, u_int32_t size,
u_int8_t align_power);
--HPS
More information about the freebsd-drivers
mailing list