Commit 760e7353 authored by Nathan Chancellor's avatar Nathan Chancellor Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: Remove self assignment in get_rx_power_val_by_reg() 



Clang warns:

drivers/staging/r8188eu/hal/rtl8188e_rf6052.c:344:13: warning:
explicitly assigning value of variable of type 'u32' (aka 'unsigned
int') to itself [-Wself-assign]
                        writeVal = writeVal;
                        ~~~~~~~~ ^ ~~~~~~~~
1 warning generated.

Remove this branch as it is useless.

Reviewed-by: default avatarNick Desaulniers <ndesaulniers@google.com>
Signed-off-by: default avatarNathan Chancellor <nathan@kernel.org>
Link: https://lore.kernel.org/r/20210805185807.3296077-3-nathan@kernel.org


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 717d933d

drivers/staging/r8188eu/hal/rtl8188e_rf6052.c

+0 −2
Original line number Diff line number Diff line
@@ -340,8 +340,6 @@ static void get_rx_power_val_by_reg(struct adapter *Adapter, u8 Channel, 
		/*  This mechanism is only applied when Driver-Highpower-Mechanism is OFF. */

		if (pdmpriv->DynamicTxHighPowerLvl == TxHighPwrLevel_BT1)

			writeVal = writeVal - 0x06060606;

		else if (pdmpriv->DynamicTxHighPowerLvl == TxHighPwrLevel_BT2)

			writeVal = writeVal;

		*(pOutWriteVal+rf) = writeVal;

	}

}