> + * @timeout_reason: reason for connection timeout. This is used when > the > + * connection fails due to a timeout instead of an explicit > rejection from > + * the AP. 0 (NL80211_CONNECT_TIMEOUT_UNSPECIFIED) is used > for other cases. I think this description is misleading - one could easily understand "for other cases" to indicate for the cases that the AP did explicitly reject it, but that's obviously not true. Perhaps that could be reworded, to say it's used when it's not known, or such? I'd not indicate the value (0) either, just specify the name, and put a % in front to get better formatting for it please. > + resp_ie_len, status, gfp, > + NL80211_TIMEOUT_UNSPECIFIED); > } NL80211_CONNECT_TIMEOUT_UNSPECIFIED in the comment is wrong then. johannes
