On Fri, 14 Aug 2020, Kalle Valo wrote: > Lee Jones <lee.jo...@linaro.org> writes: > > > On Fri, 14 Aug 2020, Kalle Valo wrote: > > > >> Lee Jones <lee.jo...@linaro.org> 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 <kv...@codeaurora.org> > >> > Cc: "David S. Miller" <da...@davemloft.net> > >> > Cc: Jakub Kicinski <k...@kernel.org> > >> > Cc: Martin Langer <martin-lan...@gmx.de> > >> > Cc: Stefano Brivio <stefano.bri...@polimi.it> > >> > Cc: Michael Buesch <m...@bues.ch> > >> > Cc: van Dyk <kugelf...@gentoo.org> > >> > Cc: Andreas Jaggi <andreas.ja...@waterwave.ch> > >> > Cc: Albert Herranz <albert_herr...@yahoo.es> > >> > Cc: linux-wirel...@vger.kernel.org > >> > Cc: b43-...@lists.infradead.org > >> > Cc: netdev@vger.kernel.org > >> > Signed-off-by: Lee Jones <lee.jo...@linaro.org> > >> > --- > >> > 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.
So I agree that: "net: wireless: broadcom: b43: main" ... seems unnecessarily long and verbose. However, IMHO: "b43:" ... is too short and not forthcoming enough. Obviously this fine when something like `git log -- net/wireless`, as you already know what the patch pertains to, however when someone who is not in the know (like I would be) does `git log` and sees a "b43:" patch, they would have no idea which subsystem this patch is adapting. Even: "wireless: b43:" ... would be worlds better. A Git log which omitted all subsystem tags would be of limited use. > >> 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. I see. Thanks for the clarification. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
