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
