All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
@ 2021-03-03  2:04 ` menglong8.dong
  0 siblings, 0 replies; 6+ messages in thread
From: menglong8.dong @ 2021-03-03  2:04 UTC (permalink / raw)
  To: vladimir.murzin
  Cc: linux, akpm, rppt, maz, geert, anshuman.khandual, zhang.yunkai,
	linux-arm-kernel, linux-kernel

From: Zhang Yunkai <zhang.yunkai@zte.com.cn>

'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.

Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>
---
 arch/arm/include/asm/pgtable.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/pgtable.h b/arch/arm/include/asm/pgtable.h
index c02f24400369..f3124e57be83 100644
--- a/arch/arm/include/asm/pgtable.h
+++ b/arch/arm/include/asm/pgtable.h
@@ -9,15 +9,14 @@
 
 #include <linux/const.h>
 #include <asm/proc-fns.h>
+#include <asm-generic/pgtable-nopud.h>
 
 #ifndef CONFIG_MMU
 
-#include <asm-generic/pgtable-nopud.h>
 #include <asm/pgtable-nommu.h>
 
 #else
 
-#include <asm-generic/pgtable-nopud.h>
 #include <asm/memory.h>
 #include <asm/pgtable-hwdef.h>
 
-- 
2.25.1


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

* [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
@ 2021-03-03  2:04 ` menglong8.dong
  0 siblings, 0 replies; 6+ messages in thread
From: menglong8.dong @ 2021-03-03  2:04 UTC (permalink / raw)
  To: vladimir.murzin
  Cc: linux, akpm, rppt, maz, geert, anshuman.khandual, zhang.yunkai,
	linux-arm-kernel, linux-kernel

From: Zhang Yunkai <zhang.yunkai@zte.com.cn>

'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.

Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>
---
 arch/arm/include/asm/pgtable.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/pgtable.h b/arch/arm/include/asm/pgtable.h
index c02f24400369..f3124e57be83 100644
--- a/arch/arm/include/asm/pgtable.h
+++ b/arch/arm/include/asm/pgtable.h
@@ -9,15 +9,14 @@
 
 #include <linux/const.h>
 #include <asm/proc-fns.h>
+#include <asm-generic/pgtable-nopud.h>
 
 #ifndef CONFIG_MMU
 
-#include <asm-generic/pgtable-nopud.h>
 #include <asm/pgtable-nommu.h>
 
 #else
 
-#include <asm-generic/pgtable-nopud.h>
 #include <asm/memory.h>
 #include <asm/pgtable-hwdef.h>
 
-- 
2.25.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
  2021-03-03  2:04 ` menglong8.dong
@ 2021-03-03  8:22   ` Geert Uytterhoeven
  -1 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2021-03-03  8:22 UTC (permalink / raw)
  To: menglong8.dong
  Cc: Vladimir Murzin, Russell King, Andrew Morton, Mike Rapoport,
	Marc Zyngier, Anshuman Khandual, zhang.yunkai, Linux ARM,
	Linux Kernel Mailing List

On Wed, Mar 3, 2021 at 3:04 AM <menglong8.dong@gmail.com> wrote:
> From: Zhang Yunkai <zhang.yunkai@zte.com.cn>
>
> 'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.
>
> Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
@ 2021-03-03  8:22   ` Geert Uytterhoeven
  0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2021-03-03  8:22 UTC (permalink / raw)
  To: menglong8.dong
  Cc: Vladimir Murzin, Russell King, Andrew Morton, Mike Rapoport,
	Marc Zyngier, Anshuman Khandual, zhang.yunkai, Linux ARM,
	Linux Kernel Mailing List

On Wed, Mar 3, 2021 at 3:04 AM <menglong8.dong@gmail.com> wrote:
> From: Zhang Yunkai <zhang.yunkai@zte.com.cn>
>
> 'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.
>
> Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
  2021-03-03  2:04 ` menglong8.dong
@ 2021-03-03 10:24   ` Russell King - ARM Linux admin
  -1 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux admin @ 2021-03-03 10:24 UTC (permalink / raw)
  To: menglong8.dong
  Cc: vladimir.murzin, akpm, rppt, maz, geert, anshuman.khandual,
	zhang.yunkai, linux-arm-kernel, linux-kernel

On Tue, Mar 02, 2021 at 06:04:22PM -0800, menglong8.dong@gmail.com wrote:
> From: Zhang Yunkai <zhang.yunkai@zte.com.cn>
> 
> 'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.
> 
> Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>

I don't see this change has anything to do with module code, so what is
the reason for "module:" in the subject line? What am I missing here?

I also think that this patch needs a better explanation, since it's not
a removal of a redundant include, whereas your other similarly described
changes are.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!

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

* Re: [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h
@ 2021-03-03 10:24   ` Russell King - ARM Linux admin
  0 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux admin @ 2021-03-03 10:24 UTC (permalink / raw)
  To: menglong8.dong
  Cc: vladimir.murzin, akpm, rppt, maz, geert, anshuman.khandual,
	zhang.yunkai, linux-arm-kernel, linux-kernel

On Tue, Mar 02, 2021 at 06:04:22PM -0800, menglong8.dong@gmail.com wrote:
> From: Zhang Yunkai <zhang.yunkai@zte.com.cn>
> 
> 'asm-generic/pgtable-nopud.h' included in 'pgtable.h' is duplicated.
> 
> Signed-off-by: Zhang Yunkai <zhang.yunkai@zte.com.cn>

I don't see this change has anything to do with module code, so what is
the reason for "module:" in the subject line? What am I missing here?

I also think that this patch needs a better explanation, since it's not
a removal of a redundant include, whereas your other similarly described
changes are.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2021-03-03 21:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-03  2:04 [PATCH] module: remove duplicate include in ./arch/arm/include/asm/pgtable.h menglong8.dong
2021-03-03  2:04 ` menglong8.dong
2021-03-03  8:22 ` Geert Uytterhoeven
2021-03-03  8:22   ` Geert Uytterhoeven
2021-03-03 10:24 ` Russell King - ARM Linux admin
2021-03-03 10:24   ` Russell King - ARM Linux admin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.