linux-kernel - Re: [PATCH -next] dma/sh: fix rcar-dma.c printk format warnings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 28 Jul 2014 13:20:59 +0200
From:	Laurent Pinchart <laurent.pinchart@...asonboard.com>
To:	Vinod Koul <vinod.koul@...el.com>
Cc:	Randy Dunlap <rdunlap@...radead.org>,
	Simon Horman <horms@...ge.net.au>,
	"dmaengine@...r.kernel.org" <dmaengine@...r.kernel.org>,
	Dan Williams <dan.j.williams@...el.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH -next] dma/sh: fix rcar-dma.c printk format warnings

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@...radead.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@...radead.org>
> > Cc: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> > Cc: Dan Williams <dan.j.williams@...el.com>
> > Cc: dmaengine@...r.kernel.org

Acked-by: Laurent Pinchart <laurent.pinchart@...asonboard.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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/