Hi Liviu, On Wed, Jan 31, 2018 at 03:08:08AM +0000, Liviu Dudau wrote: > On Fri, Jan 26, 2018 at 11:00:41AM +0800, Yong wrote: > > Hi Maxime, > > > > On Fri, 26 Jan 2018 09:46:58 +0800 > > Yong wrote: > > > > > Hi Maxime, > > > > > > Do you have any experience in solving this problem? > > > It seems the PHYS_OFFSET maybe undeclared when the ARCH is not arm. > > > > Got it. > > Should I add 'depends on ARM' in Kconfig? > > No, I don't think you should do that, you should fix the code. > > The dma_addr_t addr that you've got is ideally coming from dma_alloc_coherent(), > in which case the addr is already "suitable" for use by the device (because the > bus where the device is attached to does all the address translations). Like we're discussing in that other part of the thread with Thierry and Arnd, things are slightly more complicated than that :) In our case, the bus where the device is attached will not do the address translations, and shouldn't. > If you apply PHYS_OFFSET forcefully to it you might get unexpected > results. Out of curiosity, what would be these unexpected results? Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com