mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] fs: ext2: Fix duplicate included linux/dax.h
@ 2022-03-23  4:02 Haowen Bai
  2022-03-23 11:33 ` Ritesh Harjani
  0 siblings, 1 reply; 3+ messages in thread
From: Haowen Bai @ 2022-03-23  4:02 UTC (permalink / raw)
  To: jack; +Cc: linux-ext4, linux-kernel, Haowen Bai

Clean up the following includecheck warning:

fs/ext2/inode.c: linux/dax.h is included more than once.

No functional change.

Signed-off-by: Haowen Bai <baihaowen@meizu.com>
---
 fs/ext2/inode.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
index 602578b..da4c301 100644
--- a/fs/ext2/inode.c
+++ b/fs/ext2/inode.c
@@ -36,7 +36,6 @@
 #include <linux/iomap.h>
 #include <linux/namei.h>
 #include <linux/uio.h>
-#include <linux/dax.h>
 #include "ext2.h"
 #include "acl.h"
 #include "xattr.h"
-- 
2.7.4


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] fs: ext2: Fix duplicate included linux/dax.h
  2022-03-23  4:02 [PATCH] fs: ext2: Fix duplicate included linux/dax.h Haowen Bai
@ 2022-03-23 11:33 ` Ritesh Harjani
  2022-04-04  7:14   ` Jan Kara
  0 siblings, 1 reply; 3+ messages in thread
From: Ritesh Harjani @ 2022-03-23 11:33 UTC (permalink / raw)
  To: Haowen Bai; +Cc: jack, linux-ext4, linux-kernel

On 22/03/23 12:02PM, Haowen Bai wrote:
> Clean up the following includecheck warning:
>
> fs/ext2/inode.c: linux/dax.h is included more than once.
>


Checked "make includecheck"
This is the only warning coming from fs/ext2/

Thanks for the cleanup. Looks good to me.
Feel free to add -

Reviewed-by: Ritesh Harjani <riteshh@linux.ibm.com>



> No functional change.
>
> Signed-off-by: Haowen Bai <baihaowen@meizu.com>
> ---
>  fs/ext2/inode.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
> index 602578b..da4c301 100644
> --- a/fs/ext2/inode.c
> +++ b/fs/ext2/inode.c
> @@ -36,7 +36,6 @@
>  #include <linux/iomap.h>
>  #include <linux/namei.h>
>  #include <linux/uio.h>
> -#include <linux/dax.h>
>  #include "ext2.h"
>  #include "acl.h"
>  #include "xattr.h"
> --
> 2.7.4
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] fs: ext2: Fix duplicate included linux/dax.h
  2022-03-23 11:33 ` Ritesh Harjani
@ 2022-04-04  7:14   ` Jan Kara
  0 siblings, 0 replies; 3+ messages in thread
From: Jan Kara @ 2022-04-04  7:14 UTC (permalink / raw)
  To: Ritesh Harjani; +Cc: Haowen Bai, jack, linux-ext4, linux-kernel

On Wed 23-03-22 17:03:51, Ritesh Harjani wrote:
> On 22/03/23 12:02PM, Haowen Bai wrote:
> > Clean up the following includecheck warning:
> >
> > fs/ext2/inode.c: linux/dax.h is included more than once.
> >
> 
> 
> Checked "make includecheck"
> This is the only warning coming from fs/ext2/
> 
> Thanks for the cleanup. Looks good to me.
> Feel free to add -
> 
> Reviewed-by: Ritesh Harjani <riteshh@linux.ibm.com>
> 

Thanks. I've added the patch to my tree.

								Honza

-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-04-04  7:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-23  4:02 [PATCH] fs: ext2: Fix duplicate included linux/dax.h Haowen Bai
2022-03-23 11:33 ` Ritesh Harjani
2022-04-04  7:14   ` Jan Kara

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®