[PATCH] dma: mv_xor: Silence a bunch of LPAE-related warnings

Dan Williams dan.j.williams at intel.com
Tue Feb 4 13:57:48 EST 2014


On Tue, Feb 4, 2014 at 10:30 AM, Olof Johansson <olof at lixom.net> wrote:
> On Tue, Feb 4, 2014 at 9:00 AM, Jason Cooper <jason at lakedaemon.net> wrote:
>> On Tue, Feb 04, 2014 at 10:53:29AM +0530, Vinod Koul wrote:
>>> On Mon, Feb 03, 2014 at 05:13:23PM -0800, Olof Johansson wrote:
>>> > Enabling some of the mvebu platforms in the multiplatform config for ARM
>>> > enabled these drivers, which also triggered a bunch of warnings when LPAE
>>> > is enabled (thus making phys_addr_t 64-bit).
>>> >
>>> > Most changes are switching printk formats, but also a bit of changes to what
>>> > used to be array-based pointer arithmetic that could just be done with the
>>> > address types instead.
>>> >
>>> > The warnings were:
>>> >
>>> > drivers/dma/mv_xor.c: In function 'mv_xor_tx_submit':
>>> > drivers/dma/mv_xor.c:500:3: warning: format '%x' expects argument of type
>>> >     'unsigned int', but argument 4 has type 'dma_addr_t' [-Wformat]
>>> > drivers/dma/mv_xor.c: In function 'mv_xor_alloc_chan_resources':
>>> > drivers/dma/mv_xor.c:553:13: warning: cast to pointer from integer of
>>> >     different size [-Wint-to-pointer-cast]
>>> > drivers/dma/mv_xor.c:555:4: warning: cast from pointer to integer of
>>> >     different size [-Wpointer-to-int-cast]
>>> > drivers/dma/mv_xor.c: In function 'mv_xor_prep_dma_memcpy':
>>> > drivers/dma/mv_xor.c:584:2: warning: format '%x' expects argument of type
>>> >     'unsigned int', but argument 5 has type 'dma_addr_t' [-Wformat]
>>> > drivers/dma/mv_xor.c:584:2: warning: format '%x' expects argument of type
>>> >     'unsigned int', but argument 6 has type 'dma_addr_t' [-Wformat]
>>> > drivers/dma/mv_xor.c: In function 'mv_xor_prep_dma_xor':
>>> > drivers/dma/mv_xor.c:628:2: warning: format '%u' expects argument of type
>>> >     'unsigned int', but argument 7 has type 'dma_addr_t' [-Wformat]
>>> >
>>> > Signed-off-by: Olof Johansson <olof at lixom.net>
>>> Acked-by: Vinod Koul <vinod.koul at intel.com>
>>
>> Olof, would you like me to queue it up?  Or do you want to take it
>> directly?
>>
>> Acked-by: Jason Cooper <jason at lakedaemon.net>
>
> I'm confused. I sent the patch to the drivers/dma maintainers and they
> just acked it without asking me to pick it up myself.
>
> Vinod, did you ack it for me to pick it up, or for some other reason?
> If you don't want to take it through your tree I'll be happy to take
> it through arm-soc, just looking to clarify.
>
> (Jason, I can apply it directly)

Vinod will correct me if I am wrong, but I take his acks to mean "yup,
take it through your tree".  For this specific problem I thought we
had a comprehensive fix pending from Joe Perches to add a new %pX
format specifier for dma addresses?



More information about the linux-arm-kernel mailing list