Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932911AbbFWNEu (ORCPT ); Tue, 23 Jun 2015 09:04:50 -0400 Received: from mail-wi0-f173.google.com ([209.85.212.173]:35340 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932689AbbFWNEd (ORCPT ); Tue, 23 Jun 2015 09:04:33 -0400 MIME-Version: 1.0 In-Reply-To: <20150623125224.GA991@goodgumbo.baconseed.org> References: <20150623125224.GA991@goodgumbo.baconseed.org> Date: Tue, 23 Jun 2015 15:04:32 +0200 Message-ID: Subject: Re: [PATCH] staging: rtl8192u: bool tests don't need comparisons From: Frans Klaver To: Luis de Bethencourt Cc: "linux-kernel@vger.kernel.org" , Greg Kroah-Hartman , Antoine Schweitzer-Chaput , Cristina Opriceana , Aya Mahfouz , Rickard Strandqvist , Koray Gulcu , Greg Donald , Lorenzo Stoakes , Ebru Akagunduz , Karthik Nayak , devel@driverdev.osuosl.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7220 Lines: 146 On Tue, Jun 23, 2015 at 2:52 PM, Luis de Bethencourt wrote: > Remove explicit true/false comparations to bool variables. > > Signed-off-by: Luis de Bethencourt > --- > drivers/staging/rtl8192u/r8192U_core.c | 7 ++++--- > drivers/staging/rtl8192u/r8192U_dm.c | 21 +++++++++++---------- > 2 files changed, 15 insertions(+), 13 deletions(-) > > diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c > index a4795af..c53d670 100644 > --- a/drivers/staging/rtl8192u/r8192U_core.c > +++ b/drivers/staging/rtl8192u/r8192U_core.c > @@ -2047,7 +2047,7 @@ static bool GetHalfNmodeSupportByAPs819xUsb(struct net_device *dev) > struct r8192_priv *priv = ieee80211_priv(dev); > struct ieee80211_device *ieee = priv->ieee80211; > > - if (ieee->bHalfWirelessN24GMode == true) > + if (ieee->bHalfWirelessN24GMode) > Reval = true; > else > Reval = false; > @@ -2762,7 +2762,7 @@ static bool rtl8192_adapter_start(struct net_device *dev) > // > #ifdef TO_DO_LIST > if (Adapter->ResetProgress == RESET_TYPE_NORESET) { > - if (pMgntInfo->RegRfOff == true) { /* User disable RF via registry. */ > + if (pMgntInfo->RegRfOff) { /* User disable RF via registry. */ > RT_TRACE((COMP_INIT|COMP_RF), DBG_LOUD, ("InitializeAdapter819xUsb(): Turn off RF for RegRfOff ----------\n")); > MgntActSet_RF_State(Adapter, eRfOff, RF_CHANGE_BY_SW); > // Those actions will be discard in MgntActSet_RF_State because of the same state > @@ -4406,7 +4406,8 @@ static void query_rxdesc_status(struct sk_buff *skb, > /* RTL8190 set this bit to indicate that Hw does not decrypt packet */ > stats->Decrypted = !desc->SWDec; > > - if ((priv->ieee80211->pHTInfo->bCurrentHTSupport == true) && (priv->ieee80211->pairwise_key_type == KEY_TYPE_CCMP)) > + if ((priv->ieee80211->pHTInfo->bCurrentHTSupport) && > + (priv->ieee80211->pairwise_key_type == KEY_TYPE_CCMP)) > stats->bHwError = false; > else > stats->bHwError = stats->bCRC|stats->bICV; > diff --git a/drivers/staging/rtl8192u/r8192U_dm.c b/drivers/staging/rtl8192u/r8192U_dm.c > index 12dd19e..9946615 100644 > --- a/drivers/staging/rtl8192u/r8192U_dm.c > +++ b/drivers/staging/rtl8192u/r8192U_dm.c > @@ -438,7 +438,7 @@ static void dm_bandwidth_autoswitch(struct net_device *dev) > > if (priv->CurrentChannelBW == HT_CHANNEL_WIDTH_20 || !priv->ieee80211->bandwidth_auto_switch.bautoswitch_enable) > return; > - if (priv->ieee80211->bandwidth_auto_switch.bforced_tx20Mhz == false) { /* If send packets in 40 Mhz in 20/40 */ > + if (!priv->ieee80211->bandwidth_auto_switch.bforced_tx20Mhz) { /* If send packets in 40 Mhz in 20/40 */ > if (priv->undecorated_smoothed_pwdb <= priv->ieee80211->bandwidth_auto_switch.threshold_40Mhzto20Mhz) > priv->ieee80211->bandwidth_auto_switch.bforced_tx20Mhz = true; > } else { /* in force send packets in 20 Mhz in 20/40 */ > @@ -563,7 +563,7 @@ static void dm_TXPowerTrackingCallback_TSSI(struct net_device *dev) > break; > } > } > - if (viviflag == true) { > + if (viviflag) { > write_nic_byte(dev, 0x1ba, 0); > viviflag = false; > RT_TRACE(COMP_POWER_TRACKING, "we filtered the data\n"); > @@ -766,7 +766,7 @@ void dm_txpower_trackingcallback(struct work_struct *work) > struct r8192_priv *priv = container_of(dwork, struct r8192_priv, txpower_tracking_wq); > struct net_device *dev = priv->ieee80211->dev; > > - if (priv->bDcut == true) > + if (priv->bDcut) > dm_TXPowerTrackingCallback_TSSI(dev); > else > dm_TXPowerTrackingCallback_ThermalMeter(dev); > @@ -1301,7 +1301,7 @@ void dm_initialize_txpower_tracking(struct net_device *dev) > { > struct r8192_priv *priv = ieee80211_priv(dev); > > - if (priv->bDcut == true) > + if (priv->bDcut) > dm_InitializeTXPowerTracking_TSSI(dev); > else > dm_InitializeTXPowerTracking_ThermalMeter(dev); > @@ -1357,7 +1357,7 @@ static void dm_check_txpower_tracking(struct net_device *dev) > #ifdef RTL8190P > dm_CheckTXPowerTracking_TSSI(dev); > #else > - if (priv->bDcut == true) > + if (priv->bDcut) > dm_CheckTXPowerTracking_TSSI(dev); > else > dm_CheckTXPowerTracking_ThermalMeter(dev); > @@ -1467,7 +1467,7 @@ void dm_cck_txpower_adjust(struct net_device *dev, bool binch14) > { /* dm_CCKTxPowerAdjust */ > struct r8192_priv *priv = ieee80211_priv(dev); > > - if (priv->bDcut == true) > + if (priv->bDcut) > dm_CCKTxPowerAdjust_TSSI(dev, binch14); > else > dm_CCKTxPowerAdjust_ThermalMeter(dev, binch14); > @@ -1731,7 +1731,7 @@ static void dm_dig_init(struct net_device *dev) > *---------------------------------------------------------------------------*/ > static void dm_ctrl_initgain_byrssi(struct net_device *dev) > { > - if (dm_digtable.dig_enable_flag == false) > + if (!dm_digtable.dig_enable_flag) > return; > > if (dm_digtable.dig_algorithm == DIG_ALGO_BY_FALSE_ALARM) > @@ -1750,7 +1750,7 @@ static void dm_ctrl_initgain_byrssi_by_driverrssi( > u8 i; > static u8 fw_dig; > > - if (dm_digtable.dig_enable_flag == false) > + if (!dm_digtable.dig_enable_flag) > return; > > /*DbgPrint("Dig by Sw Rssi\n");*/ > @@ -1792,7 +1792,7 @@ static void dm_ctrl_initgain_byrssi_by_fwfalse_alarm( > static u32 reset_cnt; > u8 i; > > - if (dm_digtable.dig_enable_flag == false) > + if (!dm_digtable.dig_enable_flag) > return; > > if (dm_digtable.dig_algorithm_switch) { > @@ -3062,7 +3062,8 @@ static void dm_dynamic_txpower(struct net_device *dev) > priv->bDynamicTxLowPower = false; > } else { > /* high power state check */ > - if (priv->undecorated_smoothed_pwdb < txlowpower_threshold && priv->bDynamicTxHighPower == true) > + if (priv->undecorated_smoothed_pwdb < > + txlowpower_threshold && priv->bDynamicTxHighPower) > priv->bDynamicTxHighPower = false; Oh, this has a misleading air hanging over it. It focuses the eyes on "txlowpower_threshold && priv->bDynamicTxHighPower", while that probably isn't the intent. Frans -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/