Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Dan Williams
On Thu, Sep 12, 2013 at 10:11 AM, Olof Johansson wrote: > On Thu, Sep 12, 2013 at 10:05 AM, Randy Dunlap wrote: >> On 09/11/13 21:38, Olof Johansson wrote: >>> This resolves some warnings seen when building with CONFIG_ARM_LPAE=y, since >>> dma_addr_t might then be 64-bit: >>> >>> drivers/dma/i

Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Olof Johansson
On Thu, Sep 12, 2013 at 10:35 AM, Russell King - ARM Linux wrote: > On Thu, Sep 12, 2013 at 10:11:44AM -0700, Olof Johansson wrote: >> Any >32bit-addressable machine will likely want 64-bit dma_addr_t as >> well. The only architecture that doesn't seem to set >> ARCH_DMA_ADDR_T_64BIT based on PHYS

Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Russell King - ARM Linux
On Thu, Sep 12, 2013 at 10:11:44AM -0700, Olof Johansson wrote: > Any >32bit-addressable machine will likely want 64-bit dma_addr_t as > well. The only architecture that doesn't seem to set > ARCH_DMA_ADDR_T_64BIT based on PHYS_ADDR_T size is ARM, and I think > that should just be changed there as

Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Joe Perches
On Thu, 2013-09-12 at 10:05 -0700, Randy Dunlap wrote: > On 09/11/13 21:38, Olof Johansson wrote: > > This resolves some warnings seen when building with CONFIG_ARM_LPAE=y, since > > dma_addr_t might then be 64-bit: > > > > drivers/dma/imx-sdma.c:1092:3: warning: format '%x' expects argument of

Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Olof Johansson
On Thu, Sep 12, 2013 at 10:05 AM, Randy Dunlap wrote: > On 09/11/13 21:38, Olof Johansson wrote: >> This resolves some warnings seen when building with CONFIG_ARM_LPAE=y, since >> dma_addr_t might then be 64-bit: >> >> drivers/dma/imx-sdma.c:1092:3: warning: format '%x' expects argument of >> t

Re: [PATCH] dma: use %pa to print dma_addr_t

2013-09-12 Thread Randy Dunlap
On 09/11/13 21:38, Olof Johansson wrote: > This resolves some warnings seen when building with CONFIG_ARM_LPAE=y, since > dma_addr_t might then be 64-bit: > > drivers/dma/imx-sdma.c:1092:3: warning: format '%x' expects argument of > type 'unsigned int', but argument 6 has type 'dma_addr_t' [-Wf

[PATCH] dma: use %pa to print dma_addr_t

2013-09-11 Thread Olof Johansson
This resolves some warnings seen when building with CONFIG_ARM_LPAE=y, since dma_addr_t might then be 64-bit: drivers/dma/imx-sdma.c:1092:3: warning: format '%x' expects argument of type 'unsigned int', but argument 6 has type 'dma_addr_t' [-Wformat=] drivers/dma/imx-sdma.c:1166:3: warning: f