[PATCH -next] pcmcia: fix controller printk warnings

Randy Dunlap randy.dunlap at oracle.com
Fri Sep 18 11:01:02 EDT 2009


On Fri, 18 Sep 2009 09:05:45 +0200 Wolfram Sang wrote:

> On Thu, Sep 17, 2009 at 02:14:23PM -0700, Randy Dunlap wrote:
> > From: Randy Dunlap <randy.dunlap at oracle.com>
> > 
> > Fix new pcmcia printk format warnings:
> > 
> > drivers/pcmcia/i82365.c:1055: warning: format '%#x' expects type 'unsigned int', but argument 6 has type 'phys_addr_t'
> > drivers/pcmcia/i82365.c:1055: warning: format '%#x' expects type 'unsigned int', but argument 7 has type 'phys_addr_t'
> > drivers/pcmcia/tcic.c:734: warning: format '%#x' expects type 'unsigned int', but argument 6 has type 'phys_addr_t'
> > drivers/pcmcia/tcic.c:734: warning: format '%#x' expects type 'unsigned int', but argument 7 has type 'phys_addr_t'
> > 
> > Signed-off-by: Randy Dunlap <randy.dunlap at oracle.com>
> Acked-by: Wolfram Sang <w.sang at pengutronix.de>
> 
> I spotted another four drivers needing a similar patch (soc_common, m32r_* and
> m8xx_pcmcia). Do you want to add them, shall I make a patch on top of this one
> or incorporate all into one patch?

You add those, please.  Thanks.


> > ---
> >  drivers/pcmcia/i82365.c |    4 ++--
> >  drivers/pcmcia/tcic.c   |    4 ++--
> >  2 files changed, 4 insertions(+), 4 deletions(-)
> > 
> > --- linux-next-20090917.orig/drivers/pcmcia/tcic.c
> > +++ linux-next-20090917/drivers/pcmcia/tcic.c
> > @@ -732,8 +732,8 @@ static int tcic_set_io_map(struct pcmcia
> >      u_short base, len, ioctl;
> >      
> >      debug(1, "SetIOMap(%d, %d, %#2.2x, %d ns, "
> > -	  "%#x-%#x)\n", psock, io->map, io->flags,
> > -	  io->speed, io->start, io->stop);
> > +	  "%#llx-%#llx)\n", psock, io->map, io->flags, io->speed,
> > +	  (unsigned long long)io->start, (unsigned long long)io->stop);
> >      if ((io->map > 1) || (io->start > 0xffff) || (io->stop > 0xffff) ||
> >  	(io->stop < io->start)) return -EINVAL;
> >      tcic_setw(TCIC_ADDR+2, TCIC_ADR2_INDREG | (psock << TCIC_SS_SHFT));
> > --- linux-next-20090917.orig/drivers/pcmcia/i82365.c
> > +++ linux-next-20090917/drivers/pcmcia/i82365.c
> > @@ -1053,8 +1053,8 @@ static int i365_set_io_map(u_short sock,
> >      u_char map, ioctl;
> >      
> >      debug(1, "SetIOMap(%d, %d, %#2.2x, %d ns, "
> > -	  "%#x-%#x)\n", sock, io->map, io->flags,
> > -	  io->speed, io->start, io->stop);
> > +	  "%#llx-%#llx)\n", sock, io->map, io->flags, io->speed,
> > +	  (unsigned long long)io->start, (unsigned long long)io->stop);
> >      map = io->map;
> >      if ((map > 1) || (io->start > 0xffff) || (io->stop > 0xffff) ||
> >  	(io->stop < io->start)) return -EINVAL;
> 
> -- 
> Pengutronix e.K.                           | Wolfram Sang                |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |


---
~Randy
LPC 2009, Sept. 23-25, Portland, Oregon
http://linuxplumbersconf.org/2009/



More information about the linux-pcmcia mailing list