Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Kalle Valo
Ian Molton writes: > On 27/07/17 07:17, Kalle Valo wrote: >>> Here's a v4 of the cleanup patchset - checkpatch clean(er - I have purposely >>> left some warnings unaddressed). >>> >>> I also dropped an accidental adjustment of a debug macro from v3. >> Like I said already

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Ian Molton
On 27/07/17 11:09, Arend van Spriel wrote: >> >> Hi Arend, >> >> Noted for future series. I take it I shouldn't re-spin if you're going >> through them already? > > My "guess" is a v5 is likely ;-) No prob. I've spotted a gem in it myself (FIXME comment that crept through). I'll split it into

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Arend van Spriel
On 7/27/2017 12:00 PM, Ian Molton wrote: On 27/07/17 10:52, Arend van Spriel wrote: How many patches would you like at once? Hi Ian, ~10-15 patches should be fine although I am simply ploughing through the lot right now. Hi Arend, Noted for future series. I take it I shouldn't re-spin if

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Ian Molton
On 27/07/17 10:52, Arend van Spriel wrote: >> >> How many patches would you like at once? > > Hi Ian, > > ~10-15 patches should be fine although I am simply ploughing through the > lot right now. Hi Arend, Noted for future series. I take it I shouldn't re-spin if you're going through them

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Arend van Spriel
On 7/27/2017 11:47 AM, Ian Molton wrote: On 27/07/17 07:17, Kalle Valo wrote: Here's a v4 of the cleanup patchset - checkpatch clean(er - I have purposely left some warnings unaddressed). I also dropped an accidental adjustment of a debug macro from v3. Like I said already last time, please

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Ian Molton
On 27/07/17 07:17, Kalle Valo wrote: >> Here's a v4 of the cleanup patchset - checkpatch clean(er - I have purposely >> left some warnings unaddressed). >> >> I also dropped an accidental adjustment of a debug macro from v3. > Like I said already last time, please split the patchset into

Re: Patch v4: Clean up brcmfmac driver

2017-07-27 Thread Kalle Valo
Ian Molton writes: > Here's a v4 of the cleanup patchset - checkpatch clean(er - I have purposely > left some warnings unaddressed). > > I also dropped an accidental adjustment of a debug macro from v3. Like I said already last time, please split the patchset into