Re: [PATCH v5] brcmfmac cleanup

On 22-08-17 13:25, Ian Molton wrote:

Hi folks,

Arend, as requested - a respin to take account of your comments.

Unfortunately, although I've only included the patches you requested from v4,
breaking out some of the simpler changes (whitespace, macos, etc.) has grown
the set back up to 30 patches. I hope this is OK.

:-( Grumbl, you also introduced some new ideas, eg. patch 26/30. I 
reviewed 15 patches that mostly involved in cleaning up bcmsdh.c. My 
hopes were that you addressed those 15 patches with a respin to get that 
part at least in for 4.14 as we are steadily moving towards the merge 
window. Maybe you did, but going from 15 to 30 feels like things are 
moving in the wrong direction. I will dig through it again.

Regards,
Arend