On 03/29/2017 01:42 AM, Johannes Berg wrote:
+ if (data->center_freq2)
+ if (nla_put_u32(skb, HWSIM_ATTR_CENTER_FREQ2, data-
center_freq2))
+ goto nla_put_failure;
That could be simplified (but isn't really interesting)
+ * @HWSIM_ATTR_CENTER_FREQ2: Configured center-freq-2. Not reported
if value is zero.
The value 0 really just means "unused" - so this should say "if used"
or so.
+ * @HWSIM_ATTR_CH_WIDTH: Configured channel width.
That should point to &enum nl80211_chan_width I guess.
Anyway, all of that is minor. The biggest problem I have with this
patch upon close review is that it only handles one case, and actually
ties that case to the nl80211 API.
hwsim also can deal with channel contexts already, is there much point
in making the userspace API ignorant of that?
The patch appeared to solve my problem, and it seems an improvement
over what is there currently. Whoever wants it to support even more things
can add that in the future?
If you remember, the first iteration of my patch had the NL API defines
more general, so that someone could just add more data members
as the needs grew. It can still grow, however.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
