Hi Johannes,

>> ... [snip]
> 
> Please trim quotes a bit.
> 
>>>     NL80211_SURVEY_INFO_PAD,
>>> +   NL80211_SURVEY_INFO_TIME_BSS_RX,
>> 
>> wouldn’t this go before the PAD attribute?
> 
> No, as usual, that would break ABI. PAD is a regular attribute, just
> empty and ignored for aligning 64-bit values.

then I do not grok on how the nla_put_u64_64bit works, but that is fine.

I assumed these are similar to the NL80211_SURVEY_INFO_MAX which we also always 
move, but also not expected to be part of the API as a fixed value.

Regards

Marcel

Reply via email to