On Tue, Mar 6, 2018 at 1:46 PM, Greg Kroah-Hartman
[off-list ref] wrote:

On Tue, Mar 06, 2018 at 12:48:29PM +0200, Andy Shevchenko wrote:

quoted

On Mon, Mar 5, 2018 at 9:08 PM, Logan Gunthorpe [off-list ref] wrote:

quoted

Clean up the ifdefs which conditionally defined the io{read|write}64
functions in favour of the new common io-64-nonatomic-lo-hi header.

quoted

 #include <linux/kthread.h>
 #include <linux/interrupt.h>
 #include <linux/ntb.h>
+#include <linux/io-64-nonatomic-lo-hi.h>

A nit, can we preserve alphabetical ordering?

Ugh, why, it doesn't matter at all :)

I'm fine with either way, but ordering is a good thing to have in long
term prospective.
Easy to find what headers are in there, easy to understand if you need
another one or not, easy to maintain the list w/o duplications.

I saw already few examples where long list of headers contained
duplications b/c it was in messy order.

-- 
With Best Regards,
Andy Shevchenko