staging: rtl8712: Remove the unnecessary parantheses
authorG Pooja Shamili <poojashamili@gmail.com>
Sun, 6 Mar 2016 09:42:30 +0000 (15:12 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 12 Mar 2016 06:09:09 +0000 (22:09 -0800)
The unnecessary parantheses on the right side of assignments were removed,
as in most cases (expect for ==, >=, <=, !=), they are futile.

This was done using Coccinelle, the semantic patch being:

@@
expression E1,E2,E3;
binary operator bin_op = {==,>=,<=,!=};
@@

E1 =
(
  (
   E2 bin_op E3
  )
|
-(
E2
-)
)
;

Signed-off-by: G Pooja Shamili <poojashamili@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8712/rtl871x_mp.c

index 9b5a4e7..5e4fda1 100644 (file)
@@ -235,7 +235,7 @@ static u8 set_bb_reg(struct _adapter *pAdapter,
        if (bitmask != bMaskDWord) {
                org_value = r8712_bb_reg_read(pAdapter, offset);
                bit_shift = bitshift(bitmask);
-               new_value = ((org_value & (~bitmask)) | (value << bit_shift));
+               new_value = (org_value & (~bitmask)) | (value << bit_shift);
        } else {
                new_value = value;
        }
@@ -260,7 +260,7 @@ static u8 set_rf_reg(struct _adapter *pAdapter, u8 path, u8 offset, u32 bitmask,
        if (bitmask != bMaskDWord) {
                org_value = r8712_rf_reg_read(pAdapter, path, offset);
                bit_shift = bitshift(bitmask);
-               new_value = ((org_value & (~bitmask)) | (value << bit_shift));
+               new_value = (org_value & (~bitmask)) | (value << bit_shift);
        } else {
                new_value = value;
        }
@@ -327,10 +327,10 @@ void r8712_SetTxAGCOffset(struct _adapter *pAdapter, u32 ulTxAGCOffset)
 {
        u32 TxAGCOffset_B, TxAGCOffset_C, TxAGCOffset_D, tmpAGC;
 
-       TxAGCOffset_B = (ulTxAGCOffset & 0x000000ff);
+       TxAGCOffset_B = ulTxAGCOffset & 0x000000ff;
        TxAGCOffset_C = (ulTxAGCOffset & 0x0000ff00) >> 8;
        TxAGCOffset_D = (ulTxAGCOffset & 0x00ff0000) >> 16;
-       tmpAGC = (TxAGCOffset_D << 8 | TxAGCOffset_C << 4 | TxAGCOffset_B);
+       tmpAGC = TxAGCOffset_D << 8 | TxAGCOffset_C << 4 | TxAGCOffset_B;
        set_bb_reg(pAdapter, rFPGA0_TxGainStage,
                        (bXBTxAGC | bXCTxAGC | bXDTxAGC), tmpAGC);
 }