On Fri, 2019-03-22 at 15:29 -0700, thomas wrote: > > I guess it's a tradeoff between leaving MHz-only drivers alone and > wrapping existing assignments and comparisons in maybe a macro to > extract the final KHz value. The current set of changes is about 150 > lines in net/mac80211/ and net/wireless/ [1] with drivers/ being about > the same. > > Adding the KHz part to center_freq, center_freq1, and rx_status->freq > may be fewer overall changes in mac80211 and cfg80211, and should be > none for existing drivers.
Yes. I was also thinking that perhaps keeping "MHz-mostly" would lead in less overall new binary size, since you don't need to do "/1000" in that many places. I can't see it any more, but IIRC your patch did add a number of "KHZ_TO_MHZ()" or so? johannes
