> + 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? johannes
