2018-07-25 20:09:47

by Varsha Rao

Subject: [PATCH] net: wireless: brcmsmac: Remove extra parentheses

Remove the unnecessary parentheses to fix the clang warning of
extraneous parentheses.

Signed-off-by: Varsha Rao <[email protected]>
---
drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
index 1a187557982e..4deba3075083 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/phy/phy_n.c
@@ -25453,12 +25453,12 @@ void wlc_phy_cal_perical_nphy_run(struct brcms_phy *pi, u8 caltype)
(pi->cal_type_override ==
PHY_PERICAL_FULL) ? true : false;

- if ((pi->mphase_cal_phase_id > MPHASE_CAL_STATE_INIT)) {
+ if (pi->mphase_cal_phase_id > MPHASE_CAL_STATE_INIT) {
if (pi->nphy_txiqlocal_chanspec != pi->radio_chanspec)
wlc_phy_cal_perical_mphase_restart(pi);
}

- if ((pi->mphase_cal_phase_id == MPHASE_CAL_STATE_RXCAL))
+ if (pi->mphase_cal_phase_id == MPHASE_CAL_STATE_RXCAL)
wlapi_bmac_write_shm(pi->sh->physhim, M_CTS_DURATION, 10000);

wlapi_suspend_mac_and_wait(pi->sh->physhim);
--
2.17.0

2018-08-31 19:49:57

by Kalle Valo

Subject: Re: [PATCH] brcmsmac: Remove extra parentheses

Varsha Rao <[email protected]> wrote:

> Remove the unnecessary parentheses to fix the clang warning of
> extraneous parentheses.
>
> Signed-off-by: Varsha Rao <[email protected]>

Patch applied to wireless-drivers-next.git, thanks.

9e9e11d83238 brcmsmac: Remove extra parentheses

--
https://patchwork.kernel.org/patch/10544573/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches