staging: wilc1000: fix line over 80 charas in wilc_wfi_remain_on_channel_expired()
authorAjay Singh <ajay.kathat@microchip.com>
Fri, 11 May 2018 08:13:16 +0000 (13:43 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 14 May 2018 12:04:48 +0000 (14:04 +0200)
Refactor wilc_wfi_remain_on_channel_expired() to avoid line over 80
character issue reported by checkpatch.pl script. Also assigned value in the
variable at the time of declaration.

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 32bdefb..93511de 100644 (file)
@@ -1487,18 +1487,16 @@ static void wilc_wfi_remain_on_channel_ready(void *priv_data)
 
 static void wilc_wfi_remain_on_channel_expired(void *data, u32 session_id)
 {
-       struct wilc_priv *priv;
+       struct wilc_priv *priv = data;
+       struct wilc_wfi_p2p_listen_params *params = &priv->remain_on_ch_params;
 
-       priv = data;
+       if (session_id != params->listen_session_id)
+               return;
 
-       if (session_id == priv->remain_on_ch_params.listen_session_id) {
-               priv->p2p_listen_state = false;
+       priv->p2p_listen_state = false;
 
-               cfg80211_remain_on_channel_expired(priv->wdev,
-                                                  priv->remain_on_ch_params.listen_cookie,
-                                                  priv->remain_on_ch_params.listen_ch,
-                                                  GFP_KERNEL);
-       }
+       cfg80211_remain_on_channel_expired(priv->wdev, params->listen_cookie,
+                                          params->listen_ch, GFP_KERNEL);
 }
 
 static int remain_on_channel(struct wiphy *wiphy,