linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 2/2] staging: rtl8192e: fix line over 80 characters
@ 2018-06-09  6:07 thtlwlsmsgkfla
  2018-06-09  6:45 ` Joe Perches
  0 siblings, 1 reply; 3+ messages in thread
From: thtlwlsmsgkfla @ 2018-06-09  6:07 UTC (permalink / raw)
  To: gregkh
  Cc: davem, colin.king, johannes.berg, Linyu.Yuan, keescook, tvboxspy,
	devel, linux-kernel, kernel-janitors

*fix checkpatch.pl warnings:
 WARNING: line over 80 characters

Signed-off-by: Hyunil Kim <thtlwlsmsgkfla@gmail.com>
---
 drivers/staging/rtl8192e/rtllib_softmac.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
index d172e87..3c5def2 100644
--- a/drivers/staging/rtl8192e/rtllib_softmac.c
+++ b/drivers/staging/rtl8192e/rtllib_softmac.c
@@ -2596,7 +2596,8 @@ static void rtllib_start_ibss_wq(void *data)
 
 	if (ieee->current_network.ssid_len == 0) {
 		strcpy(ieee->current_network.ssid, RTLLIB_DEFAULT_TX_ESSID);
-		ieee->current_network.ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
+		ieee->current_network.ssid_len =
+			strlen(RTLLIB_DEFAULT_TX_ESSID);
 		ieee->ssid_set = 1;
 	}
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 2/2] staging: rtl8192e: fix line over 80 characters
  2018-06-09  6:07 [PATCH 2/2] staging: rtl8192e: fix line over 80 characters thtlwlsmsgkfla
@ 2018-06-09  6:45 ` Joe Perches
  2018-06-16  4:20   ` 김현일
  0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2018-06-09  6:45 UTC (permalink / raw)
  To: thtlwlsmsgkfla, gregkh
  Cc: davem, colin.king, johannes.berg, Linyu.Yuan, keescook, tvboxspy,
	devel, linux-kernel, kernel-janitors

On Sat, 2018-06-09 at 15:07 +0900, thtlwlsmsgkfla@gmail.com wrote:
> *fix checkpatch.pl warnings:
>  WARNING: line over 80 characters

Consider using a temporary for these ieee->current_network.
dereferences.

> diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
[]
> @@ -2596,7 +2596,8 @@ static void rtllib_start_ibss_wq(void *data)
>  
>  	if (ieee->current_network.ssid_len == 0) {
>  		strcpy(ieee->current_network.ssid, RTLLIB_DEFAULT_TX_ESSID);
> -		ieee->current_network.ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
> +		ieee->current_network.ssid_len =
> +			strlen(RTLLIB_DEFAULT_TX_ESSID);
>  		ieee->ssid_set = 1;
>  	}

For instance:

Adding a temporary for all ieee->current_network
deferences actually reduces line count in this file.

---
 drivers/staging/rtl8192e/rtllib_softmac.c | 281 +++++++++++++++---------------
 1 file changed, 139 insertions(+), 142 deletions(-)

diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
index 919231fec09c..5733a1ab59d1 100644
--- a/drivers/staging/rtl8192e/rtllib_softmac.c
+++ b/drivers/staging/rtl8192e/rtllib_softmac.c
@@ -157,10 +157,10 @@ MgntQuery_TxRateExcludeCCKRates(struct rtllib_device *ieee)
 	u16	i;
 	u8	QueryRate = 0;
 	u8	BasicRate;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
-
-	for (i = 0; i < ieee->current_network.rates_len; i++) {
-		BasicRate = ieee->current_network.rates[i]&0x7F;
+	for (i = 0; i < cur_net->rates_len; i++) {
+		BasicRate = cur_net->rates[i] & 0x7F;
 		if (!rtllib_is_cck_rate(BasicRate)) {
 			if (QueryRate == 0) {
 				QueryRate = BasicRate;
@@ -338,8 +338,9 @@ static inline struct sk_buff *rtllib_probe_req(struct rtllib_device *ieee)
 	u8 *tag;
 	struct sk_buff *skb;
 	struct rtllib_probe_request *req;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
-	len = ieee->current_network.ssid_len;
+	len = cur_net->ssid_len;
 
 	rate_len = rtllib_MFIE_rate_len(ieee);
 
@@ -363,7 +364,7 @@ static inline struct sk_buff *rtllib_probe_req(struct rtllib_device *ieee)
 
 	*tag++ = MFIE_TYPE_SSID;
 	*tag++ = len;
-	memcpy(tag, ieee->current_network.ssid, len);
+	memcpy(tag, cur_net->ssid, len);
 	tag += len;
 
 	rtllib_MFIE_Brate(ieee, &tag);
@@ -377,6 +378,7 @@ static struct sk_buff *rtllib_get_beacon_(struct rtllib_device *ieee);
 static void rtllib_send_beacon(struct rtllib_device *ieee)
 {
 	struct sk_buff *skb;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	if (!ieee->ieee_up)
 		return;
@@ -389,7 +391,7 @@ static void rtllib_send_beacon(struct rtllib_device *ieee)
 
 	if (ieee->beacon_txing && ieee->ieee_up)
 		mod_timer(&ieee->beacon_timer, jiffies +
-			  (msecs_to_jiffies(ieee->current_network.beacon_interval - 5)));
+			  (msecs_to_jiffies(cur_net->beacon_interval - 5)));
 }
 
 
@@ -578,7 +580,8 @@ static void rtllib_softmac_scan_wq(void *data)
 {
 	struct rtllib_device *ieee = container_of_dwork_rsl(data,
 				     struct rtllib_device, softmac_scan_wq);
-	u8 last_channel = ieee->current_network.channel;
+	struct rtllib_network *cur_net = &ieee->current_network;
+	u8 last_channel = cur_net->channel;
 
 	rtllib_update_active_chan_map(ieee);
 
@@ -597,22 +600,20 @@ static void rtllib_softmac_scan_wq(void *data)
 	}
 
 	do {
-		ieee->current_network.channel =
-			(ieee->current_network.channel + 1) %
-			MAX_CHANNEL_NUMBER;
+		cur_net->channel = (cur_net->channel + 1) % MAX_CHANNEL_NUMBER;
 		if (ieee->scan_watch_dog++ > MAX_CHANNEL_NUMBER) {
-			if (!ieee->active_channel_map[ieee->current_network.channel])
-				ieee->current_network.channel = 6;
+			if (!ieee->active_channel_map[cur_net->channel])
+				cur_net->channel = 6;
 			goto out; /* no good chans */
 		}
-	} while (!ieee->active_channel_map[ieee->current_network.channel]);
+	} while (!ieee->active_channel_map[cur_net->channel]);
 
 	if (ieee->scanning_continue == 0)
 		goto out;
 
-	ieee->set_chan(ieee->dev, ieee->current_network.channel);
+	ieee->set_chan(ieee->dev, cur_net->channel);
 
-	if (ieee->active_channel_map[ieee->current_network.channel] == 1)
+	if (ieee->active_channel_map[cur_net->channel] == 1)
 		rtllib_send_probe_requests(ieee, 0);
 
 	schedule_delayed_work(&ieee->softmac_scan_wq,
@@ -624,7 +625,7 @@ static void rtllib_softmac_scan_wq(void *data)
 out:
 	if (IS_DOT11D_ENABLE(ieee))
 		DOT11D_ScanComplete(ieee);
-	ieee->current_network.channel = last_channel;
+	cur_net->channel = last_channel;
 
 out1:
 	ieee->actscanning = false;
@@ -822,11 +823,11 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
 	int encrypt;
 	int atim_len, erp_len;
 	struct lib80211_crypt_data *crypt;
-
-	char *ssid = ieee->current_network.ssid;
-	int ssid_len = ieee->current_network.ssid_len;
-	int rate_len = ieee->current_network.rates_len+2;
-	int rate_ex_len = ieee->current_network.rates_ex_len;
+	struct rtllib_network *cur_net = &ieee->current_network;
+	char *ssid = cur_net->ssid;
+	int ssid_len = cur_net->ssid_len;
+	int rate_len = cur_net->rates_len + 2;
+	int rate_ex_len = cur_net->rates_ex_len;
 	int wpa_ie_len = ieee->wpa_ie_len;
 	u8 erpinfo_content = 0;
 
@@ -841,17 +842,17 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
 	if (rate_ex_len > 0)
 		rate_ex_len += 2;
 
-	if (ieee->current_network.capability & WLAN_CAPABILITY_IBSS)
+	if (cur_net->capability & WLAN_CAPABILITY_IBSS)
 		atim_len = 4;
 	else
 		atim_len = 0;
 
-	if ((ieee->current_network.mode == IEEE_G) ||
-	   (ieee->current_network.mode == IEEE_N_24G &&
+	if ((cur_net->mode == IEEE_G) ||
+	   (cur_net->mode == IEEE_N_24G &&
 	   ieee->pHTInfo->bCurSuppCCK)) {
 		erp_len = 3;
 		erpinfo_content = 0;
-		if (ieee->current_network.buseprotection)
+		if (cur_net->buseprotection)
 			erpinfo_content |= ERP_UseProtection;
 	} else
 		erp_len = 0;
@@ -890,20 +891,17 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
 	beacon_buf = skb_put(skb, (beacon_size - ieee->tx_headroom));
 	ether_addr_copy(beacon_buf->header.addr1, dest);
 	ether_addr_copy(beacon_buf->header.addr2, ieee->dev->dev_addr);
-	ether_addr_copy(beacon_buf->header.addr3, ieee->current_network.bssid);
+	ether_addr_copy(beacon_buf->header.addr3, cur_net->bssid);
 
 	beacon_buf->header.duration_id = 0;
-	beacon_buf->beacon_interval =
-		cpu_to_le16(ieee->current_network.beacon_interval);
-	beacon_buf->capability =
-		cpu_to_le16(ieee->current_network.capability &
-		WLAN_CAPABILITY_IBSS);
-	beacon_buf->capability |=
-		cpu_to_le16(ieee->current_network.capability &
-		WLAN_CAPABILITY_SHORT_PREAMBLE);
-
-	if (ieee->short_slot && (ieee->current_network.capability &
-	    WLAN_CAPABILITY_SHORT_SLOT_TIME))
+	beacon_buf->beacon_interval = cpu_to_le16(cur_net->beacon_interval);
+	beacon_buf->capability = cpu_to_le16(cur_net->capability &
+					     WLAN_CAPABILITY_IBSS);
+	beacon_buf->capability |= cpu_to_le16(cur_net->capability &
+					      WLAN_CAPABILITY_SHORT_PREAMBLE);
+
+	if (ieee->short_slot &&
+	    (cur_net->capability & WLAN_CAPABILITY_SHORT_SLOT_TIME))
 		beacon_buf->capability |=
 			cpu_to_le16(WLAN_CAPABILITY_SHORT_SLOT_TIME);
 
@@ -924,18 +922,18 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
 
 	*(tag++) = MFIE_TYPE_RATES;
 	*(tag++) = rate_len-2;
-	memcpy(tag, ieee->current_network.rates, rate_len-2);
+	memcpy(tag, cur_net->rates, rate_len - 2);
 	tag += rate_len-2;
 
 	*(tag++) = MFIE_TYPE_DS_SET;
 	*(tag++) = 1;
-	*(tag++) = ieee->current_network.channel;
+	*(tag++) = cur_net->channel;
 
 	if (atim_len) {
 		u16 val16;
 		*(tag++) = MFIE_TYPE_IBSS_SET;
 		*(tag++) = 2;
-		val16 = ieee->current_network.atim_window;
+		val16 = cur_net->atim_window;
 		memcpy((u8 *)tag, (u8 *)&val16, 2);
 		tag += 2;
 	}
@@ -948,7 +946,7 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
 	if (rate_ex_len) {
 		*(tag++) = MFIE_TYPE_RATES_EX;
 		*(tag++) = rate_ex_len-2;
-		memcpy(tag, ieee->current_network.rates_ex, rate_ex_len-2);
+		memcpy(tag, cur_net->rates_ex, rate_ex_len - 2);
 		tag += rate_ex_len-2;
 	}
 
@@ -1053,6 +1051,7 @@ static struct sk_buff *rtllib_null_func(struct rtllib_device *ieee, short pwr)
 {
 	struct sk_buff *skb;
 	struct rtllib_hdr_3addr *hdr;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	skb = dev_alloc_skb(sizeof(struct rtllib_hdr_3addr)+ieee->tx_headroom);
 	if (!skb)
@@ -1062,9 +1061,9 @@ static struct sk_buff *rtllib_null_func(struct rtllib_device *ieee, short pwr)
 
 	hdr = skb_put(skb, sizeof(struct rtllib_hdr_3addr));
 
-	ether_addr_copy(hdr->addr1, ieee->current_network.bssid);
+	ether_addr_copy(hdr->addr1, cur_net->bssid);
 	ether_addr_copy(hdr->addr2, ieee->dev->dev_addr);
-	ether_addr_copy(hdr->addr3, ieee->current_network.bssid);
+	ether_addr_copy(hdr->addr3, cur_net->bssid);
 
 	hdr->frame_ctl = cpu_to_le16(RTLLIB_FTYPE_DATA |
 		RTLLIB_STYPE_NULLFUNC | RTLLIB_FCTL_TODS |
@@ -1079,6 +1078,7 @@ static struct sk_buff *rtllib_pspoll_func(struct rtllib_device *ieee)
 {
 	struct sk_buff *skb;
 	struct rtllib_pspoll_hdr *hdr;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	skb = dev_alloc_skb(sizeof(struct rtllib_pspoll_hdr)+ieee->tx_headroom);
 	if (!skb)
@@ -1088,7 +1088,7 @@ static struct sk_buff *rtllib_pspoll_func(struct rtllib_device *ieee)
 
 	hdr = skb_put(skb, sizeof(struct rtllib_pspoll_hdr));
 
-	ether_addr_copy(hdr->bssid, ieee->current_network.bssid);
+	ether_addr_copy(hdr->bssid, cur_net->bssid);
 	ether_addr_copy(hdr->ta, ieee->dev->dev_addr);
 
 	hdr->aid = cpu_to_le16(ieee->assoc_id | 0xc000);
@@ -1162,6 +1162,7 @@ rtllib_association_req(struct rtllib_network *beacon,
 	struct lib80211_crypt_data *crypt;
 	int encrypt;
 	int	PMKCacheIdx;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	unsigned int rate_len = (beacon->rates_len ?
 				(beacon->rates_len + 2) : 0) +
@@ -1209,7 +1210,7 @@ rtllib_association_req(struct rtllib_network *beacon,
 	if (beacon->BssCcxVerNumber >= 2)
 		cxvernum_ie_len = 5+2;
 
-	PMKCacheIdx = SecIsInPMKIDList(ieee, ieee->current_network.bssid);
+	PMKCacheIdx = SecIsInPMKIDList(ieee, cur_net->bssid);
 	if (PMKCacheIdx >= 0) {
 		wpa_ie_len += 18;
 		netdev_info(ieee->dev, "[PMK cache]: WPA2 IE length: %x\n",
@@ -1515,10 +1516,11 @@ static void rtllib_associate_complete_wq(void *data)
 				     container_of_work_rsl(data,
 				     struct rtllib_device,
 				     associate_complete_wq);
+	struct rtllib_network *cur_net = &ieee->current_network;
 	struct rt_pwr_save_ctrl *pPSC = &(ieee->PowerSaveControl);
 
 	netdev_info(ieee->dev, "Associated successfully with %pM\n",
-		    ieee->current_network.bssid);
+		    cur_net->bssid);
 	if (!ieee->is_silent_reset) {
 		netdev_info(ieee->dev, "normal associate\n");
 		notify_wx_assoc_event(ieee);
@@ -1545,9 +1547,7 @@ static void rtllib_associate_complete_wq(void *data)
 			    ieee->pHTInfo->bEnableHT);
 		memset(ieee->dot11HTOperationalRateSet, 0, 16);
 	}
-	ieee->LinkDetectInfo.SlotNum = 2 * (1 +
-				       ieee->current_network.beacon_interval /
-				       500);
+	ieee->LinkDetectInfo.SlotNum = 2 * (1 + cur_net->beacon_interval / 500);
 	if (ieee->LinkDetectInfo.NumRecvBcnInPeriod == 0 ||
 	    ieee->LinkDetectInfo.NumRecvDataInPeriod == 0) {
 		ieee->LinkDetectInfo.NumRecvBcnInPeriod = 1;
@@ -1585,6 +1585,8 @@ static void rtllib_associate_procedure_wq(void *data)
 	struct rtllib_device *ieee = container_of_dwork_rsl(data,
 				     struct rtllib_device,
 				     associate_procedure_wq);
+	struct rtllib_network *cur_net = &ieee->current_network;
+
 	rtllib_stop_scan_syncro(ieee);
 	if (ieee->rtllib_ips_leave != NULL)
 		ieee->rtllib_ips_leave(ieee->dev);
@@ -1594,8 +1596,7 @@ static void rtllib_associate_procedure_wq(void *data)
 		ieee->data_hard_stop(ieee->dev);
 
 	rtllib_stop_scan(ieee);
-	RT_TRACE(COMP_DBG, "===>%s(), chan:%d\n", __func__,
-		 ieee->current_network.channel);
+	RT_TRACE(COMP_DBG, "===>%s(), chan:%d\n", __func__, cur_net->channel);
 	HTSetConnectBwMode(ieee, HT_CHANNEL_WIDTH_20, HT_EXTCHNL_OFFSET_NO_EXT);
 	if (ieee->eRFPowerState == eRfOff) {
 		RT_TRACE(COMP_DBG,
@@ -1608,7 +1609,7 @@ static void rtllib_associate_procedure_wq(void *data)
 	}
 	ieee->associate_seq = 1;
 
-	rtllib_associate_step1(ieee, ieee->current_network.bssid);
+	rtllib_associate_step1(ieee, cur_net->bssid);
 
 	mutex_unlock(&ieee->wx_mutex);
 }
@@ -1618,6 +1619,7 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
 {
 	u8 tmp_ssid[IW_ESSID_MAX_SIZE + 1];
 	int tmp_ssid_len = 0;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	short apset, ssidset, ssidbroad, apmatch, ssidmatch;
 
@@ -1646,13 +1648,11 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
 		apset = ieee->wap_set;
 		ssidset = ieee->ssid_set;
 		ssidbroad =  !(net->ssid_len == 0 || net->ssid[0] == '\0');
-		apmatch = (memcmp(ieee->current_network.bssid, net->bssid,
-				  ETH_ALEN) == 0);
+		apmatch = (memcmp(cur_net->bssid, net->bssid, ETH_ALEN) == 0);
 		if (!ssidbroad) {
-			ssidmatch = (ieee->current_network.ssid_len ==
-				    net->hidden_ssid_len) &&
-				    (!strncmp(ieee->current_network.ssid,
-				    net->hidden_ssid, net->hidden_ssid_len));
+			ssidmatch = ((cur_net->ssid_len == net->hidden_ssid_len) &&
+				     (!strncmp(cur_net->ssid, net->hidden_ssid,
+					       net->hidden_ssid_len)));
 			if (net->hidden_ssid_len > 0) {
 				strncpy(net->ssid, net->hidden_ssid,
 					net->hidden_ssid_len);
@@ -1660,10 +1660,9 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
 				ssidbroad = 1;
 			}
 		} else
-			ssidmatch =
-			   (ieee->current_network.ssid_len == net->ssid_len) &&
-			   (!strncmp(ieee->current_network.ssid, net->ssid,
-			   net->ssid_len));
+			ssidmatch = ((cur_net->ssid_len == net->ssid_len) &&
+				     (!strncmp(cur_net->ssid, net->ssid,
+					       net->ssid_len)));
 
 		/* if the user set the AP check if match.
 		 * if the network does not broadcast essid we check the
@@ -1684,26 +1683,26 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
 			 * essid provided by the user.
 			 */
 			if (!ssidbroad) {
-				strncpy(tmp_ssid, ieee->current_network.ssid,
+				strncpy(tmp_ssid, cur_net->ssid,
 					IW_ESSID_MAX_SIZE);
-				tmp_ssid_len = ieee->current_network.ssid_len;
+				tmp_ssid_len = cur_net->ssid_len;
 			}
 			memcpy(&ieee->current_network, net,
 			       sizeof(struct rtllib_network));
 			if (!ssidbroad) {
-				strncpy(ieee->current_network.ssid, tmp_ssid,
+				strncpy(cur_net->ssid, tmp_ssid,
 					IW_ESSID_MAX_SIZE);
-				ieee->current_network.ssid_len = tmp_ssid_len;
+				cur_net->ssid_len = tmp_ssid_len;
 			}
 			netdev_info(ieee->dev,
 				    "Linking with %s,channel:%d, qos:%d, myHT:%d, networkHT:%d, mode:%x cur_net.flags:0x%x\n",
-				    ieee->current_network.ssid,
-				    ieee->current_network.channel,
-				    ieee->current_network.qos_data.supported,
+				    cur_net->ssid,
+				    cur_net->channel,
+				    cur_net->qos_data.supported,
 				    ieee->pHTInfo->bEnableHT,
-				    ieee->current_network.bssht.bdSupportHT,
-				    ieee->current_network.mode,
-				    ieee->current_network.flags);
+				    cur_net->bssht.bdSupportHT,
+				    cur_net->mode,
+				    cur_net->flags);
 
 			if ((rtllib_act_scanning(ieee, false)) &&
 			   !(ieee->softmac_features & IEEE_SOFTMAC_SCAN))
@@ -1714,8 +1713,8 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
 			if (ieee->iw_mode == IW_MODE_INFRA) {
 				/* Join the network for the first time */
 				ieee->AsocRetryCount = 0;
-				if ((ieee->current_network.qos_data.supported == 1) &&
-				    ieee->current_network.bssht.bdSupportHT)
+				if (((cur_net->qos_data.supported == 1) &&
+				     cur_net->bssht.bdSupportHT))
 					HTResetSelfAndSavePeerSetting(ieee,
 						 &(ieee->current_network));
 				else
@@ -1829,13 +1828,13 @@ static short probe_rq_parse(struct rtllib_device *ieee, struct sk_buff *skb,
 	struct rtllib_hdr_3addr   *header =
 		(struct rtllib_hdr_3addr   *) skb->data;
 	bool bssid_match;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	if (skb->len < sizeof(struct rtllib_hdr_3addr))
 		return -1; /* corrupted */
 
-	bssid_match =
-	  (!ether_addr_equal(header->addr3, ieee->current_network.bssid)) &&
-	  (!is_broadcast_ether_addr(header->addr3));
+	bssid_match = ((!ether_addr_equal(header->addr3, cur_net->bssid)) &&
+		       (!is_broadcast_ether_addr(header->addr3)));
 	if (bssid_match)
 		return -1;
 
@@ -1862,7 +1861,7 @@ static short probe_rq_parse(struct rtllib_device *ieee, struct sk_buff *skb,
 	if (!ssid)
 		return 1; /* ssid not found in tagged param */
 
-	return !strncmp(ssid, ieee->current_network.ssid, ssidlen);
+	return !strncmp(ssid, cur_net->ssid, ssidlen);
 }
 
 static int assoc_rq_parse(struct net_device *dev, struct sk_buff *skb, u8 *dest)
@@ -1887,6 +1886,7 @@ static inline u16 assoc_parse(struct rtllib_device *ieee, struct sk_buff *skb,
 {
 	struct rtllib_assoc_response_frame *response_head;
 	u16 status_code;
+	struct rtllib_network *cur_net = &ieee->current_network;
 
 	if (skb->len <  sizeof(struct rtllib_assoc_response_frame)) {
 		netdev_dbg(ieee->dev, "Invalid len in auth resp: %d\n",
@@ -1901,7 +1901,7 @@ static inline u16 assoc_parse(struct rtllib_device *ieee, struct sk_buff *skb,
 	if ((status_code == WLAN_STATUS_ASSOC_DENIED_RATES ||
 	   status_code == WLAN_STATUS_CAPS_UNSUPPORTED) &&
 	   ((ieee->mode == IEEE_G) &&
-	   (ieee->current_network.mode == IEEE_N_24G) &&
+	   (cur_net->mode == IEEE_N_24G) &&
 	   (ieee->AsocRetryCount++ < (RT_ASOC_RETRY_LIMIT-1)))) {
 		ieee->pHTInfo->IOTAction |= HT_IOT_ACT_PURE_N_MODE;
 	} else {
@@ -1970,6 +1970,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 {
 	int timeout;
 	u8 dtim;
+	struct rtllib_network *cur_net = &ieee->current_network;
 	struct rt_pwr_save_ctrl *pPSC = &(ieee->PowerSaveControl);
 
 	if (ieee->LPSDelayCnt) {
@@ -1977,11 +1978,11 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 		return 0;
 	}
 
-	dtim = ieee->current_network.dtim_data;
+	dtim = cur_net->dtim_data;
 	if (!(dtim & RTLLIB_DTIM_VALID))
 		return 0;
-	timeout = ieee->current_network.beacon_interval;
-	ieee->current_network.dtim_data = RTLLIB_DTIM_INVALID;
+	timeout = cur_net->beacon_interval;
+	cur_net->dtim_data = RTLLIB_DTIM_INVALID;
 	/* there's no need to nofity AP that I find you buffered
 	 * with broadcast packet
 	 */
@@ -2009,7 +2010,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 			if (pPSC->RegMaxLPSAwakeIntvl == 0)
 				MaxPeriod = 1;
 			else if (pPSC->RegMaxLPSAwakeIntvl == 0xFF)
-				MaxPeriod = ieee->current_network.dtim_period;
+				MaxPeriod = cur_net->dtim_period;
 			else
 				MaxPeriod = pPSC->RegMaxLPSAwakeIntvl;
 			pPSC->LPSAwakeIntvl = (pPSC->LPSAwakeIntvl >=
@@ -2018,8 +2019,8 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 		}
 		{
 			u8 LPSAwakeIntvl_tmp = 0;
-			u8 period = ieee->current_network.dtim_period;
-			u8 count = ieee->current_network.tim.tim_count;
+			u8 period = cur_net->dtim_period;
+			u8 count = cur_net->tim.tim_count;
 
 			if (count == 0) {
 				if (pPSC->LPSAwakeIntvl > period)
@@ -2033,7 +2034,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 
 			} else {
 				if (pPSC->LPSAwakeIntvl >
-				    ieee->current_network.tim.tim_count)
+				    cur_net->tim.tim_count)
 					LPSAwakeIntvl_tmp = count +
 					(pPSC->LPSAwakeIntvl - count) -
 					((pPSC->LPSAwakeIntvl-count)%period);
@@ -2041,8 +2042,8 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
 					LPSAwakeIntvl_tmp = pPSC->LPSAwakeIntvl;
 			}
 
-		*time = ieee->current_network.last_dtim_sta_time
-			+ msecs_to_jiffies(ieee->current_network.beacon_interval *
+		*time = cur_net->last_dtim_sta_time
+			+ msecs_to_jiffies(cur_net->beacon_interval *
 			LPSAwakeIntvl_tmp);
 	}
 	}
@@ -2211,6 +2212,7 @@ static inline int
 rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
 		     struct rtllib_rx_stats *rx_stats)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	u16 errcode;
 	int aid;
 	u8 *ies;
@@ -2238,7 +2240,7 @@ rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
 			/* station support qos */
 			/* Let the register setting default with Legacy station */
 			assoc_resp = (struct rtllib_assoc_response_frame *)skb->data;
-			if (ieee->current_network.qos_data.supported == 1) {
+			if (cur_net->qos_data.supported == 1) {
 				if (rtllib_parse_info_param(ieee, assoc_resp->info_element,
 							rx_stats->len - sizeof(*assoc_resp),
 							network, rx_stats)) {
@@ -2292,6 +2294,7 @@ rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
 
 static void rtllib_rx_auth_resp(struct rtllib_device *ieee, struct sk_buff *skb)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	u16 errcode;
 	u8 *challenge;
 	int chlen = 0;
@@ -2324,15 +2327,13 @@ static void rtllib_rx_auth_resp(struct rtllib_device *ieee, struct sk_buff *skb)
 		}
 		/* Dummy wirless mode setting to avoid encryption issue */
 		if (bSupportNmode) {
-			ieee->SetWirelessMode(ieee->dev,
-					      ieee->current_network.mode);
+			ieee->SetWirelessMode(ieee->dev, cur_net->mode);
 		} else {
 			/*TODO*/
 			ieee->SetWirelessMode(ieee->dev, IEEE_G);
 		}
 
-		if ((ieee->current_network.mode == IEEE_N_24G) &&
-		    bHalfSupportNmode) {
+		if ((cur_net->mode == IEEE_N_24G) && bHalfSupportNmode) {
 			netdev_info(ieee->dev, "======>enter half N mode\n");
 			ieee->bHalfWirelessN24GMode = true;
 		} else {
@@ -2366,9 +2367,10 @@ static inline int
 rtllib_rx_deauth(struct rtllib_device *ieee, struct sk_buff *skb)
 {
 	struct rtllib_hdr_3addr *header = (struct rtllib_hdr_3addr *) skb->data;
+	struct rtllib_network *cur_net = &ieee->current_network;
 	u16 frame_ctl;
 
-	if (memcmp(header->addr3, ieee->current_network.bssid, ETH_ALEN) != 0)
+	if (memcmp(header->addr3, cur_net->bssid, ETH_ALEN) != 0)
 		return 0;
 
 	/* FIXME for now repeat all the association procedure
@@ -2542,21 +2544,20 @@ void rtllib_wake_all_queues(struct rtllib_device *ieee)
 /* called in user context only */
 static void rtllib_start_master_bss(struct rtllib_device *ieee)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	ieee->assoc_id = 1;
 
-	if (ieee->current_network.ssid_len == 0) {
-		strncpy(ieee->current_network.ssid,
-			RTLLIB_DEFAULT_TX_ESSID,
+	if (cur_net->ssid_len == 0) {
+		strncpy(cur_net->ssid, RTLLIB_DEFAULT_TX_ESSID,
 			IW_ESSID_MAX_SIZE);
 
-		ieee->current_network.ssid_len =
-				 strlen(RTLLIB_DEFAULT_TX_ESSID);
+		cur_net->ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
 		ieee->ssid_set = 1;
 	}
 
-	ether_addr_copy(ieee->current_network.bssid, ieee->dev->dev_addr);
+	ether_addr_copy(cur_net->bssid, ieee->dev->dev_addr);
 
-	ieee->set_chan(ieee->dev, ieee->current_network.channel);
+	ieee->set_chan(ieee->dev, cur_net->channel);
 	ieee->state = RTLLIB_LINKED;
 	ieee->link_change(ieee->dev);
 	notify_wx_assoc_event(ieee);
@@ -2582,6 +2583,8 @@ static void rtllib_start_ibss_wq(void *data)
 {
 	struct rtllib_device *ieee = container_of_dwork_rsl(data,
 				     struct rtllib_device, start_ibss_wq);
+	struct rtllib_network *cur_net = &ieee->current_network;
+
 	/* iwconfig mode ad-hoc will schedule this and return
 	 * on the other hand this will block further iwconfig SET
 	 * operations because of the wx_mutex hold.
@@ -2595,9 +2598,9 @@ static void rtllib_start_ibss_wq(void *data)
 	}
 	mutex_lock(&ieee->wx_mutex);
 
-	if (ieee->current_network.ssid_len == 0) {
-		strcpy(ieee->current_network.ssid, RTLLIB_DEFAULT_TX_ESSID);
-		ieee->current_network.ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
+	if (cur_net->ssid_len == 0) {
+		strcpy(cur_net->ssid, RTLLIB_DEFAULT_TX_ESSID);
+		cur_net->ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
 		ieee->ssid_set = 1;
 	}
 
@@ -2627,57 +2630,49 @@ static void rtllib_start_ibss_wq(void *data)
 	/* the network definitively is not here.. create a new cell */
 	if (ieee->state == RTLLIB_NOLINK) {
 		netdev_info(ieee->dev, "creating new IBSS cell\n");
-		ieee->current_network.channel = ieee->IbssStartChnl;
+		cur_net->channel = ieee->IbssStartChnl;
 		if (!ieee->wap_set)
-			eth_random_addr(ieee->current_network.bssid);
+			eth_random_addr(cur_net->bssid);
 
 		if (ieee->modulation & RTLLIB_CCK_MODULATION) {
 
-			ieee->current_network.rates_len = 4;
+			cur_net->rates_len = 4;
 
-			ieee->current_network.rates[0] =
+			cur_net->rates[0] =
 				 RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_1MB;
-			ieee->current_network.rates[1] =
+			cur_net->rates[1] =
 				 RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_2MB;
-			ieee->current_network.rates[2] =
+			cur_net->rates[2] =
 				 RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_5MB;
-			ieee->current_network.rates[3] =
+			cur_net->rates[3] =
 				 RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_11MB;
 
 		} else
-			ieee->current_network.rates_len = 0;
+			cur_net->rates_len = 0;
 
 		if (ieee->modulation & RTLLIB_OFDM_MODULATION) {
-			ieee->current_network.rates_ex_len = 8;
-
-			ieee->current_network.rates_ex[0] =
-						 RTLLIB_OFDM_RATE_6MB;
-			ieee->current_network.rates_ex[1] =
-						 RTLLIB_OFDM_RATE_9MB;
-			ieee->current_network.rates_ex[2] =
-						 RTLLIB_OFDM_RATE_12MB;
-			ieee->current_network.rates_ex[3] =
-						 RTLLIB_OFDM_RATE_18MB;
-			ieee->current_network.rates_ex[4] =
-						 RTLLIB_OFDM_RATE_24MB;
-			ieee->current_network.rates_ex[5] =
-						 RTLLIB_OFDM_RATE_36MB;
-			ieee->current_network.rates_ex[6] =
-						 RTLLIB_OFDM_RATE_48MB;
-			ieee->current_network.rates_ex[7] =
-						 RTLLIB_OFDM_RATE_54MB;
+			cur_net->rates_ex_len = 8;
+
+			cur_net->rates_ex[0] = RTLLIB_OFDM_RATE_6MB;
+			cur_net->rates_ex[1] = RTLLIB_OFDM_RATE_9MB;
+			cur_net->rates_ex[2] = RTLLIB_OFDM_RATE_12MB;
+			cur_net->rates_ex[3] = RTLLIB_OFDM_RATE_18MB;
+			cur_net->rates_ex[4] = RTLLIB_OFDM_RATE_24MB;
+			cur_net->rates_ex[5] = RTLLIB_OFDM_RATE_36MB;
+			cur_net->rates_ex[6] = RTLLIB_OFDM_RATE_48MB;
+			cur_net->rates_ex[7] = RTLLIB_OFDM_RATE_54MB;
 
 			ieee->rate = 108;
 		} else {
-			ieee->current_network.rates_ex_len = 0;
+			cur_net->rates_ex_len = 0;
 			ieee->rate = 22;
 		}
 
-		ieee->current_network.qos_data.supported = 0;
+		cur_net->qos_data.supported = 0;
 		ieee->SetWirelessMode(ieee->dev, IEEE_G);
-		ieee->current_network.mode = ieee->mode;
-		ieee->current_network.atim_window = 0;
-		ieee->current_network.capability = WLAN_CAPABILITY_IBSS;
+		cur_net->mode = ieee->mode;
+		cur_net->atim_window = 0;
+		cur_net->capability = WLAN_CAPABILITY_IBSS;
 	}
 
 	netdev_info(ieee->dev, "%s(): ieee->mode = %d\n", __func__, ieee->mode);
@@ -2913,6 +2908,7 @@ EXPORT_SYMBOL(rtllib_softmac_start_protocol);
 
 void rtllib_start_protocol(struct rtllib_device *ieee)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	short ch = 0;
 	int i = 0;
 
@@ -2923,17 +2919,17 @@ void rtllib_start_protocol(struct rtllib_device *ieee)
 
 	ieee->proto_started = 1;
 
-	if (ieee->current_network.channel == 0) {
+	if (cur_net->channel == 0) {
 		do {
 			ch++;
 			if (ch > MAX_CHANNEL_NUMBER)
 				return; /* no channel found */
 		} while (!ieee->active_channel_map[ch]);
-		ieee->current_network.channel = ch;
+		cur_net->channel = ch;
 	}
 
-	if (ieee->current_network.beacon_interval == 0)
-		ieee->current_network.beacon_interval = 100;
+	if (cur_net->beacon_interval == 0)
+		cur_net->beacon_interval = 100;
 
 	for (i = 0; i < 17; i++) {
 		ieee->last_rxseq_num[i] = -1;
@@ -3136,6 +3132,7 @@ void SendDisassociation(struct rtllib_device *ieee, bool deauth, u16 asRsn)
 
 u8 rtllib_ap_sec_type(struct rtllib_device *ieee)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	static u8 ccmp_ie[4] = {0x00, 0x50, 0xf2, 0x04};
 	static u8 ccmp_rsn_ie[4] = {0x00, 0x0f, 0xac, 0x04};
 	int wpa_ie_len = ieee->wpa_ie_len;
@@ -3143,7 +3140,7 @@ u8 rtllib_ap_sec_type(struct rtllib_device *ieee)
 	int encrypt;
 
 	crypt = ieee->crypt_info.crypt[ieee->crypt_info.tx_keyidx];
-	encrypt = (ieee->current_network.capability & WLAN_CAPABILITY_PRIVACY)
+	encrypt = (cur_net->capability & WLAN_CAPABILITY_PRIVACY)
 		  || (ieee->host_encrypt && crypt && crypt->ops &&
 		  (strcmp(crypt->ops->name, "R-WEP") == 0));
 
@@ -3250,6 +3247,7 @@ EXPORT_SYMBOL(rtllib_MgntDisconnect);
 
 void notify_wx_assoc_event(struct rtllib_device *ieee)
 {
+	struct rtllib_network *cur_net = &ieee->current_network;
 	union iwreq_data wrqu;
 
 	if (ieee->cannot_notify)
@@ -3257,8 +3255,7 @@ void notify_wx_assoc_event(struct rtllib_device *ieee)
 
 	wrqu.ap_addr.sa_family = ARPHRD_ETHER;
 	if (ieee->state == RTLLIB_LINKED)
-		memcpy(wrqu.ap_addr.sa_data, ieee->current_network.bssid,
-		       ETH_ALEN);
+		memcpy(wrqu.ap_addr.sa_data, cur_net->bssid, ETH_ALEN);
 	else {
 
 		netdev_info(ieee->dev, "%s(): Tell user space disconnected\n",

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 2/2] staging: rtl8192e: fix line over 80 characters
  2018-06-09  6:45 ` Joe Perches
