staging: vt6656: Remove unnecessary parentheses.
authorElise Lennion <elise.lennion@gmail.com>
Tue, 18 Oct 2016 00:01:26 +0000 (22:01 -0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 24 Oct 2016 13:35:03 +0000 (15:35 +0200)
The removed parentheses are unnecessary and don't add readability.

Found using Coccinelle semantic patch:
@@ expression e, e1, e2; @@
e +=
(
        (e1 == e2)
|
-       (e1)
+       e1
)

@@ expression e, e1, e2; @@
e =
(
        (e1 == e2)
|
-       (e1)
+       e1
)

Signed-off-by: Elise Lennion <elise.lennion@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6656/mac.c
drivers/staging/vt6656/rf.c

index eeed16e9124ed9fe0ca608bdfe90066d3011b8c8..611da4929ddc18efbd596445f1f32b6fe51aee58 100644 (file)
@@ -121,7 +121,7 @@ void vnt_mac_set_keyentry(struct vnt_private *priv, u16 key_ctl, u32 entry_idx,
        u16 offset;
 
        offset = MISCFIFO_KEYETRY0;
-       offset += (entry_idx * MISCFIFO_KEYENTRYSIZE);
+       offset += entry_idx * MISCFIFO_KEYENTRYSIZE;
 
        set_key.u.write.key_ctl = cpu_to_le16(key_ctl);
        ether_addr_copy(set_key.u.write.addr, addr);
index 79a3108719a678c225cc519106b11fa909773318..8b724797734bb6d5fa83174c098dad49a1adece8 100644 (file)
@@ -800,8 +800,8 @@ int vnt_rf_set_txpower(struct vnt_private *priv, u8 power, u32 rate)
 /* Convert rssi to dbm */
 void vnt_rf_rssi_to_dbm(struct vnt_private *priv, u8 rssi, long *dbm)
 {
-       u8 idx = (((rssi & 0xc0) >> 6) & 0x03);
-       long b = (rssi & 0x3f);
+       u8 idx = ((rssi & 0xc0) >> 6) & 0x03;
+       long b = rssi & 0x3f;
        long a = 0;
        u8 airoharf[4] = {0, 18, 0, 40};