Marek Szyprowski wrote: > On Friday, June 29, 2012 1:10 PM Clemens Ladisch wrote: >> Marek Szyprowski wrote: >>> +++ b/drivers/base/dma-mapping.c >>> ... >>> +int dma_common_mmap(struct device *dev, struct vm_area_struct *vma, >>> + void *cpu_addr, dma_addr_t dma_addr, size_t size) >>> +{ >>> + int ret = -ENXIO; >>> + ... >>> + if (dma_mmap_from_coherent(dev, vma, cpu_addr, size, &ret)) >>> + return ret; >> >> This will return -ENXIO if dma_mmap_from_coherent() succeeds. > > Thanks for spotting this! Sorry, I was wrong; ret is actually set by dma_mmap_from_coherent's output parameter. (That function's documentation appears to be incomplete.) Regards, Clemens -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>