From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752314Ab2LWRbd (ORCPT ); Sun, 23 Dec 2012 12:31:33 -0500 Received: from mms2.broadcom.com ([216.31.210.18]:4409 "EHLO mms2.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751511Ab2LWRbb convert rfc822-to-8bit (ORCPT ); Sun, 23 Dec 2012 12:31:31 -0500 X-Server-Uuid: 4500596E-606A-40F9-852D-14843D8201B2 Message-ID: <50D73FE1.6080907@broadcom.com> Date: Sun, 23 Dec 2012 18:31:13 +0100 From: "Arend van Spriel" User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/17.0 Thunderbird/17.0 MIME-Version: 1.0 To: "Geert Uytterhoeven" cc: "Piotr Haber" , "John W. Linville" , linux-wireless@vger.kernel.org, brcm80211-dev-list@broadcom.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] brcmsmac: initialize morepending in brcms_b_recv() References: <1356210434-28435-1-git-send-email-geert@linux-m68k.org> In-Reply-To: <1356210434-28435-1-git-send-email-geert@linux-m68k.org> X-Enigmail-Version: 1.4.6 X-WSS-ID: 7CC9E0BF3R026456104-02-01 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/22/2012 10:07 PM, Geert Uytterhoeven wrote: > drivers/net/wireless/brcm80211/brcmsmac/main.c: In function ‘brcms_b_recv’: > drivers/net/wireless/brcm80211/brcmsmac/main.c:7636: warning: ‘morepending’ may be used uninitialized in this function > Reading the code this warning seems to be invalid except when RXBND is defined as zero, which would make the whole function totally bogus. Would be good to have a WARN_ON(!RXBND) in brcms_b_attach(). I don't like pleasing a compiler with unnecessary initializers but I am also for builds with zero warnings. So... Acked-by: Arend van Spriel > Signed-off-by: Geert Uytterhoeven > --- > drivers/net/wireless/brcm80211/brcmsmac/main.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/brcm80211/brcmsmac/main.c b/drivers/net/wireless/brcm80211/brcmsmac/main.c > index 17594de..5f0f444 100644 > --- a/drivers/net/wireless/brcm80211/brcmsmac/main.c > +++ b/drivers/net/wireless/brcm80211/brcmsmac/main.c > @@ -7633,7 +7633,7 @@ brcms_b_recv(struct brcms_hardware *wlc_hw, uint fifo, bool bound) > > uint n = 0; > uint bound_limit = bound ? RXBND : -1; > - bool morepending; > + bool morepending = false; > > skb_queue_head_init(&recv_frames); >