On Tue, 2019-02-05 at 15:36 +0530, Tamizh chelvam wrote:
>
> + [NL80211_ATTR_CQM] = {
> + .type = NLA_NESTED,
> + .validation_data = nl80211_attr_cqm_policy,
> + },
This is wrong, you need to use NLA_POLICY_NESTED().
> + [NL80211_ATTR_STA_MON_FIXED_THOLD] = { .type = NLA_U8 },
Please use NLA_POLICY_RANGE(NLA_U8, 0, 1).
Also, I'm not sure I really understand the documentation for this value.
> /* Check all values negative and sorted */
> - for (i = 0; i < n_thresholds; i++) {
> + for (i = 0; i < n_tholds; i++) {
Why rename the perfectly fine variable to some abbreviation?
johannes
