> Ivan Kokshaysky wrote:
>>
>> This doesn't work. You will always return success if mask = ~0ULL.

Whoops.  I didn't copy what I thought I was copying:

            (mask & *dev->dma_mask) == mask  /* not "== dma_mask" */

Updated code is appended ... which also provides a BUG()-free
version of dma_set_mask().  I think the only arch/platform hook
needed would be for dma_supported().

- Dave