Commit 4cad1589 authored Mar 15, 2017 by Gargi Sharma Committed by Greg Kroah-Hartman Mar 16, 2017

staging: rtl8192u: Remove multiple assignments



This patch removes multiple assignments by factorizing them.
This was done with Coccinelle for the if branch. For the else part
the change was done manually since Coccinelle only detects constants.
Braces were also added to the else part to remove the checkpatch
warning, "braces should be on all arms of if-else statements".

@@
identifier i1,i2;
constant c;
@@
- i1=i2=c;
+ i1=c;
+ i2=c;

Signed-off-by: Gargi Sharma <gs051095@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent adab24a4

drivers/staging/rtl8192u/r8192U_dm.c

+9 −4

Original line number	Diff line number	Diff line
		@@ -725,10 +725,15 @@ static void dm_TXPowerTrackingCallback_ThermalMeter(struct net_device *dev)
		} else {
		tmpval = (u8)tmpRegA - priv->ThermalMeter[0];

		if (tmpval >= 6) /* higher temperature */
		tmpOFDMindex = tmpCCK20Mindex = 0; /* max to +6dB */
		else
		tmpOFDMindex = tmpCCK20Mindex = 6 - tmpval;
		if (tmpval >= 6) {
		/* higher temperature */
		tmpOFDMindex = 0;
		tmpCCK20Mindex = 0;
		} else {
		/* max to +6dB */
		tmpOFDMindex = 6 - tmpval;
		tmpCCK20Mindex = 6 - tmpval;
		}
		tmpCCK40Mindex = 0;
		}
		/*DbgPrint("%ddb, tmpOFDMindex = %d, tmpCCK20Mindex = %d, tmpCCK40Mindex = %d",