See inline On Tue, May 10, 2011 at 12:56 PM, <greearb@xxxxxxxxxxxxxxx> wrote: > > + > + /* > + printk("start_sw_scan, can_scan_one: %i n_channels: %i\n", > + local->scan_req->can_scan_one, local->scan_req->n_channels); > + WARN_ON(!local->scan_req->can_scan_one); > + */ Remove the debug print? > + > + /* If one sta is associated, we don't want another to start > + * scanning on all channels, as that will interfere with the > + * one already associated. > + */ Comment style is wrong. > > + /* Special hack: channel -1 means 'scan only active > + * channel if any VIFs on this device are associated > + * on the channel. > + */ Same. > > + /* If at least one VIF on this hardware is already associated, then > + * only scan on the active channel. > + */ Same. Dan -- 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