Hi Kalle,

Thanks for the review, I have refactor code to avoid ugly indent in v2.

Regards,
Simon
________________________________________
From: Kalle Valo <redacted>
Sent: Friday, July 28, 2017 22:39
To: Xinming Hu
Cc: Linux Wireless; Brian Norris; Dmitry Torokhov; rajatja@google.com; Zhiyuan Yang; Tim Song; Cathy Luo; Xinming Hu
Subject: Re: [PATCH] mwifiex: correct IE parse during association

Xinming Hu [off-list ref] writes:

From: Xinming Hu <redacted>

It is observed that some IEs get missed during association.
This patch correct the old IE parse code. sme->ie will be
store as wpa ie, wps ie, wapi ie and gen ie accordingly.

Signed-off-by: Xinming Hu <redacted>
Signed-off-by: Cathy Luo <redacted>

[...]

+             } else {
+                     if (pvendor_ie->element_id ==
+                         WLAN_EID_VENDOR_SPECIFIC) {
+                             /* Test to see if it is a WPA IE,
+                              * if not, then it is a gen IE
+                              */
+                             if (!memcmp(pvendor_ie->oui, wpa_oui,
+                                         sizeof(wpa_oui))) {
+                                     /* IE is a WPA/WPA2 IE so call
+                                      * set_wpa function
+                                      */
+                                     mwifiex_set_wpa_ie(priv,
+                                                        (u8 *)pvendor_ie,
+                                                        cur_ie_len);

The indentation is getting ugly here. A helper function or similar would
really make this more readable.


[...]

+NEXT_IE:

Goto labels in lower case, please.

--
Kalle Valo