On Wed, Jul 15, 2009 at 09:05:06AM -0700, Johannes Berg wrote: > On Wed, 2009-07-15 at 08:14 -0700, Luis R. Rodriguez wrote: > > > > + if (!(wiphy->wow_triggers_supported & triggers_requested)) { > > > > + err = -EOPNOTSUPP; > > > > + goto out; > > > > > > This seems wrong, should be "requested & supported != requested", I > > > think. > > > > The above one won't return -EOPNOTSUPP when you add additional junk, you're > > right, better to just inform userspace of that. > > Your version also disallows turning off WoW again, afaict. That is something we do want, as otherwise we'll keep the device powered on. Will consider both. 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