diff mbox series

[34/41] rtl8821ae: phy: Place braces around empty if() body

Message ID	20201102112410.1049272-35-lee.jones@linaro.org (mailing list archive)
State	Accepted
Commit	398d816a64eb3614c7a7b58291292ce2c756b6a1
Delegated to:	Kalle Valo
Headers	show Return-Path: <SRS0=EpEU=EI=vger.kernel.org=linux-wireless-owner@kernel.org> From: Lee Jones <lee.jones@linaro.org> To: kvalo@codeaurora.org Cc: linux-kernel@vger.kernel.org, Lee Jones <lee.jones@linaro.org>, Ping-Ke Shih <pkshih@realtek.com>, "David S. Miller" <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org>, Larry Finger <Larry.Finger@lwfinger.net>, linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH 34/41] rtl8821ae: phy: Place braces around empty if() body Date: Mon, 2 Nov 2020 11:24:03 +0000 Message-Id: <20201102112410.1049272-35-lee.jones@linaro.org> In-Reply-To: <20201102112410.1049272-1-lee.jones@linaro.org> References: <20201102112410.1049272-1-lee.jones@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	[01/41] wil6210: wmi: Correct misnamed function parameter 'ptr_' \| expand [01/41] wil6210: wmi: Correct misnamed function parameter 'ptr_' [02/41] rsi_91x_usb: Fix some basic kernel-doc issues [03/41] rtl8192cu: mac: Fix some missing/ill-documented function parameters [04/41] rsi_91x_usb_ops: Source file headers are not good candidates for kernel-doc [05/41] rtl8192cu: trx: Demote clear abuse of kernel-doc format [06/41] brcmfmac: bcmsdh: Fix description for function parameter 'pktlist' [07/41] brcmfmac: pcie: Provide description for missing function parameter 'devinfo' [08/41] iwlwifi: mvm: rs: Demote non-conformant function documentation headers [09/41] wl1251: cmd: Rename 'len' to 'buf_len' in the documentation [10/41] ath9k: ar9330_1p1_initvals: Remove unused const variable 'ar9331_common_tx_gain_offset1_1' [11/41] ath9k: ar9340_initvals: Remove unused const variable 'ar9340Modes_ub124_tx_gain_table_1p0' [12/41] ath9k: ar9485_initvals: Remove unused const variable 'ar9485_fast_clock_1_1_baseband_postam… [13/41] ath9k: ar9003_2p2_initvals: Remove unused const variables [14/41] prism54: isl_ioctl: Fix one function header and demote another [15/41] iwlwifi: iwl-eeprom-read: Demote one nonconformant function header [16/41] iwlwifi: iwl-eeprom-parse: Fix 'struct iwl_eeprom_enhanced_txpwr's header [17/41] intersil: hostap_hw: Prevent incompatible function type cast [18/41] brcm80211: fweh: Add missing description for 'gfp' [19/41] ath: regd: Provide description for ath_reg_apply_ir_flags's 'reg' param [20/41] ath9k: ar5008_phy: Demote half completed function headers [21/41] ath: dfs_pattern_detector: Fix some function kernel-doc headers [22/41] ath: dfs_pri_detector: Demote zero/half completed kernel-doc headers [23/41] iwlwifi: iwl-phy-db: Add missing struct member description for 'trans' [24/41] wl3501_cs: Fix misspelling and provide missing documentation [25/41] rtlwifi: halbtc8723b2ant: Remove a bunch of set but unused variables [26/41] rtlwifi: phy: Remove set but unused variable 'bbvalue' [27/41] rtlwifi: halbtc8821a1ant: Remove set but unused variable 'wifi_rssi_state' [28/41] rtl8723be: phy: Remove set but unused variable 'lc_cal' [29/41] rtl8188ee: phy: Remove set but unused variable 'reg_ea4' [30/41] rtlwifi: halbtc8821a2ant: Remove a bunch of unused variables [31/41] mwifiex: pcie: Remove a couple of unchecked 'ret's [32/41] rtl8723be: trx: Remove set but unused variable 'cck_highpwr' [33/41] rtlwifi: rtl8821ae: phy: Remove a couple of unused variables [34/41] rtl8821ae: phy: Place braces around empty if() body [35/41] wlcore: spi: Demote a non-compliant function header, fix another [36/41] rtw8822b: Return type is not const [37/41] iwlwifi: fw: dbg: Fix misspelling of 'reg_data' in function header [38/41] realtek: rtw8822c: Remove unused variable 'corr_val' [39/41] ath9k: dynack: Demote non-compliant function header [40/41] iwlwifi: fw: acpi: Demote non-conformant function headers [41/41] realtek: rtw88: pci: Add prototypes for .probe, .remove and .shutdown

Commit Message

Lee Jones Nov. 2, 2020, 11:24 a.m. UTC

Fixes the following W=1 kernel build warning(s):

 drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c: In function ‘_rtl8812ae_phy_get_txpower_limit’:
 drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2453:3: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body]

Cc: Ping-Ke Shih <pkshih@realtek.com>
Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Larry Finger <Larry.Finger@lwfinger.net>
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c
index 72ee0700a5497..8a1a2277e137b 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c
@@ -2448,8 +2448,9 @@  static s8 _rtl8812ae_phy_get_txpower_limit(struct ieee80211_hw *hw,
 	else if (band == BAND_ON_5G)
 		channel_temp = _rtl8812ae_phy_get_chnl_idx_of_txpwr_lmt(hw,
 		BAND_ON_5G, channel);
-	else if (band == BAND_ON_BOTH)
+	else if (band == BAND_ON_BOTH) {
 		;/* BAND_ON_BOTH don't care temporarily */
+	}
 
 	if (band_temp == -1 || regulation == -1 || bandwidth_temp == -1 ||
 		rate_section == -1 || channel_temp == -1) {