From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753455AbdHPCb0 (ORCPT <rfc822;w@1wt.eu>);
        Tue, 15 Aug 2017 22:31:26 -0400
Received: from icp-osb-irony-out4.external.iinet.net.au ([203.59.1.220]:7064
        "EHLO icp-osb-irony-out4.external.iinet.net.au" rhost-flags-OK-OK-OK-OK)
        by vger.kernel.org with ESMTP id S1752612AbdHPCbW (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 15 Aug 2017 22:31:22 -0400
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A2BaAgBDrZNZ/0Tj/dxeHAEBBAEBCgEBg1qBeY4SkBE4AQEBAQEBBoEIIhGWB4ISHIUrAoQ2PxgBAgEBAQEBAQFrKIUZBiNWEBgBBwUCGA4CAkcQBhOKIgyscoImIgKLRQEBCCiBC4IdggKBBYJgineCYQWJe4gjjh+UQItdhnyWGB84gQpTLwpJhyczNooCAQEB
X-IPAS-Result: A2BaAgBDrZNZ/0Tj/dxeHAEBBAEBCgEBg1qBeY4SkBE4AQEBAQEBBoEIIhGWB4ISHIUrAoQ2PxgBAgEBAQEBAQFrKIUZBiNWEBgBBwUCGA4CAkcQBhOKIgyscoImIgKLRQEBCCiBC4IdggKBBYJgineCYQWJe4gjjh+UQItdhnyWGB84gQpTLwpJhyczNooCAQEB
X-IronPort-AV: E=Sophos;i="5.41,380,1498492800"; 
   d="scan'208";a="285016056"
Subject: [PATCH 2/5] autofs: non functional header inclusion cleanup
From: Ian Kent <raven@themaw.net>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-fsdevel <linux-fsdevel@vger.kernel.org>,
        Tomohiro Kusumi <kusumi.tomohiro@gmail.com>,
        autofs mailing list <autofs@vger.kernel.org>,
        Kernel Mailing List <linux-kernel@vger.kernel.org>
Date: Wed, 16 Aug 2017 10:31:20 +0800
Message-ID: <150285068011.4670.10271483982093996996.stgit@pluto.themaw.net>
In-Reply-To: <150285067347.4670.11494624644273072003.stgit@pluto.themaw.net>
References: <150285067347.4670.11494624644273072003.stgit@pluto.themaw.net>
User-Agent: StGit/0.17.1-dirty
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Tomohiro Kusumi <tkusumi@tuxera.com>

Having header includes before any macro (without any dependency)
simply looks normal. No reason to have these macros in between.

Signed-off-by: Tomohiro Kusumi <tkusumi@tuxera.com>
Signed-off-by: Ian Kent <raven@themaw.net>
---
 fs/autofs4/autofs_i.h |   22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/fs/autofs4/autofs_i.h b/fs/autofs4/autofs_i.h
index beef981aa54f..4737615f0eaa 100644
--- a/fs/autofs4/autofs_i.h
+++ b/fs/autofs4/autofs_i.h
@@ -11,10 +11,21 @@
 
 #include <linux/auto_fs4.h>
 #include <linux/auto_dev-ioctl.h>
+
+#include <linux/kernel.h>
+#include <linux/slab.h>
+#include <linux/time.h>
+#include <linux/string.h>
+#include <linux/wait.h>
+#include <linux/sched.h>
+#include <linux/mount.h>
+#include <linux/namei.h>
+#include <linux/uaccess.h>
 #include <linux/mutex.h>
 #include <linux/spinlock.h>
 #include <linux/list.h>
 #include <linux/completion.h>
+#include <asm/current.h>
 
 /* This is the range of ioctl() numbers we claim as ours */
 #define AUTOFS_IOC_FIRST     AUTOFS_IOC_READY
@@ -24,17 +35,6 @@
 #define AUTOFS_DEV_IOCTL_IOC_COUNT \
 	(AUTOFS_DEV_IOCTL_ISMOUNTPOINT_CMD - AUTOFS_DEV_IOCTL_VERSION_CMD)
 
-#include <linux/kernel.h>
-#include <linux/slab.h>
-#include <linux/time.h>
-#include <linux/string.h>
-#include <linux/wait.h>
-#include <linux/sched.h>
-#include <linux/mount.h>
-#include <linux/namei.h>
-#include <asm/current.h>
-#include <linux/uaccess.h>
-
 #ifdef pr_fmt
 #undef pr_fmt
 #endif