diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-15 06:23:32 -0500 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2011-12-15 14:46:38 -0500 |
| commit | aa1f2f0a3218a9b6ce979fca3d6ebdb1c5544dd8 (patch) | |
| tree | 59ab8f7ad97a7607722c109d0084e5a9188b7202 /drivers | |
| parent | 29942bc125374b5aa049a438fb628ea729538ca5 (diff) | |
brcm80211: smac: precendence bug in wlc_phy_attach()
Negate has higher precendence than compare and since neither zero nor
one are equal to four or eight the original condition is always false.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.c b/drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.c index 5b57caa7c3af..264f8c4c703d 100644 --- a/drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.c +++ b/drivers/net/wireless/brcm80211/brcmsmac/phy/phy_cmn.c | |||
| @@ -482,8 +482,8 @@ wlc_phy_attach(struct shared_phy *sh, struct bcma_device *d11core, | |||
| 482 | pi->pubpi.phy_corenum = PHY_CORE_NUM_2; | 482 | pi->pubpi.phy_corenum = PHY_CORE_NUM_2; |
| 483 | pi->pubpi.ana_rev = (phyversion & PV_AV_MASK) >> PV_AV_SHIFT; | 483 | pi->pubpi.ana_rev = (phyversion & PV_AV_MASK) >> PV_AV_SHIFT; |
| 484 | 484 | ||
| 485 | if (!pi->pubpi.phy_type == PHY_TYPE_N && | 485 | if (pi->pubpi.phy_type != PHY_TYPE_N && |
| 486 | !pi->pubpi.phy_type == PHY_TYPE_LCN) | 486 | pi->pubpi.phy_type != PHY_TYPE_LCN) |
| 487 | goto err; | 487 | goto err; |
| 488 | 488 | ||
| 489 | if (bandtype == BRCM_BAND_5G) { | 489 | if (bandtype == BRCM_BAND_5G) { |