Kalle Valo <[email protected]> writes: > Toke Høiland-Jørgensen <[email protected]> writes: > >> Kalle Valo <[email protected]> writes: >> >>> Toke Høiland-Jørgensen <[email protected]> writes: >>> >>>> Kalle Valo <[email protected]> writes: >>>> >>>>> Johannes Berg <[email protected]> writes: >>>>> >>>>>> On Wed, 2018-05-09 at 11:36 +0200, Toke Høiland-Jørgensen wrote: >>>>>>> Johannes Berg <[email protected]> writes: >>>>>>> >>>>>>> > On Wed, 2018-05-09 at 11:56 +0300, Kalle Valo wrote: >>>>>>> > > Johannes Berg <[email protected]> writes: >>>>>>> > > >>>>>>> > > > On Wed, 2018-05-09 at 11:47 +0300, Kalle Valo wrote: >>>>>>> > > > > >>>>>>> > > > > I guess these warnings come because Toke's patch increased size >>>>>>> > > > > of >>>>>>> > > > > struct cfg80211_tid_stats (which is included in struct >>>>>>> > > > > station_info) and >>>>>>> > > > > both wil6210 and qtnfmac allocate a struct station_info from >>>>>>> > > > > stack? >>>>>>> > > > >>>>>>> > > > Yes. >>>>>>> > > > >>>>>>> > > > > Can >>>>>>> > > > > someone send a fix for the drivers? >>>>>>> > > > >>>>>>> > > > I guess Toke/I should do that through my tree. >>>>>>> > > >>>>>>> > > IMHO the fix could go through my tree as well, less risk of >>>>>>> > > conflicts in >>>>>>> > > drivers. AFAICS the fix (allocating station_info dynamically?) >>>>>>> > > would not >>>>>>> > > depend on Toke's patch and could be applied separately. >>>>>>> > >>>>>>> > That's true, if you prefer that it's fine with me. >>>>>>> >>>>>>> I'll send a patch. >>>>>>> >>>>>>> What's the right tag to put in the commit for this? >>>>>>> Fixes-but-is-independent-from: ? ;) >>>>>> >>>>>> Heh. You can still put Fixes: I think. >>>>> >>>>> Yeah, I think so too. >>>> >>>> Cool. My "git grep 'struct station_info sinfo'" also shows up a driver >>>> in staging; that should be fixed as well, right? In the same commit? >>> >>> Not in the same commit at least, I don't want to touch staging even with >>> a ten foot pole :) I guess either Greg or Johannes would take that >>> patch. >> >> What about batman-adv and wext-compat? Should I split those out as well? > > So normally I only take patches touching drivers/net/wireless, anything > else has to go via other trees.
Right, thought so; will send a series as soon as I've verified that it compiles :) -Toke
