On Tue, 2009-12-15 at 10:11 -0800, Luis R. Rodriguez wrote:

> > +       if (changed & WIPHY_PARAM_COVERAGE_CLASS) {
> > +               if (!local->ops->set_coverage_class)
> > +                       return -EOPNOTSUPP;
> 
> Hm, it seems best to just add the capability bit that way userspace
> can stuff what it wishes and the kernel will only set what is supported.
> As is now this would lead to -EOPNOTSUPP but we'd have no way of knowing
> from userspace what failed.

I think if it cares, userspace should just be careful to only change one
thing at a time.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Reply via email to