From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Evans Date: Mon, 12 Dec 2016 14:42:29 +0000 Subject: [lustre-devel] [PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move In-Reply-To: <20161210181433.GA8477@kroah.com> References: <1481393161-22623-1-git-send-email-jsimmons@infradead.org> <1481393161-22623-4-git-send-email-jsimmons@infradead.org> <20161210181433.GA8477@kroah.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Greg Kroah-Hartman , James Simmons Cc: "devel@driverdev.osuosl.org" , Andreas Dilger , Oleg Drokin , Ben Evans , Linux Kernel Mailing List , Lustre Development List This was done to conform to the Lustre Coding Guidelines. -Ben On 12/10/16, 1:14 PM, "Greg Kroah-Hartman" wrote: >On Sat, Dec 10, 2016 at 01:05:59PM -0500, James Simmons wrote: >> From: Ben Evans >> >> It was found if you sort the headers alphabetically >> that it reduced patch conflicts. This patch sorts >> the headers alphabetically and also place linux >> header first, then uapi header and finally the >> lustre kernel headers. > >I still don't agree, when did you last have a patch conflict with this >file in the .h section? And exactly how hard was it to fix it up? > >I'm all for cleanups, but really, this is useless. And I said so the >last time you sent it... > >greg k-h