On Thu, 2018-06-14 at 13:11 +0300, Omer Efrat wrote: > Since 'filled' member in station_info changed to u64, BIT_ULL macro > should be used with NL80211_STA_INFO_* attribute types instead of BIT. > > The BIT macro uses unsigned long type which some architectures handle as 32bit > and this results in compilation warnings such as: > > net/mac80211/sta_info.c:2223:2: warning: left shift count >= width of type > sinfo->filled |= BIT(NL80211_STA_INFO_TID_STATS); > ^
It seems like the only change needed is with BIT(NL80211_STA_INFO_TID_STATS), so I'd argue you should restrict the patch to that. Perhaps, though I'm not sure I see it, there's some value in switching them all so that if you copy something and change it to a new value you don't run into this problem again, but if anything that should be (a) separate patch(es) since this one is a bugfix and the others aren't. johannes
