On Tue, 2017-05-23 at 16:48 -0700, Bart Van Assche wrote: > Hello Nic, > > This patch series consists mostly of bug fixes and code cleanup. There > are some new patches in this series but many patches in this series have > been posted before. Please consider these patches for kernel v4.13. > Starting to dig into this. However, again, this patch series intermixes improvements, new features and bug-fixes with no specific context. This is probably the 7th time I've requested to put bug-fixes first followed by other improvements, but it still falls on deaf ears.. In case the last times didn't sink in, putting bug-fixes first is important because: - To know which bug-fixes are for existing code, instead of bug-fixes that are a consequence of other changes. - To know bug-fixes for existing code have been tested + verified separate from any other changes. For the former, every series I've gotten from you in the last seven months has had at least one patch that is a bug-fix for some issue the series itself introduced, and it's never been obvious. Everybody else manages to follow this protocol, so it would be nice if you could do the same, and it would certainly get your patches merged faster. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html