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 last time, please split the patchset into reasonable
>> sizes (and submit one patchset at a time). I will automatically drop
>> patchbombs like this.
>
> How many patches would you like at once?

Check my previous email:

https://lkml.kernel.org/r/87k23124gp@purkki.adurom.net

-- 
Kalle Valo

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 two series at v5 then.

-Ian

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 you're going
through them already?


My "guess" is a v5 is likely ;-)

Regards,
Arend

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 already?

Thanks,

-Ian

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 split the patchset into reasonable
sizes (and submit one patchset at a time). I will automatically drop
patchbombs like this.


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.


Regards,
Arend

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 reasonable
> sizes (and submit one patchset at a time). I will automatically drop
> patchbombs like this.

How many patches would you like at once?

-Ian

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 reasonable
sizes (and submit one patchset at a time). I will automatically drop
patchbombs like this.

-- 
Kalle Valo