On Thu, Feb 19, 2009 at 5:29 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Wed, 2009-02-18 at 18:25 -0500, Luis R. Rodriguez wrote: >> This series has a few cleanups and minor fixes but mainly >> adds a workqueue for cfg80211 to use to process all regulatory >> hints. This v4 just changes the spin_lock to only be held _only_ during >> dequeing (not like this matters much). >> >> This series is also rebased on today's pull of wl. Using a new >> SMTP server now, so hopefully this goes through fine in one shot. > > Except for the unlikely(WARN_ON()) which still looks weird, this seems > ok to me. Great thanks, I'll post a new series without the unlikely on the WARN_ON() but also I will change around the spin_lock() so it doesn't happen before the cfg80211_mutex, that's just not needed as you had hinted before. > I don't know why you insist on having a wrapper for > wiphy->wiphy_idx but it's ok :) Heh its just because wiphy->wiphy_idx does not exist, the drv has so you must go to the container and then use that. If we move the idx to the wiphy then I'm good to use wiphy->wiphy_idx. Want to do that? Luis -- 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
