On Tue, Aug 30, 2011 at 11:17:43AM +0530, Raja Mani wrote: > On Monday 29 August 2011 05:53 PM, Jouni Malinen wrote: > >+static int ath6kl_ap_beacon(struct wiphy *wiphy, struct net_device *dev, > >+ struct beacon_parameters *info, bool add) > >+{ > >+ if (!test_bit(WMI_READY,&ar->flag)) > >+ return -EIO; > >+ > >+ if (!test_bit(WLAN_ENABLED,&ar->flag)) > >+ return -EIO; > > How about calling ath6kl_cfg80211_ready() function to check > WMI_READY and WLAN_ENABLED status ? Sure, I can do that, but unless something else comes up, I would rather do it after the patch series goes in. It looks like I missed addition of that helper function since I started working on this before the cleanup was complete. By the way.. How did that message lost the space after the commas on those test_bit lines? They are included in the patch I sent. -- Jouni Malinen PGP id EFC895FA -- 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