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