Commit a36e4e4a authored by Lee Jones's avatar Lee Jones Committed by Kalle Valo

brcmsmac: phy_n: Remove a bunch of unused variables

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

 In file included from drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:16:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_spurwar_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:19036:6: warning: variable ‘tempval’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_tempsense_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:21983:28: warning: variable ‘RfctrlMiscReg6_save’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_rssi_compute_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:22986:6: warning: variable ‘phyRx0_l’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_runsamples_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:23101:6: warning: variable ‘lpf_bw_ctl_miscreg4’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:23100:50: warning: variable ‘lpf_bw_ctl_miscreg3’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_iqcal_gainparams_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:23406:6: warning: variable ‘idx’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_a2_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:24707:7: warning: variable ‘phy_a6’ set but not used [-Wunused-but-set-variable]
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function ‘wlc_phy_a3_nphy’:
 drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c:24999:7: warning: variable ‘phy_a11’ set but not used [-Wunused-but-set-variable]

Cc: Arend van Spriel <arend.vanspriel@broadcom.com>
Cc: Franky Lin <franky.lin@broadcom.com>
Cc: Hante Meuleman <hante.meuleman@broadcom.com>
Cc: Chi-Hsien Lin <chi-hsien.lin@cypress.com>
Cc: Wright Feng <wright.feng@cypress.com>
Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: zhong jiang <zhongjiang@huawei.com>
Cc: linux-wireless@vger.kernel.org
Cc: brcm80211-dev-list.pdl@broadcom.com
Cc: brcm80211-dev-list@cypress.com
Cc: netdev@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200826093401.1458456-29-lee.jones@linaro.org
parent 38c95e02
...@@ -19033,7 +19033,6 @@ static void wlc_phy_spurwar_nphy(struct brcms_phy *pi) ...@@ -19033,7 +19033,6 @@ static void wlc_phy_spurwar_nphy(struct brcms_phy *pi)
u32 nphy_adj_noise_var_buf[] = { 0x3ff, 0x3ff }; u32 nphy_adj_noise_var_buf[] = { 0x3ff, 0x3ff };
bool isAdjustNoiseVar = false; bool isAdjustNoiseVar = false;
uint numTonesAdjust = 0; uint numTonesAdjust = 0;
u32 tempval = 0;
if (NREV_GE(pi->pubpi.phy_rev, 3)) { if (NREV_GE(pi->pubpi.phy_rev, 3)) {
if (pi->phyhang_avoid) if (pi->phyhang_avoid)
...@@ -19139,9 +19138,6 @@ static void wlc_phy_spurwar_nphy(struct brcms_phy *pi) ...@@ -19139,9 +19138,6 @@ static void wlc_phy_spurwar_nphy(struct brcms_phy *pi)
numTonesAdjust, numTonesAdjust,
nphy_adj_tone_id_buf, nphy_adj_tone_id_buf,
nphy_adj_noise_var_buf); nphy_adj_noise_var_buf);
tempval = 0;
} else { } else {
wlc_phy_adjust_min_noisevar_nphy(pi, 0, NULL, wlc_phy_adjust_min_noisevar_nphy(pi, 0, NULL,
NULL); NULL);
...@@ -21980,7 +21976,7 @@ s16 wlc_phy_tempsense_nphy(struct brcms_phy *pi) ...@@ -21980,7 +21976,7 @@ s16 wlc_phy_tempsense_nphy(struct brcms_phy *pi)
u16 auxADC_rssi_ctrlL, auxADC_rssi_ctrlH; u16 auxADC_rssi_ctrlL, auxADC_rssi_ctrlH;
s32 auxADC_Vl; s32 auxADC_Vl;
u16 RfctrlOverride5_save, RfctrlOverride6_save; u16 RfctrlOverride5_save, RfctrlOverride6_save;
u16 RfctrlMiscReg5_save, RfctrlMiscReg6_save; u16 RfctrlMiscReg5_save;
u16 RSSIMultCoef0QPowerDet_save; u16 RSSIMultCoef0QPowerDet_save;
u16 tempsense_Rcal; u16 tempsense_Rcal;
...@@ -21995,7 +21991,7 @@ s16 wlc_phy_tempsense_nphy(struct brcms_phy *pi) ...@@ -21995,7 +21991,7 @@ s16 wlc_phy_tempsense_nphy(struct brcms_phy *pi)
RfctrlOverride5_save = read_phy_reg(pi, 0x346); RfctrlOverride5_save = read_phy_reg(pi, 0x346);
RfctrlOverride6_save = read_phy_reg(pi, 0x347); RfctrlOverride6_save = read_phy_reg(pi, 0x347);
RfctrlMiscReg5_save = read_phy_reg(pi, 0x344); RfctrlMiscReg5_save = read_phy_reg(pi, 0x344);
RfctrlMiscReg6_save = read_phy_reg(pi, 0x345); read_phy_reg(pi, 0x345); /* RfctrlMiscReg6_save */
wlc_phy_table_read_nphy(pi, NPHY_TBL_ID_AFECTRL, 1, 0x0A, 16, wlc_phy_table_read_nphy(pi, NPHY_TBL_ID_AFECTRL, 1, 0x0A, 16,
&auxADC_Vmid_save); &auxADC_Vmid_save);
...@@ -22983,7 +22979,7 @@ int ...@@ -22983,7 +22979,7 @@ int
wlc_phy_rssi_compute_nphy(struct brcms_phy *pi, struct d11rxhdr *rxh) wlc_phy_rssi_compute_nphy(struct brcms_phy *pi, struct d11rxhdr *rxh)
{ {
s16 rxpwr, rxpwr0, rxpwr1; s16 rxpwr, rxpwr0, rxpwr1;
s16 phyRx0_l, phyRx2_l; s16 phyRx2_l;
rxpwr = 0; rxpwr = 0;
rxpwr0 = rxh->PhyRxStatus_1 & PRXS1_nphy_PWR0_MASK; rxpwr0 = rxh->PhyRxStatus_1 & PRXS1_nphy_PWR0_MASK;
...@@ -22994,7 +22990,6 @@ wlc_phy_rssi_compute_nphy(struct brcms_phy *pi, struct d11rxhdr *rxh) ...@@ -22994,7 +22990,6 @@ wlc_phy_rssi_compute_nphy(struct brcms_phy *pi, struct d11rxhdr *rxh)
if (rxpwr1 > 127) if (rxpwr1 > 127)
rxpwr1 -= 256; rxpwr1 -= 256;
phyRx0_l = rxh->PhyRxStatus_0 & 0x00ff;
phyRx2_l = rxh->PhyRxStatus_2 & 0x00ff; phyRx2_l = rxh->PhyRxStatus_2 & 0x00ff;
if (phyRx2_l > 127) if (phyRx2_l > 127)
phyRx2_l -= 256; phyRx2_l -= 256;
...@@ -23097,8 +23092,7 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops, ...@@ -23097,8 +23092,7 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops,
u16 bb_mult; u16 bb_mult;
u8 phy_bw, sample_cmd; u8 phy_bw, sample_cmd;
u16 orig_RfseqCoreActv; u16 orig_RfseqCoreActv;
u16 lpf_bw_ctl_override3, lpf_bw_ctl_override4, lpf_bw_ctl_miscreg3, u16 lpf_bw_ctl_override3, lpf_bw_ctl_override4;
lpf_bw_ctl_miscreg4;
if (pi->phyhang_avoid) if (pi->phyhang_avoid)
wlc_phy_stay_in_carriersearch_nphy(pi, true); wlc_phy_stay_in_carriersearch_nphy(pi, true);
...@@ -23111,12 +23105,7 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops, ...@@ -23111,12 +23105,7 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops,
lpf_bw_ctl_override3 = read_phy_reg(pi, 0x342) & (0x1 << 7); lpf_bw_ctl_override3 = read_phy_reg(pi, 0x342) & (0x1 << 7);
lpf_bw_ctl_override4 = read_phy_reg(pi, 0x343) & (0x1 << 7); lpf_bw_ctl_override4 = read_phy_reg(pi, 0x343) & (0x1 << 7);
if (lpf_bw_ctl_override3 | lpf_bw_ctl_override4) { if (!(lpf_bw_ctl_override3 | lpf_bw_ctl_override4)) {
lpf_bw_ctl_miscreg3 = read_phy_reg(pi, 0x340) &
(0x7 << 8);
lpf_bw_ctl_miscreg4 = read_phy_reg(pi, 0x341) &
(0x7 << 8);
} else {
wlc_phy_rfctrl_override_nphy_rev7( wlc_phy_rfctrl_override_nphy_rev7(
pi, pi,
(0x1 << 7), (0x1 << 7),
...@@ -23126,12 +23115,9 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops, ...@@ -23126,12 +23115,9 @@ wlc_phy_runsamples_nphy(struct brcms_phy *pi, u16 num_samps, u16 loops,
NPHY_REV7_RFCTRLOVERRIDE_ID1); NPHY_REV7_RFCTRLOVERRIDE_ID1);
pi->nphy_sample_play_lpf_bw_ctl_ovr = true; pi->nphy_sample_play_lpf_bw_ctl_ovr = true;
lpf_bw_ctl_miscreg3 = read_phy_reg(pi, 0x340) &
(0x7 << 8);
lpf_bw_ctl_miscreg4 = read_phy_reg(pi, 0x341) &
(0x7 << 8);
} }
read_phy_reg(pi, 0x340); /* lpf_bw_ctl_miscreg3 */
read_phy_reg(pi, 0x341); /* lpf_bw_ctl_miscreg4 */
} }
if ((pi->nphy_bb_mult_save & BB_MULT_VALID_MASK) == 0) { if ((pi->nphy_bb_mult_save & BB_MULT_VALID_MASK) == 0) {
...@@ -23403,7 +23389,6 @@ wlc_phy_iqcal_gainparams_nphy(struct brcms_phy *pi, u16 core_no, ...@@ -23403,7 +23389,6 @@ wlc_phy_iqcal_gainparams_nphy(struct brcms_phy *pi, u16 core_no,
struct nphy_iqcal_params *params) struct nphy_iqcal_params *params)
{ {
u8 k; u8 k;
int idx;
u16 gain_index; u16 gain_index;
u8 band_idx = (CHSPEC_IS5G(pi->radio_chanspec) ? 1 : 0); u8 band_idx = (CHSPEC_IS5G(pi->radio_chanspec) ? 1 : 0);
...@@ -23436,14 +23421,11 @@ wlc_phy_iqcal_gainparams_nphy(struct brcms_phy *pi, u16 core_no, ...@@ -23436,14 +23421,11 @@ wlc_phy_iqcal_gainparams_nphy(struct brcms_phy *pi, u16 core_no,
(target_gain.pga[core_no] << 4) | (target_gain.pga[core_no] << 4) |
(target_gain.txgm[core_no] << 8)); (target_gain.txgm[core_no] << 8));
idx = -1;
for (k = 0; k < NPHY_IQCAL_NUMGAINS; k++) { for (k = 0; k < NPHY_IQCAL_NUMGAINS; k++) {
if (tbl_iqcal_gainparams_nphy[band_idx][k][0] == if (tbl_iqcal_gainparams_nphy[band_idx][k][0] ==
gain_index) { gain_index)
idx = k;
break; break;
} }
}
params->txgm = tbl_iqcal_gainparams_nphy[band_idx][k][1]; params->txgm = tbl_iqcal_gainparams_nphy[band_idx][k][1];
params->pga = tbl_iqcal_gainparams_nphy[band_idx][k][2]; params->pga = tbl_iqcal_gainparams_nphy[band_idx][k][2];
...@@ -24704,7 +24686,6 @@ wlc_phy_a2_nphy(struct brcms_phy *pi, struct nphy_ipa_txcalgains *txgains, ...@@ -24704,7 +24686,6 @@ wlc_phy_a2_nphy(struct brcms_phy *pi, struct nphy_ipa_txcalgains *txgains,
{ {
u16 phy_a1, phy_a2, phy_a3; u16 phy_a1, phy_a2, phy_a3;
u16 phy_a4, phy_a5; u16 phy_a4, phy_a5;
bool phy_a6;
u8 phy_a7, m[2]; u8 phy_a7, m[2];
u32 phy_a8 = 0; u32 phy_a8 = 0;
struct nphy_txgains phy_a9; struct nphy_txgains phy_a9;
...@@ -24714,9 +24695,6 @@ wlc_phy_a2_nphy(struct brcms_phy *pi, struct nphy_ipa_txcalgains *txgains, ...@@ -24714,9 +24695,6 @@ wlc_phy_a2_nphy(struct brcms_phy *pi, struct nphy_ipa_txcalgains *txgains,
phy_a7 = (core == PHY_CORE_0) ? 1 : 0; phy_a7 = (core == PHY_CORE_0) ? 1 : 0;
phy_a6 = ((cal_mode == CAL_GCTRL)
|| (cal_mode == CAL_SOFT)) ? true : false;
if (NREV_GE(pi->pubpi.phy_rev, 7)) { if (NREV_GE(pi->pubpi.phy_rev, 7)) {
phy_a9 = wlc_phy_get_tx_gain_nphy(pi); phy_a9 = wlc_phy_get_tx_gain_nphy(pi);
...@@ -24996,7 +24974,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core) ...@@ -24996,7 +24974,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core)
s32 phy_a7, phy_a8; s32 phy_a7, phy_a8;
u32 phy_a9; u32 phy_a9;
int phy_a10; int phy_a10;
bool phy_a11 = false;
int phy_a12; int phy_a12;
u8 phy_a13 = 0; u8 phy_a13 = 0;
u8 phy_a14; u8 phy_a14;
...@@ -25064,8 +25041,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core) ...@@ -25064,8 +25041,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core)
if (!phy_a6 && (phy_a3 != phy_a5)) { if (!phy_a6 && (phy_a3 != phy_a5)) {
if (!phy_a3) if (!phy_a3)
phy_a12 -= (u8) phy_a1; phy_a12 -= (u8) phy_a1;
phy_a11 = true;
break; break;
} }
...@@ -25079,8 +25054,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core) ...@@ -25079,8 +25054,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core)
phy_a12 = phy_a14; phy_a12 = phy_a14;
else else
phy_a12 = phy_a13; phy_a12 = phy_a13;
phy_a11 = true;
break; break;
} }
...@@ -25110,8 +25083,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core) ...@@ -25110,8 +25083,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core)
if (!phy_a6 && (phy_a3 != phy_a5)) { if (!phy_a6 && (phy_a3 != phy_a5)) {
if (!phy_a3) if (!phy_a3)
phy_a12 -= (u8) phy_a1; phy_a12 -= (u8) phy_a1;
phy_a11 = true;
break; break;
} }
...@@ -25125,8 +25096,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core) ...@@ -25125,8 +25096,6 @@ static u8 wlc_phy_a3_nphy(struct brcms_phy *pi, u8 start_gain, u8 core)
phy_a12 = 0; phy_a12 = 0;
else else
phy_a12 = 127; phy_a12 = 127;
phy_a11 = true;
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment