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.kat...@microchip.com>
---
 drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c 
b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
index 4f35178..8dea414 100644
--- a/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
+++ b/drivers/staging/wilc1000/wilc_wfi_cfgoperations.c
@@ -1573,14 +1573,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;
                }
-- 
2.7.4

Reply via email to