Return-path: Received: from mail-oi0-f53.google.com ([209.85.218.53]:35685 "EHLO mail-oi0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751408AbbHOQry (ORCPT ); Sat, 15 Aug 2015 12:47:54 -0400 Subject: Re: [PATCH] rtlwifi: rtl8192ee: fix semicolon.cocci warnings To: kbuild test robot References: <201508151824.iawa6wat%fengguang.wu@intel.com> <20150815103636.GA36860@ivytown2> Cc: kbuild-all@01.org, "John W. Linville" , Chaoming Li , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org From: Larry Finger Message-ID: <55CF6D2D.5060103@lwfinger.net> (sfid-20150815_184843_605731_1CFE0A76) Date: Sat, 15 Aug 2015 11:47:41 -0500 MIME-Version: 1.0 In-Reply-To: <20150815103636.GA36860@ivytown2> Content-Type: text/plain; charset=windows-1252; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 08/15/2015 05:36 AM, kbuild test robot wrote: > drivers/net/wireless/rtlwifi/rtl8192ee/phy.c:856:2-3: Unneeded semicolon > drivers/net/wireless/rtlwifi/rtl8192ee/phy.c:492:3-4: Unneeded semicolon > drivers/net/wireless/rtlwifi/rtl8192ee/phy.c:452:3-4: Unneeded semicolon > > > Remove unneeded semicolon. > > Generated by: scripts/coccinelle/misc/semicolon.cocci > > CC: Larry Finger > Signed-off-by: Fengguang Wu Signed-off-by: Larry Finger Larry > --- > > phy.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > --- a/drivers/net/wireless/rtlwifi/rtl8192ee/phy.c > +++ b/drivers/net/wireless/rtlwifi/rtl8192ee/phy.c > @@ -449,7 +449,7 @@ static void _rtl92ee_phy_set_txpower_by_ > "Invalid RateSection %d in 2.4G,Rf %d,%dTx\n", > rate_section, path, txnum); > break; > - }; > + } > } else { > RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, > "Invalid Band %d\n", band); > @@ -489,7 +489,7 @@ static u8 _rtl92ee_phy_get_txpower_by_ra > "Invalid RateSection %d in 2.4G,Rf %d,%dTx\n", > rate_section, path, txnum); > break; > - }; > + } > } else { > RT_TRACE(rtlpriv, COMP_INIT, DBG_LOUD, > "Invalid Band %d()\n", band); > @@ -853,7 +853,7 @@ static u8 _rtl92ee_get_rate_section_inde > else if (regaddr >= 0xE20 && regaddr <= 0xE4C) > index = (u8)((regaddr - 0xE20) / 4); > break; > - }; > + } > return index; > } > >