On 10 May 2018 15:50:23 CEST, Dedy Lansky <dlansky@xxxxxxxxxxxxxx> wrote: >Hi Toke, > >> --- a/drivers/net/wireless/ath/wil6210/wmi.c >> +++ b/drivers/net/wireless/ath/wil6210/wmi.c >> @@ -824,7 +824,7 @@ static void wmi_evt_connect(struct wil6210_vif >*vif, int id, void *d, int len) >> struct wireless_dev *wdev = vif_to_wdev(vif); >> struct wmi_connect_event *evt = d; >> int ch; /* channel number */ >> - struct station_info sinfo; >> + struct station_info *sinfo; >> u8 *assoc_req_ie, *assoc_resp_ie; >> size_t assoc_req_ielen, assoc_resp_ielen; >> /* capinfo(u16) + listen_interval(u16) + IEs */ @@ -940,6 +940,7 @@ >static void wmi_evt_connect(struct wil6210_vif *vif, int id, void *d, >int len) >> vif->bss = NULL; >> } else if ((wdev->iftype == NL80211_IFTYPE_AP) || >> (wdev->iftype == NL80211_IFTYPE_P2P_GO)) { >> + > >Any reason for this extra empty line? Heh, no. Just an artifact of editing patches over a painfully bad connection on a moving train... -Toke