On Tue, 2019-06-18 at 10:57 +0530, Tamizh chelvam wrote: > > /** > + * enum nl80211_tid_config - TID config state > + * @NL80211_TID_CONFIG_DEFAULT: Default config for the TID > + * @NL80211_TID_CONFIG_ENABLE: Enable config for the TID > + * NL80211_TID_CONFIG_DISABLE: Disable config for the TID > + */ > +enum nl80211_tid_config { > + NL80211_TID_CONFIG_DEFAULT, > + NL80211_TID_CONFIG_ENABLE, > + NL80211_TID_CONFIG_DISABLE, > +}; Hmm. Looking at this in more detail in patch 3, I don't understand. How is DEFAULT different from "attribute not present", i.e. "no changes"? johannes