On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote: > dma_declare_coherent_memory() takes two addresses for a region of memory: a > "bus_addr" and a "device_addr". I think the intent is that "bus_addr" is > the physical address a *CPU* would use to access the region, and > "device_addr" is the bus address the *device* would use to address the > region. > > Rename "bus_addr" to "phys_addr" and change its type to phys_addr_t. Remind me what the difference between phys_addr_t and dma_addr_t are. I thought phys_addr_t was the maximum address the CPU could reach after address translation and dma_addr_t was the maximum physical address any bus attached memory mapped devices could appear at. (of course, mostly they're the same). The intent of dma_declare_coherent_memory() is to take a range of memory provided by some device on the bus and allow the CPU to allocate regions from it for use as things like descriptors. The CPU treats it as real memory, but, in fact, it is a mapped region of an attached device. If my definition is correct, then bus_addr should be dma_addr_t because it has to be mapped from a device and dma_addr_t is the correct type for device addresses. If I've got the definition wrong, then we should document it somewhere, because it's probably confusing other people as well. James -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
