netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] uninitialized variable in prism54 isl38xx_trigger_device
@ 2005-05-25 23:16 Olaf Hering
  2005-06-16 23:52 ` Luis R. Rodriguez
  0 siblings, 1 reply; 2+ messages in thread
From: Olaf Hering @ 2005-05-25 23:16 UTC (permalink / raw)
  To: Jeff Garzik, netdev, prism54-private


drivers/net/wireless/prism54/isl_38xx.c:131: warning: 'current_time.tv_sec' is used uninitialized in this function
drivers/net/wireless/prism54/isl_38xx.c:131: warning: 'current_time.tv_usec' is used uninitialized in this function

Signed-off-by: Olaf Hering <olh@suse.de>
Index: linux-2.6.12-rc5-olh/drivers/net/wireless/prism54/isl_38xx.c
===================================================================
--- linux-2.6.12-rc5-olh.orig/drivers/net/wireless/prism54/isl_38xx.c
+++ linux-2.6.12-rc5-olh/drivers/net/wireless/prism54/isl_38xx.c
@@ -112,10 +112,10 @@ isl38xx_handle_wakeup(isl38xx_control_bl
 void
 isl38xx_trigger_device(int asleep, void __iomem *device_base)
 {
-	struct timeval current_time;
 	u32 reg, counter = 0;
 
 #if VERBOSE > SHOW_ERROR_MESSAGES
+	struct timeval current_time;
 	DEBUG(SHOW_FUNCTION_CALLS, "isl38xx trigger device\n");
 #endif
 
@@ -126,11 +126,11 @@ isl38xx_trigger_device(int asleep, void 
 		do_gettimeofday(&current_time);
 		DEBUG(SHOW_TRACING, "%08li.%08li Device wakeup triggered\n",
 		      current_time.tv_sec, (long)current_time.tv_usec);
-#endif
 
 		DEBUG(SHOW_TRACING, "%08li.%08li Device register read %08x\n",
 		      current_time.tv_sec, (long)current_time.tv_usec,
 		      readl(device_base + ISL38XX_CTRL_STAT_REG));
+#endif
 		udelay(ISL38XX_WRITEIO_DELAY);
 
 		reg = readl(device_base + ISL38XX_INT_IDENT_REG);
@@ -148,10 +148,12 @@ isl38xx_trigger_device(int asleep, void 
 				counter++;
 			}
 
+#if VERBOSE > SHOW_ERROR_MESSAGES
 			DEBUG(SHOW_TRACING,
 			      "%08li.%08li Device register read %08x\n",
 			      current_time.tv_sec, (long)current_time.tv_usec,
 			      readl(device_base + ISL38XX_CTRL_STAT_REG));
+#endif
 			udelay(ISL38XX_WRITEIO_DELAY);
 
 #if VERBOSE > SHOW_ERROR_MESSAGES

^ permalink raw reply	[flat|nested] 2+ messages in thread
* Re: [PATCH] uninitialized variable in prism54 isl38xx_trigger_device
  2005-05-25 23:16 [PATCH] uninitialized variable in prism54 isl38xx_trigger_device Olaf Hering
@ 2005-06-16 23:52 ` Luis R. Rodriguez
  0 siblings, 0 replies; 2+ messages in thread
From: Luis R. Rodriguez @ 2005-06-16 23:52 UTC (permalink / raw)
  To: Olaf Hering; +Cc: Jeff Garzik, netdev, prism54-private

Sure, why not. This has been applied to prism54 svn tree.

On 5/25/05, Olaf Hering <olh@suse.de> wrote:
> 
> drivers/net/wireless/prism54/isl_38xx.c:131: warning: 'current_time.tv_sec' is used uninitialized in this function
> drivers/net/wireless/prism54/isl_38xx.c:131: warning: 'current_time.tv_usec' is used uninitialized in this function
> 
> Signed-off-by: Olaf Hering <olh@suse.de>
> Index: linux-2.6.12-rc5-olh/drivers/net/wireless/prism54/isl_38xx.c
> ===================================================================
> --- linux-2.6.12-rc5-olh.orig/drivers/net/wireless/prism54/isl_38xx.c
> +++ linux-2.6.12-rc5-olh/drivers/net/wireless/prism54/isl_38xx.c
> @@ -112,10 +112,10 @@ isl38xx_handle_wakeup(isl38xx_control_bl
>  void
>  isl38xx_trigger_device(int asleep, void __iomem *device_base)
>  {
> -       struct timeval current_time;
>         u32 reg, counter = 0;
> 
>  #if VERBOSE > SHOW_ERROR_MESSAGES
> +       struct timeval current_time;
>         DEBUG(SHOW_FUNCTION_CALLS, "isl38xx trigger device\n");
>  #endif
> 
> @@ -126,11 +126,11 @@ isl38xx_trigger_device(int asleep, void
>                 do_gettimeofday(&current_time);
>                 DEBUG(SHOW_TRACING, "%08li.%08li Device wakeup triggered\n",
>                       current_time.tv_sec, (long)current_time.tv_usec);
> -#endif
> 
>                 DEBUG(SHOW_TRACING, "%08li.%08li Device register read %08x\n",
>                       current_time.tv_sec, (long)current_time.tv_usec,
>                       readl(device_base + ISL38XX_CTRL_STAT_REG));
> +#endif
>                 udelay(ISL38XX_WRITEIO_DELAY);
> 
>                 reg = readl(device_base + ISL38XX_INT_IDENT_REG);
> @@ -148,10 +148,12 @@ isl38xx_trigger_device(int asleep, void
>                                 counter++;
>                         }
> 
> +#if VERBOSE > SHOW_ERROR_MESSAGES
>                         DEBUG(SHOW_TRACING,
>                               "%08li.%08li Device register read %08x\n",
>                               current_time.tv_sec, (long)current_time.tv_usec,
>                               readl(device_base + ISL38XX_CTRL_STAT_REG));
> +#endif
>                         udelay(ISL38XX_WRITEIO_DELAY);
> 
>  #if VERBOSE > SHOW_ERROR_MESSAGES
>

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2005-06-16 23:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-05-25 23:16 [PATCH] uninitialized variable in prism54 isl38xx_trigger_device Olaf Hering
2005-06-16 23:52 ` Luis R. Rodriguez

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).