staging: wilc1000: fix line over 80 chars in wilc_wfi_cfg_tx_vendor_spec()
authorAjay Singh <ajay.kathat@microchip.com>
Fri, 11 May 2018 08:13:17 +0000 (13:43 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 14 May 2018 12:04:48 +0000 (14:04 +0200)
Fix line over 80 characters issues reported by checkpatch.pl script in
wilc_wfi_cfg_tx_vendor_spec() by using temporary variable. Simplified
'if else' condition with 'if'.

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/wilc_wfi_cfgoperations.c

index 93511dee0a880fd2f585003767efd9ffe85d102d..799473665af96b842ec8ff1c0c71c0790764d85c 100644 (file)
@@ -1570,14 +1570,14 @@ static void wilc_wfi_cfg_tx_vendor_spec(struct p2p_mgmt_data *mgmt_tx,
        for (i = P2P_PUB_ACTION_SUBTYPE + 2; i < len; i++) {
                if (buf[i] == P2PELEM_ATTR_ID &&
                    !memcmp(p2p_oui, &buf[i + 2], 4)) {
+                       bool oper_ch = false;
+                       u8 *tx_buff = &mgmt_tx->buff[i + 6];
+
                        if (subtype == P2P_INV_REQ || subtype == P2P_INV_RSP)
-                               wilc_wfi_cfg_parse_tx_action(&mgmt_tx->buff[i + 6],
-                                                            len - (i + 6),
-                                                            true, iftype);
-                       else
-                               wilc_wfi_cfg_parse_tx_action(&mgmt_tx->buff[i + 6],
-                                                            len - (i + 6),
-                                                            false, iftype);
+                               oper_ch = true;
+
+                       wilc_wfi_cfg_parse_tx_action(tx_buff, len - (i + 6),
+                                                    oper_ch, iftype);
 
                        break;
                }