lkml.org 
[lkml]   [2014]   [Jul]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH -next] dma/sh: fix rcar-dma.c printk format warnings
Date
On Saturday 26 July 2014 21:19:30 Vinod Koul wrote:
> On Fri, Jul 25, 2014 at 10:02:39AM -0700, Randy Dunlap wrote:
> > From: Randy Dunlap <rdunlap@infradead.org>
> >
> > Fix printk format warnings by using size_t modifiers ('z'):
> >
> > ../drivers/dma/sh/rcar-dmac.c:759:4: warning: format '%u' expects argument
> > of type 'unsigned int', but argument 9 has type 'size_t' [-Wformat=]
> > ../drivers/dma/sh/rcar-dmac.c:759:4: warning: format '%u' expects
> > argument of type 'unsigned int', but argument 10 has type 'size_t'
> > [-Wformat=]
>
> This is not present in dmaengine trees. Simon is this in your tree?

The patch applies to a new driver under development that Simon took in his
tree. From what he told me he plans to send a pull request soon.

> > Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > Cc: Dan Williams <dan.j.williams@intel.com>
> > Cc: dmaengine@vger.kernel.org

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Simon, should I submit a new version that incorporates this change, or do you
want to include it in your pull request directly (possibly squashing it in the
original commit) ?

> > ---
> >
> > drivers/dma/sh/rcar-dmac.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Index: linux-next-20140725/drivers/dma/sh/rcar-dmac.c
> > ===================================================================
> > --- linux-next-20140725.orig/drivers/dma/sh/rcar-dmac.c
> > +++ linux-next-20140725/drivers/dma/sh/rcar-dmac.c
> > @@ -757,7 +757,7 @@ rcar_dmac_chan_prep_sg(struct rcar_dmac_
> > hwdesc->size = size;
> >
> > dev_dbg(chan->chan.device->dev,
> > - "chan%u: hwdesc %p/%p sgl %u@%p, %u/%u %pad -> %pad\n",
> > + "chan%u: hwdesc %p/%p sgl %u@%p, %zu/%zu %pad -> %pad\n",
> > chan->index, hwdesc, desc, i, sg, size, len,
> > &hwdesc->src_addr, &hwdesc->dst_addr);

--
Regards,

Laurent Pinchart



\
 
 \ /
  Last update: 2014-07-28 14:01    [W:0.046 / U:0.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site