From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: [PATCH] Add a missing break in parse_station_flags Date: Mon, 23 Jul 2018 12:30:41 +0300 Message-ID: <87r2juti2m.fsf@purkki.adurom.net> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Johannes Berg , "David S. Miller" , "linux-wireless\@vger.kernel.org" , "netdev\@vger.kernel.org" , "linux-kernel\@vger.kernel.org" To: Bernd Edlinger Return-path: In-Reply-To: (Bernd Edlinger's message of "Sun, 8 Jul 2018 09:57:22 +0000") Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Bernd Edlinger writes: > I was looking at usually suppressed gcc warnings, > [-Wimplicit-fallthrough=] in this case: > > The code definitely looks like a break is missing here. > However I am not able to test the NL80211_IFTYPE_MESH_POINT, > nor do I actually know what might be :) > So please use this patch with caution and only if you are > able to do some testing. > > Signed-off-by: Bernd Edlinger > --- > net/wireless/nl80211.c | 1 + > 1 file changed, 1 insertion(+) You should use prefix "nl80211: " in the title. https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#subject -- Kalle Valo