From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Date: Fri, 14 Aug 2020 20:25:31 +0300 Subject: [PATCH 07/30] net: wireless: broadcom: b43: main: Add braces around empty statements In-Reply-To: <20200814164322.GP4354@dell> (Lee Jones's message of "Fri, 14 Aug 2020 17:43:22 +0100") References: <20200814113933.1903438-1-lee.jones@linaro.org> <20200814113933.1903438-8-lee.jones@linaro.org> <87v9hll0ro.fsf@codeaurora.org> <20200814164322.GP4354@dell> Message-ID: <87eeo9kulw.fsf@codeaurora.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Lee Jones Cc: davem@davemloft.net, kuba@kernel.org, linux-kernel@vger.kernel.org, Martin Langer , Stefano Brivio , Michael Buesch , van Dyk , Andreas Jaggi , Albert Herranz , linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org, netdev@vger.kernel.org Lee Jones writes: > On Fri, 14 Aug 2020, Kalle Valo wrote: > >> Lee Jones writes: >> >> > Fixes the following W=1 kernel build warning(s): >> > >> > drivers/net/wireless/broadcom/b43/main.c: In function ?b43_dummy_transmission?: >> > drivers/net/wireless/broadcom/b43/main.c:785:3: warning: suggest >> > braces around empty body in an ?if? statement [-Wempty-body] >> > drivers/net/wireless/broadcom/b43/main.c: In function ?b43_do_interrupt_thread?: >> > drivers/net/wireless/broadcom/b43/main.c:2017:3: warning: suggest >> > braces around empty body in an ?if? statement [-Wempty-body] >> > >> > Cc: Kalle Valo >> > Cc: "David S. Miller" >> > Cc: Jakub Kicinski >> > Cc: Martin Langer >> > Cc: Stefano Brivio >> > Cc: Michael Buesch >> > Cc: van Dyk >> > Cc: Andreas Jaggi >> > Cc: Albert Herranz >> > Cc: linux-wireless at vger.kernel.org >> > Cc: b43-dev at lists.infradead.org >> > Cc: netdev at vger.kernel.org >> > Signed-off-by: Lee Jones >> > --- >> > drivers/net/wireless/broadcom/b43/main.c | 6 ++++-- >> > 1 file changed, 4 insertions(+), 2 deletions(-) >> >> Please don't copy the full directory structure to the title. I'll change >> the title to more simple version: >> >> b43: add braces around empty statements > > This seems to go the other way. > > "net: wireless: b43" seems sensible. Sorry, not understanding what you mean here. >> I'll do similar changes to other wireless-drivers patches. > > Thanks. > > Does that mean it's been applied, or is this future tense? It's not applied yet, there will be an automatic "applied" email once I have done that. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches