That inverts the meaning of the condition though. On 03/09/2014 10:42 PM, Joe Perches wrote: ... > - if (bss == NULL) { ... > - } else > + if (!bss) { if (bss) { > ath6kl_dbg(ATH6KL_DBG_WLAN_CFG, "cfg80211 already has a bss\n"); > + return NULL; > + } ... -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html