From: Logan Gunthorpe <logang@deltatee.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Linux-Arch <linux-arch@vger.kernel.org>,
linux-ntb@googlegroups.com,
linux-crypto <linux-crypto@vger.kernel.org>,
"Arnd Bergmann" <arnd@arndb.de>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Horia Geantă" <horia.geanta@nxp.com>,
"Stephen Bates" <sbates@raithlin.com>
Subject: Re: [PATCH v7 6/7] ntb: ntb_hw_intel: use io-64-nonatomic instead of in-driver hacks
Date: Tue, 22 Aug 2017 11:56:34 -0600 [thread overview]
Message-ID: <b4b38bb0-b97f-2240-0ddc-ef5444e289ef@deltatee.com> (raw)
In-Reply-To: <CAHp75VfPXjw-gTX=xQsk_0vUhyeTEYiTvoacTzw43RgyGKKaVg@mail.gmail.com>
On 22/08/17 11:41 AM, Andy Shevchenko wrote:
>> @@ -59,6 +59,7 @@
>> #include <linux/random.h>
>> #include <linux/slab.h>
>> #include <linux/ntb.h>
>> +#include <linux/io-64-nonatomic-lo-hi.h>
>
> I would rather try to squeeze to most alpabetiacelly ordered part of this block.
> Otherwise it's a pretty nice clean up!
>
I don't understand. The includes were not alphabetically in any order
originally and I'd rather not reorder them in my patch.
Logan
next prev parent reply other threads:[~2017-08-22 17:56 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-22 17:02 [PATCH v7 0/7] make io{read|write}64 globally usable Logan Gunthorpe
2017-08-22 17:02 ` [PATCH v7 1/7] drm/tilcdc: ensure nonatomic iowrite64 is not used Logan Gunthorpe
2017-08-22 17:02 ` [PATCH v7 2/7] powerpc: io.h: move iomap.h include so that it can use readq/writeq defs Logan Gunthorpe
2017-08-22 17:02 ` [PATCH v7 3/7] powerpc: iomap.c: introduce io{read|write}64_{lo_hi|hi_lo} Logan Gunthorpe
2017-08-22 17:02 ` [PATCH v7 4/7] iomap: " Logan Gunthorpe
2017-08-22 17:02 ` [PATCH v7 5/7] io-64-nonatomic: add io{read|write}64[be]{_lo_hi|_hi_lo} macros Logan Gunthorpe
2017-08-22 17:40 ` Andy Shevchenko
2017-08-22 17:55 ` Logan Gunthorpe
2017-08-22 21:07 ` Andy Shevchenko
2017-08-22 17:02 ` [PATCH v7 6/7] ntb: ntb_hw_intel: use io-64-nonatomic instead of in-driver hacks Logan Gunthorpe
2017-08-22 17:41 ` Andy Shevchenko
2017-08-22 17:56 ` Logan Gunthorpe [this message]
2017-08-22 20:14 ` Andy Shevchenko
2017-08-22 17:02 ` [PATCH v7 7/7] crypto: caam: cleanup CONFIG_64BIT ifdefs when using io{read|write}64 Logan Gunthorpe
2017-08-22 17:43 ` [PATCH v7 0/7] make io{read|write}64 globally usable Andy Shevchenko
2017-08-22 17:58 ` Logan Gunthorpe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b4b38bb0-b97f-2240-0ddc-ef5444e289ef@deltatee.com \
--to=logang@deltatee.com \
--cc=andy.shevchenko@gmail.com \
--cc=arnd@arndb.de \
--cc=gregkh@linuxfoundation.org \
--cc=horia.geanta@nxp.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-crypto@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-ntb@googlegroups.com \
--cc=sbates@raithlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox