netdev
[Top] [All Lists]

Re: [PATCH] prism54: fix printk format warnings

To: "Randy.Dunlap" <rddunlap@xxxxxxxx>
Subject: Re: [PATCH] prism54: fix printk format warnings
From: "Luis R. Rodriguez" <mcgrof@xxxxxxxxx>
Date: Sun, 17 Apr 2005 18:24:53 -0400
Cc: netdev@xxxxxxxxxxx, jgarzik <jgarzik@xxxxxxxxx>, mcgrof@xxxxxxxxxxxxxxxxxx, prism54-devel@xxxxxxxxxxx
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references; b=Q2aRe1OcmIeM60GF681Z27O1WYA+G6cvL3tuYNn5SeM9I0Bx5eWwcu462T7lOXbwXF+Ije7TW9fwGezqCI2Eh6QpSq+G6Cg+PqfPInmf9I1cBhLodHBhVl7lMfZRYe+9AgVBlQXJ8tSjTYbcODphrIjg67sh+KN5TtJvf54joQc=
In-reply-to: <20050301212137.57f9532f.rddunlap@xxxxxxxx>
References: <20050301212137.57f9532f.rddunlap@xxxxxxxx>
Reply-to: "Luis R. Rodriguez" <mcgrof@xxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
Thanks Randy. This is already in prism54 svn tree. Feel free to apply Jeff.

  Luis

On 3/2/05, Randy.Dunlap <rddunlap@xxxxxxxx> wrote:
> 
> prism54 build shows some printk format complaints:
> (sparc64 build warning)
> 
> drivers/net/wireless/prism54/isl_38xx.c:131: warning: long int format, 
> different type arg (arg 3)
> drivers/net/wireless/prism54/isl_38xx.c:151: warning: long int format, 
> different type arg (arg 3)
> 
> cross-compile results:
> https://www.osdl.org/plm-cgi/plm?module=patch_info&patch_id=4240
> 
> Signed-off-by: Randy Dunlap <rddunlap@xxxxxxxx>
> 
> diffstat:=
>  drivers/net/wireless/prism54/isl_38xx.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff -Naurp ./drivers/net/wireless/prism54/isl_38xx.c~prism_printk 
> ./drivers/net/wireless/prism54/isl_38xx.c
> --- ./drivers/net/wireless/prism54/isl_38xx.c~prism_printk      2004-12-24 
> 13:34:45.000000000 -0800
> +++ ./drivers/net/wireless/prism54/isl_38xx.c   2005-03-01 20:15:00.189995120 
> -0800
> @@ -125,11 +125,11 @@ isl38xx_trigger_device(int asleep, void
>  #if VERBOSE > SHOW_ERROR_MESSAGES
>                 do_gettimeofday(&current_time);
>                 DEBUG(SHOW_TRACING, "%08li.%08li Device wakeup triggered\n",
> -                     current_time.tv_sec, current_time.tv_usec);
> +                     current_time.tv_sec, (long)current_time.tv_usec);
>  #endif
> 
>                 DEBUG(SHOW_TRACING, "%08li.%08li Device register read %08x\n",
> -                     current_time.tv_sec, current_time.tv_usec,
> +                     current_time.tv_sec, (long)current_time.tv_usec,
>                       readl(device_base + ISL38XX_CTRL_STAT_REG));
>                 udelay(ISL38XX_WRITEIO_DELAY);
> 
> @@ -139,7 +139,7 @@ isl38xx_trigger_device(int asleep, void
>                         do_gettimeofday(&current_time);
>                         DEBUG(SHOW_TRACING,
>                               "%08li.%08li Device register abadface\n",
> -                             current_time.tv_sec, current_time.tv_usec);
> +                             current_time.tv_sec, 
> (long)current_time.tv_usec);
>  #endif
>                         /* read the Device Status Register until Sleepmode 
> bit is set */
>                         while (reg = readl(device_base + 
> ISL38XX_CTRL_STAT_REG),
> @@ -150,7 +150,7 @@ isl38xx_trigger_device(int asleep, void
> 
>                         DEBUG(SHOW_TRACING,
>                               "%08li.%08li Device register read %08x\n",
> -                             current_time.tv_sec, current_time.tv_usec,
> +                             current_time.tv_sec, (long)current_time.tv_usec,
>                               readl(device_base + ISL38XX_CTRL_STAT_REG));
>                         udelay(ISL38XX_WRITEIO_DELAY);
> 
> @@ -158,7 +158,7 @@ isl38xx_trigger_device(int asleep, void
>                         do_gettimeofday(&current_time);
>                         DEBUG(SHOW_TRACING,
>                               "%08li.%08li Device asleep counter %i\n",
> -                             current_time.tv_sec, current_time.tv_usec,
> +                             current_time.tv_sec, (long)current_time.tv_usec,
>                               counter);
>  #endif
>                 }
> @@ -174,7 +174,7 @@ isl38xx_trigger_device(int asleep, void
>  #if VERBOSE > SHOW_ERROR_MESSAGES
>                 do_gettimeofday(&current_time);
>                 DEBUG(SHOW_TRACING, "%08li.%08li Device register read %08x\n",
> -                     current_time.tv_sec, current_time.tv_usec, reg);
> +                     current_time.tv_sec, (long)current_time.tv_usec, reg);
>  #endif
>         } else {
>                 /* device is (still) awake  */
> 
> ---
>


<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH] prism54: fix printk format warnings, Luis R. Rodriguez <=