On Thu, May 5, 2011 at 4:00 PM, Luciano Coelho <coelho@xxxxxx> wrote: > Implement new functionality for scheduled scan offload. With this feature we > can scan automatically at certain intervals. > > The idea is that the hardware can perform scan automatically and filter on > desired results without waking up the host unnecessarily. > > Add NL80211_CMD_START_SCHED_SCAN and NL80211_CMD_STOP_SCHED_SCAN > commands to the nl80211 interface. When results are available they are > reported by NL80211_CMD_SCHED_SCAN_RESULTS events. The userspace is > informed when the scheduled scan has stopped with a > NL80211_CMD_SCHED_SCAN_STOPPED event, which can be triggered either by > the driver or by a call to NL80211_CMD_STOP_SCHED_SCAN. > > Signed-off-by: Luciano Coelho <coelho@xxxxxx> > --- [...] > + > + err = rdev->ops->sched_scan_start(&rdev->wiphy, dev, request); > + if (!err) > + nl80211_send_sched_scan(rdev, dev, > + NL80211_CMD_START_SCHED_SCAN); > + else { > +out_free: > + kfree(request); > + rdev->sched_scan_req = NULL; > + } > + > + return err; > +} i know it's the same in nl80211_trigger_scan(), but it still looks very odd - goto into else block?! Eliad. -- 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