Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Bjorn Helgaas
On Tue, May 6, 2014 at 2:59 AM, Liviu Dudau wrote: > On Mon, May 05, 2014 at 11:42:40PM +0100, Bjorn Helgaas wrote: >> On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: >> > On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: >> > > I'm not a CPU >> > > person, so I don't

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Arnd Bergmann
On Tuesday 06 May 2014 13:42:06 James Bottomley wrote: > On Tue, 2014-05-06 at 07:18 -0600, Bjorn Helgaas wrote: > > On Mon, May 5, 2014 at 8:42 PM, James Bottomley > > wrote: > > > On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: > > >> On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread James Bottomley
On Tue, 2014-05-06 at 07:18 -0600, Bjorn Helgaas wrote: > On Mon, May 5, 2014 at 8:42 PM, James Bottomley > wrote: > > On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: > >> On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: > > > >> > I don't know about NCR_Q720, but all others

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Bjorn Helgaas
On Mon, May 5, 2014 at 8:42 PM, James Bottomley wrote: > On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: >> On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: > >> > I don't know about NCR_Q720, but all others are only used on machines >> > where physical addresses and bus

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Arnd Bergmann
On Tuesday 06 May 2014 02:42:22 James Bottomley wrote: > On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: > > On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: > > > > I don't know about NCR_Q720, but all others are only used on machines > > > where physical addresses and bus

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Liviu Dudau
On Mon, May 05, 2014 at 11:42:40PM +0100, Bjorn Helgaas wrote: > On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: > > On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: > > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > > > wrote: > > > > On Wed, 2014-04-30 at 14:33

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Liviu Dudau
On Mon, May 05, 2014 at 11:42:40PM +0100, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: On Wed, 2014-04-30

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Arnd Bergmann
On Tuesday 06 May 2014 02:42:22 James Bottomley wrote: On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: I don't know about NCR_Q720, but all others are only used on machines where physical addresses and bus addresses

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Bjorn Helgaas
On Mon, May 5, 2014 at 8:42 PM, James Bottomley jbottom...@parallels.com wrote: On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: I don't know about NCR_Q720, but all others are only used on machines where physical

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread James Bottomley
On Tue, 2014-05-06 at 07:18 -0600, Bjorn Helgaas wrote: On Mon, May 5, 2014 at 8:42 PM, James Bottomley jbottom...@parallels.com wrote: On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: I don't know about NCR_Q720, but

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Arnd Bergmann
On Tuesday 06 May 2014 13:42:06 James Bottomley wrote: On Tue, 2014-05-06 at 07:18 -0600, Bjorn Helgaas wrote: On Mon, May 5, 2014 at 8:42 PM, James Bottomley jbottom...@parallels.com wrote: On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 10:42:18AM

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-06 Thread Bjorn Helgaas
On Tue, May 6, 2014 at 2:59 AM, Liviu Dudau liviu.du...@arm.com wrote: On Mon, May 05, 2014 at 11:42:40PM +0100, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: I'm not a CPU person, so I

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread James Bottomley
On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: > On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: > > I don't know about NCR_Q720, but all others are only used on machines > > where physical addresses and bus addresses are in the same space. > > In general, the driver

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread Bjorn Helgaas
On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: > On Friday 02 May 2014 06:11:42 James Bottomley wrote: > > On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: > > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > > > wrote: > > > > On Wed, 2014-04-30 at 14:33 -0600, Bjorn

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread Bjorn Helgaas
On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: > On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > > wrote: > > > On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote: > > >> dma_declare_coherent_memory() takes two

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread Bjorn Helgaas
On Fri, May 02, 2014 at 12:18:07PM +0100, Liviu Dudau wrote: On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote:

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread Bjorn Helgaas
On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: On Friday 02 May 2014 06:11:42 James Bottomley wrote: On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: On Wed, 2014-04-30 at 14:33

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-05 Thread James Bottomley
On Mon, 2014-05-05 at 17:01 -0600, Bjorn Helgaas wrote: On Fri, May 02, 2014 at 10:42:18AM +0200, Arnd Bergmann wrote: I don't know about NCR_Q720, but all others are only used on machines where physical addresses and bus addresses are in the same space. In general, the driver doesn't

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Bjorn Helgaas
[+cc Magnus] On Fri, May 02, 2014 at 06:11:42AM +, James Bottomley wrote: > On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > > wrote: > > > On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote: > > >>

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Liviu Dudau
On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: > [+cc Arnd] > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > wrote: > > 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"

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Liviu Dudau
On Thu, May 01, 2014 at 03:08:12PM +0100, James Bottomley wrote: > 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Arnd Bergmann
On Friday 02 May 2014 06:11:42 James Bottomley wrote: > On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: > > [+cc Arnd] > > > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > > wrote: > > > On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote: > > >> dma_declare_coherent_memory()

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread James Bottomley
On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: > [+cc Arnd] > > On Thu, May 1, 2014 at 8:08 AM, James Bottomley > wrote: > > 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread James Bottomley
On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: [+cc Arnd] On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Arnd Bergmann
On Friday 02 May 2014 06:11:42 James Bottomley wrote: On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: [+cc Arnd] On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote:

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Liviu Dudau
On Thu, May 01, 2014 at 03:08:12PM +0100, James Bottomley wrote: 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Liviu Dudau
On Thu, May 01, 2014 at 08:05:04PM +0100, Bjorn Helgaas wrote: [+cc Arnd] On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-02 Thread Bjorn Helgaas
[+cc Magnus] On Fri, May 02, 2014 at 06:11:42AM +, James Bottomley wrote: On Thu, 2014-05-01 at 13:05 -0600, Bjorn Helgaas wrote: On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: On Wed, 2014-04-30 at 14:33 -0600, Bjorn Helgaas wrote:

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread Bjorn Helgaas
[+cc Arnd] On Thu, May 1, 2014 at 8:08 AM, James Bottomley wrote: > 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread James Bottomley
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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread Bjorn Helgaas
On Wed, Apr 30, 2014 at 6:07 PM, Greg Kroah-Hartman wrote: > On Wed, Apr 30, 2014 at 02:33:21PM -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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread Bjorn Helgaas
On Wed, Apr 30, 2014 at 6:07 PM, Greg Kroah-Hartman gre...@linuxfoundation.org wrote: On Wed, Apr 30, 2014 at 02:33:21PM -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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread James Bottomley
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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-05-01 Thread Bjorn Helgaas
[+cc Arnd] On Thu, May 1, 2014 at 8:08 AM, James Bottomley jbottom...@parallels.com wrote: 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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-04-30 Thread Greg Kroah-Hartman
On Wed, Apr 30, 2014 at 02:33:21PM -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 >

[PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-04-30 Thread Bjorn Helgaas
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.

[PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-04-30 Thread Bjorn Helgaas
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

Re: [PATCH] DMA-API: Change dma_declare_coherent_memory() CPU address to phys_addr_t

2014-04-30 Thread Greg Kroah-Hartman
On Wed, Apr 30, 2014 at 02:33:21PM -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