Commit c6112e48 authored by Aya Mahfouz's avatar Aya Mahfouz Committed by Greg Kroah-Hartman
Browse files

staging: rtl8188eu: rewrite the right hand side of an assignment 



This patch rewrites the right hand side of an assignment for
expressions of the form:
a = (a <op> b);
to be:
a <op>= b;
where <op> = << | >>.

This issue was detected and resolved using the following
coccinelle script:

@@
identifier i;
expression e;
@@

-i = (i >> e);
+i >>= e;

@@
identifier i;
expression e;
@@

-i = (i << e);
+i <<= e;

Signed-off-by: default avatarAya Mahfouz <mahfouz.saif.elyazal@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 945f0185

drivers/staging/rtl8188eu/hal/rf_cfg.c

+2 −2
Original line number Diff line number Diff line
@@ -38,12 +38,12 @@ static bool check_condition(struct adapter *adapt, const u32 condition) 
		return false;



	cond = condition & 0x0000FF00;

	cond = cond >> 8;

	cond >>= 8;

	if ((_interface & cond) == 0 && cond != 0x07)

		return false;



	cond = condition & 0x00FF0000;

	cond = cond >> 16;

	cond >>= 16;

	if ((_platform & cond) == 0 && cond != 0x0F)

		return false;

	return true;