@ 2018-06-16  4:20   ` 김현일
  0 siblings, 0 replies; 3+ messages in thread
From: 김현일 @ 2018-06-16  4:20 UTC (permalink / raw)
  To: Joe Perches
  Cc: gregkh, davem, Colin King, johannes.berg, Linyu.Yuan, Kees Cook,
	Malcolm Priestley, devel, linux-kernel, kernel-janitors

Hi Joe.

Thanks to your notice.
i will check again.

Regards,
Hyunil Kim.

On Sat, Jun 9, 2018 at 3:45 PM, Joe Perches <joe@perches.com> wrote:
> On Sat, 2018-06-09 at 15:07 +0900, thtlwlsmsgkfla@gmail.com wrote:
>> *fix checkpatch.pl warnings:
>>  WARNING: line over 80 characters
>
> Consider using a temporary for these ieee->current_network.
> dereferences.
>
>> diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
> []
>> @@ -2596,7 +2596,8 @@ static void rtllib_start_ibss_wq(void *data)
>>
>>       if (ieee->current_network.ssid_len == 0) {
>>               strcpy(ieee->current_network.ssid, RTLLIB_DEFAULT_TX_ESSID);
>> -             ieee->current_network.ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
>> +             ieee->current_network.ssid_len =
>> +                     strlen(RTLLIB_DEFAULT_TX_ESSID);
>>               ieee->ssid_set = 1;
>>       }
>
> For instance:
>
> Adding a temporary for all ieee->current_network
> deferences actually reduces line count in this file.
>
> ---
>  drivers/staging/rtl8192e/rtllib_softmac.c | 281 +++++++++++++++---------------
>  1 file changed, 139 insertions(+), 142 deletions(-)
>
> diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging/rtl8192e/rtllib_softmac.c
> index 919231fec09c..5733a1ab59d1 100644
> --- a/drivers/staging/rtl8192e/rtllib_softmac.c
> +++ b/drivers/staging/rtl8192e/rtllib_softmac.c
> @@ -157,10 +157,10 @@ MgntQuery_TxRateExcludeCCKRates(struct rtllib_device *ieee)
>         u16     i;
>         u8      QueryRate = 0;
>         u8      BasicRate;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
> -
> -       for (i = 0; i < ieee->current_network.rates_len; i++) {
> -               BasicRate = ieee->current_network.rates[i]&0x7F;
> +       for (i = 0; i < cur_net->rates_len; i++) {
> +               BasicRate = cur_net->rates[i] & 0x7F;
>                 if (!rtllib_is_cck_rate(BasicRate)) {
>                         if (QueryRate == 0) {
>                                 QueryRate = BasicRate;
> @@ -338,8 +338,9 @@ static inline struct sk_buff *rtllib_probe_req(struct rtllib_device *ieee)
>         u8 *tag;
>         struct sk_buff *skb;
>         struct rtllib_probe_request *req;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
> -       len = ieee->current_network.ssid_len;
> +       len = cur_net->ssid_len;
>
>         rate_len = rtllib_MFIE_rate_len(ieee);
>
> @@ -363,7 +364,7 @@ static inline struct sk_buff *rtllib_probe_req(struct rtllib_device *ieee)
>
>         *tag++ = MFIE_TYPE_SSID;
>         *tag++ = len;
> -       memcpy(tag, ieee->current_network.ssid, len);
> +       memcpy(tag, cur_net->ssid, len);
>         tag += len;
>
>         rtllib_MFIE_Brate(ieee, &tag);
> @@ -377,6 +378,7 @@ static struct sk_buff *rtllib_get_beacon_(struct rtllib_device *ieee);
>  static void rtllib_send_beacon(struct rtllib_device *ieee)
>  {
>         struct sk_buff *skb;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         if (!ieee->ieee_up)
>                 return;
> @@ -389,7 +391,7 @@ static void rtllib_send_beacon(struct rtllib_device *ieee)
>
>         if (ieee->beacon_txing && ieee->ieee_up)
>                 mod_timer(&ieee->beacon_timer, jiffies +
> -                         (msecs_to_jiffies(ieee->current_network.beacon_interval - 5)));
> +                         (msecs_to_jiffies(cur_net->beacon_interval - 5)));
>  }
>
>
> @@ -578,7 +580,8 @@ static void rtllib_softmac_scan_wq(void *data)
>  {
>         struct rtllib_device *ieee = container_of_dwork_rsl(data,
>                                      struct rtllib_device, softmac_scan_wq);
> -       u8 last_channel = ieee->current_network.channel;
> +       struct rtllib_network *cur_net = &ieee->current_network;
> +       u8 last_channel = cur_net->channel;
>
>         rtllib_update_active_chan_map(ieee);
>
> @@ -597,22 +600,20 @@ static void rtllib_softmac_scan_wq(void *data)
>         }
>
>         do {
> -               ieee->current_network.channel =
> -                       (ieee->current_network.channel + 1) %
> -                       MAX_CHANNEL_NUMBER;
> +               cur_net->channel = (cur_net->channel + 1) % MAX_CHANNEL_NUMBER;
>                 if (ieee->scan_watch_dog++ > MAX_CHANNEL_NUMBER) {
> -                       if (!ieee->active_channel_map[ieee->current_network.channel])
> -                               ieee->current_network.channel = 6;
> +                       if (!ieee->active_channel_map[cur_net->channel])
> +                               cur_net->channel = 6;
>                         goto out; /* no good chans */
>                 }
> -       } while (!ieee->active_channel_map[ieee->current_network.channel]);
> +       } while (!ieee->active_channel_map[cur_net->channel]);
>
>         if (ieee->scanning_continue == 0)
>                 goto out;
>
> -       ieee->set_chan(ieee->dev, ieee->current_network.channel);
> +       ieee->set_chan(ieee->dev, cur_net->channel);
>
> -       if (ieee->active_channel_map[ieee->current_network.channel] == 1)
> +       if (ieee->active_channel_map[cur_net->channel] == 1)
>                 rtllib_send_probe_requests(ieee, 0);
>
>         schedule_delayed_work(&ieee->softmac_scan_wq,
> @@ -624,7 +625,7 @@ static void rtllib_softmac_scan_wq(void *data)
>  out:
>         if (IS_DOT11D_ENABLE(ieee))
>                 DOT11D_ScanComplete(ieee);
> -       ieee->current_network.channel = last_channel;
> +       cur_net->channel = last_channel;
>
>  out1:
>         ieee->actscanning = false;
> @@ -822,11 +823,11 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
>         int encrypt;
>         int atim_len, erp_len;
>         struct lib80211_crypt_data *crypt;
> -
> -       char *ssid = ieee->current_network.ssid;
> -       int ssid_len = ieee->current_network.ssid_len;
> -       int rate_len = ieee->current_network.rates_len+2;
> -       int rate_ex_len = ieee->current_network.rates_ex_len;
> +       struct rtllib_network *cur_net = &ieee->current_network;
> +       char *ssid = cur_net->ssid;
> +       int ssid_len = cur_net->ssid_len;
> +       int rate_len = cur_net->rates_len + 2;
> +       int rate_ex_len = cur_net->rates_ex_len;
>         int wpa_ie_len = ieee->wpa_ie_len;
>         u8 erpinfo_content = 0;
>
> @@ -841,17 +842,17 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
>         if (rate_ex_len > 0)
>                 rate_ex_len += 2;
>
> -       if (ieee->current_network.capability & WLAN_CAPABILITY_IBSS)
> +       if (cur_net->capability & WLAN_CAPABILITY_IBSS)
>                 atim_len = 4;
>         else
>                 atim_len = 0;
>
> -       if ((ieee->current_network.mode == IEEE_G) ||
> -          (ieee->current_network.mode == IEEE_N_24G &&
> +       if ((cur_net->mode == IEEE_G) ||
> +          (cur_net->mode == IEEE_N_24G &&
>            ieee->pHTInfo->bCurSuppCCK)) {
>                 erp_len = 3;
>                 erpinfo_content = 0;
> -               if (ieee->current_network.buseprotection)
> +               if (cur_net->buseprotection)
>                         erpinfo_content |= ERP_UseProtection;
>         } else
>                 erp_len = 0;
> @@ -890,20 +891,17 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
>         beacon_buf = skb_put(skb, (beacon_size - ieee->tx_headroom));
>         ether_addr_copy(beacon_buf->header.addr1, dest);
>         ether_addr_copy(beacon_buf->header.addr2, ieee->dev->dev_addr);
> -       ether_addr_copy(beacon_buf->header.addr3, ieee->current_network.bssid);
> +       ether_addr_copy(beacon_buf->header.addr3, cur_net->bssid);
>
>         beacon_buf->header.duration_id = 0;
> -       beacon_buf->beacon_interval =
> -               cpu_to_le16(ieee->current_network.beacon_interval);
> -       beacon_buf->capability =
> -               cpu_to_le16(ieee->current_network.capability &
> -               WLAN_CAPABILITY_IBSS);
> -       beacon_buf->capability |=
> -               cpu_to_le16(ieee->current_network.capability &
> -               WLAN_CAPABILITY_SHORT_PREAMBLE);
> -
> -       if (ieee->short_slot && (ieee->current_network.capability &
> -           WLAN_CAPABILITY_SHORT_SLOT_TIME))
> +       beacon_buf->beacon_interval = cpu_to_le16(cur_net->beacon_interval);
> +       beacon_buf->capability = cpu_to_le16(cur_net->capability &
> +                                            WLAN_CAPABILITY_IBSS);
> +       beacon_buf->capability |= cpu_to_le16(cur_net->capability &
> +                                             WLAN_CAPABILITY_SHORT_PREAMBLE);
> +
> +       if (ieee->short_slot &&
> +           (cur_net->capability & WLAN_CAPABILITY_SHORT_SLOT_TIME))
>                 beacon_buf->capability |=
>                         cpu_to_le16(WLAN_CAPABILITY_SHORT_SLOT_TIME);
>
> @@ -924,18 +922,18 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
>
>         *(tag++) = MFIE_TYPE_RATES;
>         *(tag++) = rate_len-2;
> -       memcpy(tag, ieee->current_network.rates, rate_len-2);
> +       memcpy(tag, cur_net->rates, rate_len - 2);
>         tag += rate_len-2;
>
>         *(tag++) = MFIE_TYPE_DS_SET;
>         *(tag++) = 1;
> -       *(tag++) = ieee->current_network.channel;
> +       *(tag++) = cur_net->channel;
>
>         if (atim_len) {
>                 u16 val16;
>                 *(tag++) = MFIE_TYPE_IBSS_SET;
>                 *(tag++) = 2;
> -               val16 = ieee->current_network.atim_window;
> +               val16 = cur_net->atim_window;
>                 memcpy((u8 *)tag, (u8 *)&val16, 2);
>                 tag += 2;
>         }
> @@ -948,7 +946,7 @@ static struct sk_buff *rtllib_probe_resp(struct rtllib_device *ieee,
>         if (rate_ex_len) {
>                 *(tag++) = MFIE_TYPE_RATES_EX;
>                 *(tag++) = rate_ex_len-2;
> -               memcpy(tag, ieee->current_network.rates_ex, rate_ex_len-2);
> +               memcpy(tag, cur_net->rates_ex, rate_ex_len - 2);
>                 tag += rate_ex_len-2;
>         }
>
> @@ -1053,6 +1051,7 @@ static struct sk_buff *rtllib_null_func(struct rtllib_device *ieee, short pwr)
>  {
>         struct sk_buff *skb;
>         struct rtllib_hdr_3addr *hdr;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         skb = dev_alloc_skb(sizeof(struct rtllib_hdr_3addr)+ieee->tx_headroom);
>         if (!skb)
> @@ -1062,9 +1061,9 @@ static struct sk_buff *rtllib_null_func(struct rtllib_device *ieee, short pwr)
>
>         hdr = skb_put(skb, sizeof(struct rtllib_hdr_3addr));
>
> -       ether_addr_copy(hdr->addr1, ieee->current_network.bssid);
> +       ether_addr_copy(hdr->addr1, cur_net->bssid);
>         ether_addr_copy(hdr->addr2, ieee->dev->dev_addr);
> -       ether_addr_copy(hdr->addr3, ieee->current_network.bssid);
> +       ether_addr_copy(hdr->addr3, cur_net->bssid);
>
>         hdr->frame_ctl = cpu_to_le16(RTLLIB_FTYPE_DATA |
>                 RTLLIB_STYPE_NULLFUNC | RTLLIB_FCTL_TODS |
> @@ -1079,6 +1078,7 @@ static struct sk_buff *rtllib_pspoll_func(struct rtllib_device *ieee)
>  {
>         struct sk_buff *skb;
>         struct rtllib_pspoll_hdr *hdr;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         skb = dev_alloc_skb(sizeof(struct rtllib_pspoll_hdr)+ieee->tx_headroom);
>         if (!skb)
> @@ -1088,7 +1088,7 @@ static struct sk_buff *rtllib_pspoll_func(struct rtllib_device *ieee)
>
>         hdr = skb_put(skb, sizeof(struct rtllib_pspoll_hdr));
>
> -       ether_addr_copy(hdr->bssid, ieee->current_network.bssid);
> +       ether_addr_copy(hdr->bssid, cur_net->bssid);
>         ether_addr_copy(hdr->ta, ieee->dev->dev_addr);
>
>         hdr->aid = cpu_to_le16(ieee->assoc_id | 0xc000);
> @@ -1162,6 +1162,7 @@ rtllib_association_req(struct rtllib_network *beacon,
>         struct lib80211_crypt_data *crypt;
>         int encrypt;
>         int     PMKCacheIdx;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         unsigned int rate_len = (beacon->rates_len ?
>                                 (beacon->rates_len + 2) : 0) +
> @@ -1209,7 +1210,7 @@ rtllib_association_req(struct rtllib_network *beacon,
>         if (beacon->BssCcxVerNumber >= 2)
>                 cxvernum_ie_len = 5+2;
>
> -       PMKCacheIdx = SecIsInPMKIDList(ieee, ieee->current_network.bssid);
> +       PMKCacheIdx = SecIsInPMKIDList(ieee, cur_net->bssid);
>         if (PMKCacheIdx >= 0) {
>                 wpa_ie_len += 18;
>                 netdev_info(ieee->dev, "[PMK cache]: WPA2 IE length: %x\n",
> @@ -1515,10 +1516,11 @@ static void rtllib_associate_complete_wq(void *data)
>                                      container_of_work_rsl(data,
>                                      struct rtllib_device,
>                                      associate_complete_wq);
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         struct rt_pwr_save_ctrl *pPSC = &(ieee->PowerSaveControl);
>
>         netdev_info(ieee->dev, "Associated successfully with %pM\n",
> -                   ieee->current_network.bssid);
> +                   cur_net->bssid);
>         if (!ieee->is_silent_reset) {
>                 netdev_info(ieee->dev, "normal associate\n");
>                 notify_wx_assoc_event(ieee);
> @@ -1545,9 +1547,7 @@ static void rtllib_associate_complete_wq(void *data)
>                             ieee->pHTInfo->bEnableHT);
>                 memset(ieee->dot11HTOperationalRateSet, 0, 16);
>         }
> -       ieee->LinkDetectInfo.SlotNum = 2 * (1 +
> -                                      ieee->current_network.beacon_interval /
> -                                      500);
> +       ieee->LinkDetectInfo.SlotNum = 2 * (1 + cur_net->beacon_interval / 500);
>         if (ieee->LinkDetectInfo.NumRecvBcnInPeriod == 0 ||
>             ieee->LinkDetectInfo.NumRecvDataInPeriod == 0) {
>                 ieee->LinkDetectInfo.NumRecvBcnInPeriod = 1;
> @@ -1585,6 +1585,8 @@ static void rtllib_associate_procedure_wq(void *data)
>         struct rtllib_device *ieee = container_of_dwork_rsl(data,
>                                      struct rtllib_device,
>                                      associate_procedure_wq);
> +       struct rtllib_network *cur_net = &ieee->current_network;
> +
>         rtllib_stop_scan_syncro(ieee);
>         if (ieee->rtllib_ips_leave != NULL)
>                 ieee->rtllib_ips_leave(ieee->dev);
> @@ -1594,8 +1596,7 @@ static void rtllib_associate_procedure_wq(void *data)
>                 ieee->data_hard_stop(ieee->dev);
>
>         rtllib_stop_scan(ieee);
> -       RT_TRACE(COMP_DBG, "===>%s(), chan:%d\n", __func__,
> -                ieee->current_network.channel);
> +       RT_TRACE(COMP_DBG, "===>%s(), chan:%d\n", __func__, cur_net->channel);
>         HTSetConnectBwMode(ieee, HT_CHANNEL_WIDTH_20, HT_EXTCHNL_OFFSET_NO_EXT);
>         if (ieee->eRFPowerState == eRfOff) {
>                 RT_TRACE(COMP_DBG,
> @@ -1608,7 +1609,7 @@ static void rtllib_associate_procedure_wq(void *data)
>         }
>         ieee->associate_seq = 1;
>
> -       rtllib_associate_step1(ieee, ieee->current_network.bssid);
> +       rtllib_associate_step1(ieee, cur_net->bssid);
>
>         mutex_unlock(&ieee->wx_mutex);
>  }
> @@ -1618,6 +1619,7 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
>  {
>         u8 tmp_ssid[IW_ESSID_MAX_SIZE + 1];
>         int tmp_ssid_len = 0;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         short apset, ssidset, ssidbroad, apmatch, ssidmatch;
>
> @@ -1646,13 +1648,11 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
>                 apset = ieee->wap_set;
>                 ssidset = ieee->ssid_set;
>                 ssidbroad =  !(net->ssid_len == 0 || net->ssid[0] == '\0');
> -               apmatch = (memcmp(ieee->current_network.bssid, net->bssid,
> -                                 ETH_ALEN) == 0);
> +               apmatch = (memcmp(cur_net->bssid, net->bssid, ETH_ALEN) == 0);
>                 if (!ssidbroad) {
> -                       ssidmatch = (ieee->current_network.ssid_len ==
> -                                   net->hidden_ssid_len) &&
> -                                   (!strncmp(ieee->current_network.ssid,
> -                                   net->hidden_ssid, net->hidden_ssid_len));
> +                       ssidmatch = ((cur_net->ssid_len == net->hidden_ssid_len) &&
> +                                    (!strncmp(cur_net->ssid, net->hidden_ssid,
> +                                              net->hidden_ssid_len)));
>                         if (net->hidden_ssid_len > 0) {
>                                 strncpy(net->ssid, net->hidden_ssid,
>                                         net->hidden_ssid_len);
> @@ -1660,10 +1660,9 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
>                                 ssidbroad = 1;
>                         }
>                 } else
> -                       ssidmatch =
> -                          (ieee->current_network.ssid_len == net->ssid_len) &&
> -                          (!strncmp(ieee->current_network.ssid, net->ssid,
> -                          net->ssid_len));
> +                       ssidmatch = ((cur_net->ssid_len == net->ssid_len) &&
> +                                    (!strncmp(cur_net->ssid, net->ssid,
> +                                              net->ssid_len)));
>
>                 /* if the user set the AP check if match.
>                  * if the network does not broadcast essid we check the
> @@ -1684,26 +1683,26 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
>                          * essid provided by the user.
>                          */
>                         if (!ssidbroad) {
> -                               strncpy(tmp_ssid, ieee->current_network.ssid,
> +                               strncpy(tmp_ssid, cur_net->ssid,
>                                         IW_ESSID_MAX_SIZE);
> -                               tmp_ssid_len = ieee->current_network.ssid_len;
> +                               tmp_ssid_len = cur_net->ssid_len;
>                         }
>                         memcpy(&ieee->current_network, net,
>                                sizeof(struct rtllib_network));
>                         if (!ssidbroad) {
> -                               strncpy(ieee->current_network.ssid, tmp_ssid,
> +                               strncpy(cur_net->ssid, tmp_ssid,
>                                         IW_ESSID_MAX_SIZE);
> -                               ieee->current_network.ssid_len = tmp_ssid_len;
> +                               cur_net->ssid_len = tmp_ssid_len;
>                         }
>                         netdev_info(ieee->dev,
>                                     "Linking with %s,channel:%d, qos:%d, myHT:%d, networkHT:%d, mode:%x cur_net.flags:0x%x\n",
> -                                   ieee->current_network.ssid,
> -                                   ieee->current_network.channel,
> -                                   ieee->current_network.qos_data.supported,
> +                                   cur_net->ssid,
> +                                   cur_net->channel,
> +                                   cur_net->qos_data.supported,
>                                     ieee->pHTInfo->bEnableHT,
> -                                   ieee->current_network.bssht.bdSupportHT,
> -                                   ieee->current_network.mode,
> -                                   ieee->current_network.flags);
> +                                   cur_net->bssht.bdSupportHT,
> +                                   cur_net->mode,
> +                                   cur_net->flags);
>
>                         if ((rtllib_act_scanning(ieee, false)) &&
>                            !(ieee->softmac_features & IEEE_SOFTMAC_SCAN))
> @@ -1714,8 +1713,8 @@ inline void rtllib_softmac_new_net(struct rtllib_device *ieee,
>                         if (ieee->iw_mode == IW_MODE_INFRA) {
>                                 /* Join the network for the first time */
>                                 ieee->AsocRetryCount = 0;
> -                               if ((ieee->current_network.qos_data.supported == 1) &&
> -                                   ieee->current_network.bssht.bdSupportHT)
> +                               if (((cur_net->qos_data.supported == 1) &&
> +                                    cur_net->bssht.bdSupportHT))
>                                         HTResetSelfAndSavePeerSetting(ieee,
>                                                  &(ieee->current_network));
>                                 else
> @@ -1829,13 +1828,13 @@ static short probe_rq_parse(struct rtllib_device *ieee, struct sk_buff *skb,
>         struct rtllib_hdr_3addr   *header =
>                 (struct rtllib_hdr_3addr   *) skb->data;
>         bool bssid_match;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         if (skb->len < sizeof(struct rtllib_hdr_3addr))
>                 return -1; /* corrupted */
>
> -       bssid_match =
> -         (!ether_addr_equal(header->addr3, ieee->current_network.bssid)) &&
> -         (!is_broadcast_ether_addr(header->addr3));
> +       bssid_match = ((!ether_addr_equal(header->addr3, cur_net->bssid)) &&
> +                      (!is_broadcast_ether_addr(header->addr3)));
>         if (bssid_match)
>                 return -1;
>
> @@ -1862,7 +1861,7 @@ static short probe_rq_parse(struct rtllib_device *ieee, struct sk_buff *skb,
>         if (!ssid)
>                 return 1; /* ssid not found in tagged param */
>
> -       return !strncmp(ssid, ieee->current_network.ssid, ssidlen);
> +       return !strncmp(ssid, cur_net->ssid, ssidlen);
>  }
>
>  static int assoc_rq_parse(struct net_device *dev, struct sk_buff *skb, u8 *dest)
> @@ -1887,6 +1886,7 @@ static inline u16 assoc_parse(struct rtllib_device *ieee, struct sk_buff *skb,
>  {
>         struct rtllib_assoc_response_frame *response_head;
>         u16 status_code;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>
>         if (skb->len <  sizeof(struct rtllib_assoc_response_frame)) {
>                 netdev_dbg(ieee->dev, "Invalid len in auth resp: %d\n",
> @@ -1901,7 +1901,7 @@ static inline u16 assoc_parse(struct rtllib_device *ieee, struct sk_buff *skb,
>         if ((status_code == WLAN_STATUS_ASSOC_DENIED_RATES ||
>            status_code == WLAN_STATUS_CAPS_UNSUPPORTED) &&
>            ((ieee->mode == IEEE_G) &&
> -          (ieee->current_network.mode == IEEE_N_24G) &&
> +          (cur_net->mode == IEEE_N_24G) &&
>            (ieee->AsocRetryCount++ < (RT_ASOC_RETRY_LIMIT-1)))) {
>                 ieee->pHTInfo->IOTAction |= HT_IOT_ACT_PURE_N_MODE;
>         } else {
> @@ -1970,6 +1970,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>  {
>         int timeout;
>         u8 dtim;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         struct rt_pwr_save_ctrl *pPSC = &(ieee->PowerSaveControl);
>
>         if (ieee->LPSDelayCnt) {
> @@ -1977,11 +1978,11 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>                 return 0;
>         }
>
> -       dtim = ieee->current_network.dtim_data;
> +       dtim = cur_net->dtim_data;
>         if (!(dtim & RTLLIB_DTIM_VALID))
>                 return 0;
> -       timeout = ieee->current_network.beacon_interval;
> -       ieee->current_network.dtim_data = RTLLIB_DTIM_INVALID;
> +       timeout = cur_net->beacon_interval;
> +       cur_net->dtim_data = RTLLIB_DTIM_INVALID;
>         /* there's no need to nofity AP that I find you buffered
>          * with broadcast packet
>          */
> @@ -2009,7 +2010,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>                         if (pPSC->RegMaxLPSAwakeIntvl == 0)
>                                 MaxPeriod = 1;
>                         else if (pPSC->RegMaxLPSAwakeIntvl == 0xFF)
> -                               MaxPeriod = ieee->current_network.dtim_period;
> +                               MaxPeriod = cur_net->dtim_period;
>                         else
>                                 MaxPeriod = pPSC->RegMaxLPSAwakeIntvl;
>                         pPSC->LPSAwakeIntvl = (pPSC->LPSAwakeIntvl >=
> @@ -2018,8 +2019,8 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>                 }
>                 {
>                         u8 LPSAwakeIntvl_tmp = 0;
> -                       u8 period = ieee->current_network.dtim_period;
> -                       u8 count = ieee->current_network.tim.tim_count;
> +                       u8 period = cur_net->dtim_period;
> +                       u8 count = cur_net->tim.tim_count;
>
>                         if (count == 0) {
>                                 if (pPSC->LPSAwakeIntvl > period)
> @@ -2033,7 +2034,7 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>
>                         } else {
>                                 if (pPSC->LPSAwakeIntvl >
> -                                   ieee->current_network.tim.tim_count)
> +                                   cur_net->tim.tim_count)
>                                         LPSAwakeIntvl_tmp = count +
>                                         (pPSC->LPSAwakeIntvl - count) -
>                                         ((pPSC->LPSAwakeIntvl-count)%period);
> @@ -2041,8 +2042,8 @@ static short rtllib_sta_ps_sleep(struct rtllib_device *ieee, u64 *time)
>                                         LPSAwakeIntvl_tmp = pPSC->LPSAwakeIntvl;
>                         }
>
> -               *time = ieee->current_network.last_dtim_sta_time
> -                       + msecs_to_jiffies(ieee->current_network.beacon_interval *
> +               *time = cur_net->last_dtim_sta_time
> +                       + msecs_to_jiffies(cur_net->beacon_interval *
>                         LPSAwakeIntvl_tmp);
>         }
>         }
> @@ -2211,6 +2212,7 @@ static inline int
>  rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
>                      struct rtllib_rx_stats *rx_stats)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         u16 errcode;
>         int aid;
>         u8 *ies;
> @@ -2238,7 +2240,7 @@ rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
>                         /* station support qos */
>                         /* Let the register setting default with Legacy station */
>                         assoc_resp = (struct rtllib_assoc_response_frame *)skb->data;
> -                       if (ieee->current_network.qos_data.supported == 1) {
> +                       if (cur_net->qos_data.supported == 1) {
>                                 if (rtllib_parse_info_param(ieee, assoc_resp->info_element,
>                                                         rx_stats->len - sizeof(*assoc_resp),
>                                                         network, rx_stats)) {
> @@ -2292,6 +2294,7 @@ rtllib_rx_assoc_resp(struct rtllib_device *ieee, struct sk_buff *skb,
>
>  static void rtllib_rx_auth_resp(struct rtllib_device *ieee, struct sk_buff *skb)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         u16 errcode;
>         u8 *challenge;
>         int chlen = 0;
> @@ -2324,15 +2327,13 @@ static void rtllib_rx_auth_resp(struct rtllib_device *ieee, struct sk_buff *skb)
>                 }
>                 /* Dummy wirless mode setting to avoid encryption issue */
>                 if (bSupportNmode) {
> -                       ieee->SetWirelessMode(ieee->dev,
> -                                             ieee->current_network.mode);
> +                       ieee->SetWirelessMode(ieee->dev, cur_net->mode);
>                 } else {
>                         /*TODO*/
>                         ieee->SetWirelessMode(ieee->dev, IEEE_G);
>                 }
>
> -               if ((ieee->current_network.mode == IEEE_N_24G) &&
> -                   bHalfSupportNmode) {
> +               if ((cur_net->mode == IEEE_N_24G) && bHalfSupportNmode) {
>                         netdev_info(ieee->dev, "======>enter half N mode\n");
>                         ieee->bHalfWirelessN24GMode = true;
>                 } else {
> @@ -2366,9 +2367,10 @@ static inline int
>  rtllib_rx_deauth(struct rtllib_device *ieee, struct sk_buff *skb)
>  {
>         struct rtllib_hdr_3addr *header = (struct rtllib_hdr_3addr *) skb->data;
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         u16 frame_ctl;
>
> -       if (memcmp(header->addr3, ieee->current_network.bssid, ETH_ALEN) != 0)
> +       if (memcmp(header->addr3, cur_net->bssid, ETH_ALEN) != 0)
>                 return 0;
>
>         /* FIXME for now repeat all the association procedure
> @@ -2542,21 +2544,20 @@ void rtllib_wake_all_queues(struct rtllib_device *ieee)
>  /* called in user context only */
>  static void rtllib_start_master_bss(struct rtllib_device *ieee)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         ieee->assoc_id = 1;
>
> -       if (ieee->current_network.ssid_len == 0) {
> -               strncpy(ieee->current_network.ssid,
> -                       RTLLIB_DEFAULT_TX_ESSID,
> +       if (cur_net->ssid_len == 0) {
> +               strncpy(cur_net->ssid, RTLLIB_DEFAULT_TX_ESSID,
>                         IW_ESSID_MAX_SIZE);
>
> -               ieee->current_network.ssid_len =
> -                                strlen(RTLLIB_DEFAULT_TX_ESSID);
> +               cur_net->ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
>                 ieee->ssid_set = 1;
>         }
>
> -       ether_addr_copy(ieee->current_network.bssid, ieee->dev->dev_addr);
> +       ether_addr_copy(cur_net->bssid, ieee->dev->dev_addr);
>
> -       ieee->set_chan(ieee->dev, ieee->current_network.channel);
> +       ieee->set_chan(ieee->dev, cur_net->channel);
>         ieee->state = RTLLIB_LINKED;
>         ieee->link_change(ieee->dev);
>         notify_wx_assoc_event(ieee);
> @@ -2582,6 +2583,8 @@ static void rtllib_start_ibss_wq(void *data)
>  {
>         struct rtllib_device *ieee = container_of_dwork_rsl(data,
>                                      struct rtllib_device, start_ibss_wq);
> +       struct rtllib_network *cur_net = &ieee->current_network;
> +
>         /* iwconfig mode ad-hoc will schedule this and return
>          * on the other hand this will block further iwconfig SET
>          * operations because of the wx_mutex hold.
> @@ -2595,9 +2598,9 @@ static void rtllib_start_ibss_wq(void *data)
>         }
>         mutex_lock(&ieee->wx_mutex);
>
> -       if (ieee->current_network.ssid_len == 0) {
> -               strcpy(ieee->current_network.ssid, RTLLIB_DEFAULT_TX_ESSID);
> -               ieee->current_network.ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
> +       if (cur_net->ssid_len == 0) {
> +               strcpy(cur_net->ssid, RTLLIB_DEFAULT_TX_ESSID);
> +               cur_net->ssid_len = strlen(RTLLIB_DEFAULT_TX_ESSID);
>                 ieee->ssid_set = 1;
>         }
>
> @@ -2627,57 +2630,49 @@ static void rtllib_start_ibss_wq(void *data)
>         /* the network definitively is not here.. create a new cell */
>         if (ieee->state == RTLLIB_NOLINK) {
>                 netdev_info(ieee->dev, "creating new IBSS cell\n");
> -               ieee->current_network.channel = ieee->IbssStartChnl;
> +               cur_net->channel = ieee->IbssStartChnl;
>                 if (!ieee->wap_set)
> -                       eth_random_addr(ieee->current_network.bssid);
> +                       eth_random_addr(cur_net->bssid);
>
>                 if (ieee->modulation & RTLLIB_CCK_MODULATION) {
>
> -                       ieee->current_network.rates_len = 4;
> +                       cur_net->rates_len = 4;
>
> -                       ieee->current_network.rates[0] =
> +                       cur_net->rates[0] =
>                                  RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_1MB;
> -                       ieee->current_network.rates[1] =
> +                       cur_net->rates[1] =
>                                  RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_2MB;
> -                       ieee->current_network.rates[2] =
> +                       cur_net->rates[2] =
>                                  RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_5MB;
> -                       ieee->current_network.rates[3] =
> +                       cur_net->rates[3] =
>                                  RTLLIB_BASIC_RATE_MASK | RTLLIB_CCK_RATE_11MB;
>
>                 } else
> -                       ieee->current_network.rates_len = 0;
> +                       cur_net->rates_len = 0;
>
>                 if (ieee->modulation & RTLLIB_OFDM_MODULATION) {
> -                       ieee->current_network.rates_ex_len = 8;
> -
> -                       ieee->current_network.rates_ex[0] =
> -                                                RTLLIB_OFDM_RATE_6MB;
> -                       ieee->current_network.rates_ex[1] =
> -                                                RTLLIB_OFDM_RATE_9MB;
> -                       ieee->current_network.rates_ex[2] =
> -                                                RTLLIB_OFDM_RATE_12MB;
> -                       ieee->current_network.rates_ex[3] =
> -                                                RTLLIB_OFDM_RATE_18MB;
> -                       ieee->current_network.rates_ex[4] =
> -                                                RTLLIB_OFDM_RATE_24MB;
> -                       ieee->current_network.rates_ex[5] =
> -                                                RTLLIB_OFDM_RATE_36MB;
> -                       ieee->current_network.rates_ex[6] =
> -                                                RTLLIB_OFDM_RATE_48MB;
> -                       ieee->current_network.rates_ex[7] =
> -                                                RTLLIB_OFDM_RATE_54MB;
> +                       cur_net->rates_ex_len = 8;
> +
> +                       cur_net->rates_ex[0] = RTLLIB_OFDM_RATE_6MB;
> +                       cur_net->rates_ex[1] = RTLLIB_OFDM_RATE_9MB;
> +                       cur_net->rates_ex[2] = RTLLIB_OFDM_RATE_12MB;
> +                       cur_net->rates_ex[3] = RTLLIB_OFDM_RATE_18MB;
> +                       cur_net->rates_ex[4] = RTLLIB_OFDM_RATE_24MB;
> +                       cur_net->rates_ex[5] = RTLLIB_OFDM_RATE_36MB;
> +                       cur_net->rates_ex[6] = RTLLIB_OFDM_RATE_48MB;
> +                       cur_net->rates_ex[7] = RTLLIB_OFDM_RATE_54MB;
>
>                         ieee->rate = 108;
>                 } else {
> -                       ieee->current_network.rates_ex_len = 0;
> +                       cur_net->rates_ex_len = 0;
>                         ieee->rate = 22;
>                 }
>
> -               ieee->current_network.qos_data.supported = 0;
> +               cur_net->qos_data.supported = 0;
>                 ieee->SetWirelessMode(ieee->dev, IEEE_G);
> -               ieee->current_network.mode = ieee->mode;
> -               ieee->current_network.atim_window = 0;
> -               ieee->current_network.capability = WLAN_CAPABILITY_IBSS;
> +               cur_net->mode = ieee->mode;
> +               cur_net->atim_window = 0;
> +               cur_net->capability = WLAN_CAPABILITY_IBSS;
>         }
>
>         netdev_info(ieee->dev, "%s(): ieee->mode = %d\n", __func__, ieee->mode);
> @@ -2913,6 +2908,7 @@ EXPORT_SYMBOL(rtllib_softmac_start_protocol);
>
>  void rtllib_start_protocol(struct rtllib_device *ieee)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         short ch = 0;
>         int i = 0;
>
> @@ -2923,17 +2919,17 @@ void rtllib_start_protocol(struct rtllib_device *ieee)
>
>         ieee->proto_started = 1;
>
> -       if (ieee->current_network.channel == 0) {
> +       if (cur_net->channel == 0) {
>                 do {
>                         ch++;
>                         if (ch > MAX_CHANNEL_NUMBER)
>                                 return; /* no channel found */
>                 } while (!ieee->active_channel_map[ch]);
> -               ieee->current_network.channel = ch;
> +               cur_net->channel = ch;
>         }
>
> -       if (ieee->current_network.beacon_interval == 0)
> -               ieee->current_network.beacon_interval = 100;
> +       if (cur_net->beacon_interval == 0)
> +               cur_net->beacon_interval = 100;
>
>         for (i = 0; i < 17; i++) {
>                 ieee->last_rxseq_num[i] = -1;
> @@ -3136,6 +3132,7 @@ void SendDisassociation(struct rtllib_device *ieee, bool deauth, u16 asRsn)
>
>  u8 rtllib_ap_sec_type(struct rtllib_device *ieee)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         static u8 ccmp_ie[4] = {0x00, 0x50, 0xf2, 0x04};
>         static u8 ccmp_rsn_ie[4] = {0x00, 0x0f, 0xac, 0x04};
>         int wpa_ie_len = ieee->wpa_ie_len;
> @@ -3143,7 +3140,7 @@ u8 rtllib_ap_sec_type(struct rtllib_device *ieee)
>         int encrypt;
>
>         crypt = ieee->crypt_info.crypt[ieee->crypt_info.tx_keyidx];
> -       encrypt = (ieee->current_network.capability & WLAN_CAPABILITY_PRIVACY)
> +       encrypt = (cur_net->capability & WLAN_CAPABILITY_PRIVACY)
>                   || (ieee->host_encrypt && crypt && crypt->ops &&
>                   (strcmp(crypt->ops->name, "R-WEP") == 0));
>
> @@ -3250,6 +3247,7 @@ EXPORT_SYMBOL(rtllib_MgntDisconnect);
>
>  void notify_wx_assoc_event(struct rtllib_device *ieee)
>  {
> +       struct rtllib_network *cur_net = &ieee->current_network;
>         union iwreq_data wrqu;
>
>         if (ieee->cannot_notify)
> @@ -3257,8 +3255,7 @@ void notify_wx_assoc_event(struct rtllib_device *ieee)
>
>         wrqu.ap_addr.sa_family = ARPHRD_ETHER;
>         if (ieee->state == RTLLIB_LINKED)
> -               memcpy(wrqu.ap_addr.sa_data, ieee->current_network.bssid,
> -                      ETH_ALEN);
> +               memcpy(wrqu.ap_addr.sa_data, cur_net->bssid, ETH_ALEN);
>         else {
>
>                 netdev_info(ieee->dev, "%s(): Tell user space disconnected\n",
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-06-16  4:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-09  6:07 [PATCH 2/2] staging: rtl8192e: fix line over 80 characters thtlwlsmsgkfla
2018-06-09  6:45 ` Joe Perches
2018-06-16  4:20   ` 김현일

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).