On 20/06/17 14:24, Christoph Hellwig wrote: >> - sg_dma_address(sg) = (dma_addr_t)virt_to_phys(va); >> + sg_dma_address(sg) = (dma_addr_t)(virt_to_phys(va) - PFN_PHYS(dev->dma_pfn_offset)); > > Needs a line break instead of the overly long line. Will do. > > Otherwise this looks fine to me. > Thanks Vladimir