On 12-04-16 21:58, Johannes Berg wrote: > On Tue, 2016-04-12 at 21:53 +0200, Arend van Spriel wrote: >> >> On 12-04-16 15:58, Johannes Berg wrote: >>> >>> From: Johannes Berg <johannes.berg@xxxxxxxxx> >>> >>> This enum is already perfectly aliased to enum nl80211_band, and >>> the only reason for it is that we get IEEE80211_NUM_BANDS out of >>> it. There's no really good reason to not declare the number of >>> bands in nl80211 though, so do that and remove the cfg80211 one. >> So what if a band is added. That would change the value of >> IEEE80211_NUM_BANDS in nl80211. Is it not ABI? > > Well, it can't be ABI really since the value is never used. Arguably it > becomes API, but clearly we have the same with NL80211_CMD_MAX and many > others like it, so I see no point in treating the bands here specially. > I even added a comment :) Ah. Clearly I only read the commit message :-p Sorry for the noise. Gr. AvS > johannes > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
