Acked-by: David S. Miller <davem@davemloft.net>
However I'm a little unhappy with how IA64 achieves this.
The last argument for dma_map_foo() is an enum not an int,
every platform other than IA64 properly defines the last
argument as "enum dma_data_direction". It can take one
of several distinct values, it is not a mask.
This hijacking of the DMA direction argument is hokey at
best, and at worst is type bypassing which is going to
explode subtly for someone in the future and result in
a long painful debugging session.
Adding another argument could be painful to do this cleanly, but at
least with inline functions and macros it could just evaluate to
nothing on platforms that don't need it.
Either that, or we should turn the thing into an integer "flags"
across the board and audit every DMA mapping implementation so that it
can handle multiple bits being set. But that's really ugly and
invites mistakes as I detailed above.
-
