diff options
| author | Payal Kshirsagar <payal.s.kshirsagar.98@gmail.com> | 2019-03-30 18:31:11 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-03-30 19:58:20 +0100 |
| commit | 1bba5b2110b2bcef2705cb0d975182c422654383 (patch) | |
| tree | 3c98456b47dad00d98f9be7b5e31dd3afcc79962 /drivers | |
| parent | 5b07a3905fc073519707fe6a1a409d265dcd59d1 (diff) | |
staging: rtl8723bs: Compress two lines into one line
Challenge suggested by coccinelle.
Return value directly without saving it in a variable and remove that
variable.
Signed-off-by: Payal Kshirsagar <payal.s.kshirsagar.98@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c b/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c index 0fd7a9ae40cb..4f2ad54af398 100644 --- a/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c +++ b/drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c @@ -58,7 +58,7 @@ static u32 phy_CalculateBitShift(u32 BitMask) */ u32 PHY_QueryBBReg_8723B(struct adapter *Adapter, u32 RegAddr, u32 BitMask) { - u32 ReturnValue = 0, OriginalValue, BitShift; + u32 OriginalValue, BitShift; #if (DISABLE_BB_RF == 1) return 0; @@ -68,9 +68,8 @@ u32 PHY_QueryBBReg_8723B(struct adapter *Adapter, u32 RegAddr, u32 BitMask) OriginalValue = rtw_read32(Adapter, RegAddr); BitShift = phy_CalculateBitShift(BitMask); - ReturnValue = (OriginalValue & BitMask) >> BitShift; - return ReturnValue; + return (OriginalValue & BitMask) >> BitShift; } @@ -284,18 +283,16 @@ u32 PHY_QueryRFReg_8723B( u32 BitMask ) { - u32 Original_Value, Readback_Value, BitShift; + u32 Original_Value, BitShift; #if (DISABLE_BB_RF == 1) return 0; #endif Original_Value = phy_RFSerialRead_8723B(Adapter, eRFPath, RegAddr); - BitShift = phy_CalculateBitShift(BitMask); - Readback_Value = (Original_Value & BitMask) >> BitShift; - return Readback_Value; + return (Original_Value & BitMask) >> BitShift; } /** |