Search Linux Wireless

Re: [PATCH 1/2] iwl3945: fix some warnings when compiled without debug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am Montag, 19. Januar 2009 schrieb Johannes Berg:
> On Mon, 2009-01-19 at 09:55 +0100, Helmut Schaa wrote:
> > Fix the following warnings if compiled without CONFIG_IWLWIFI_DEBUG.
> > 
> > drivers/net/wireless/iwlwifi/iwl3945-base.c: In function ‘iwl3945_rx_reply_add_sta’:
> > drivers/net/wireless/iwlwifi/iwl3945-base.c:2748: warning: unused variable ‘pkt’
> > drivers/net/wireless/iwlwifi/iwl3945-base.c: In function ‘iwl3945_rx_scan_results_notif’:
> > drivers/net/wireless/iwlwifi/iwl3945-base.c:2903: warning: unused variable ‘notif’
> > drivers/net/wireless/iwlwifi/iwl3945-base.c: In function ‘iwl3945_rx_scan_complete_notif’:
> > drivers/net/wireless/iwlwifi/iwl3945-base.c:2928: warning: unused variable ‘scan_notif’
> > 
> > Signed-off-by: Helmut Schaa <helmut.schaa@xxxxxxxxxxxxxx>
> > ---
> > diff --git a/drivers/net/wireless/iwlwifi/iwl3945-base.c b/drivers/net/wireless/iwlwifi/iwl3945-base.c
> > index b437cac..15d36d8 100644
> > --- a/drivers/net/wireless/iwlwifi/iwl3945-base.c
> > +++ b/drivers/net/wireless/iwlwifi/iwl3945-base.c
> > @@ -2744,7 +2744,9 @@ static void iwl3945_rx_reply_alive(struct iwl_priv *priv,
> >  static void iwl3945_rx_reply_add_sta(struct iwl_priv *priv,
> >  				 struct iwl_rx_mem_buffer *rxb)
> >  {
> > +	#ifdef CONFIG_IWLWIFI_DEBUG
> >  	struct iwl_rx_packet *pkt = (void *)rxb->skb->data;
> > +	#endif 
> 
> I for one would definitely prefer you didn't indent preprocessor
> statements, but I don't care all that much about iwl3945.

Indeed. My fault. I'll resend this one.

Thanks,
Helmut
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux