linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: remove unused CONFIG_ARCH_HAS_BARRIERS
@ 2016-06-13 12:25 Masahiro Yamada
  2016-06-20  1:28 ` Masahiro Yamada
  0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2016-06-13 12:25 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Masahiro Yamada, Zhaoxiu Zeng, Arnd Bergmann, Kees Cook,
	Linus Walleij, linux-kernel, Michael S. Tsirkin,
	Peter Zijlstra (Intel),
	Richard Woodruff, Andrey Konovalov, Russell King, Tony Lindgren

Since commit 2b749cb3a515 ("ARM: realview: remove private barrier
implementation"), this config is not used by any platform.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 arch/arm/include/asm/barrier.h | 4 +---
 arch/arm/mm/Kconfig            | 6 ------
 2 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/arch/arm/include/asm/barrier.h b/arch/arm/include/asm/barrier.h
index 112cc1a..f5d6981 100644
--- a/arch/arm/include/asm/barrier.h
+++ b/arch/arm/include/asm/barrier.h
@@ -44,9 +44,7 @@ extern void arm_heavy_mb(void);
 #define __arm_heavy_mb(x...) dsb(x)
 #endif
 
-#ifdef CONFIG_ARCH_HAS_BARRIERS
-#include <mach/barriers.h>
-#elif defined(CONFIG_ARM_DMA_MEM_BUFFERABLE) || defined(CONFIG_SMP)
+#if defined(CONFIG_ARM_DMA_MEM_BUFFERABLE) || defined(CONFIG_SMP)
 #define mb()		__arm_heavy_mb()
 #define rmb()		dsb()
 #define wmb()		__arm_heavy_mb(st)
diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig
index cb569b6..d15a7fe 100644
--- a/arch/arm/mm/Kconfig
+++ b/arch/arm/mm/Kconfig
@@ -1025,12 +1025,6 @@ config ARM_DMA_MEM_BUFFERABLE
 
 	  You are recommended say 'Y' here and debug any affected drivers.
 
-config ARCH_HAS_BARRIERS
-	bool
-	help
-	  This option allows the use of custom mandatory barriers
-	  included via the mach/barriers.h file.
-
 config ARM_HEAVY_MB
 	bool
 
-- 
1.9.1

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

* Re: [PATCH] ARM: remove unused CONFIG_ARCH_HAS_BARRIERS
  2016-06-13 12:25 [PATCH] ARM: remove unused CONFIG_ARCH_HAS_BARRIERS Masahiro Yamada
@ 2016-06-20  1:28 ` Masahiro Yamada
  2016-06-20  7:42   ` Arnd Bergmann
  0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2016-06-20  1:28 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Richard Woodruff, Kees Cook, Arnd Bergmann, Michael S. Tsirkin,
	Peter Zijlstra (Intel),
	Andrey Konovalov, Linus Walleij, Linux Kernel Mailing List,
	Russell King, Masahiro Yamada, Tony Lindgren, Zhaoxiu Zeng

Hi.

I have not had any comment on this patch,
but it looks correct to me.

I will put it into Russell's patch tracker
if nobody is against it.



2016-06-13 21:25 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>:
> Since commit 2b749cb3a515 ("ARM: realview: remove private barrier
> implementation"), this config is not used by any platform.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
>  arch/arm/include/asm/barrier.h | 4 +---
>  arch/arm/mm/Kconfig            | 6 ------
>  2 files changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/arch/arm/include/asm/barrier.h b/arch/arm/include/asm/barrier.h
> index 112cc1a..f5d6981 100644
> --- a/arch/arm/include/asm/barrier.h
> +++ b/arch/arm/include/asm/barrier.h
> @@ -44,9 +44,7 @@ extern void arm_heavy_mb(void);
>  #define __arm_heavy_mb(x...) dsb(x)
>  #endif
>
> -#ifdef CONFIG_ARCH_HAS_BARRIERS
> -#include <mach/barriers.h>
> -#elif defined(CONFIG_ARM_DMA_MEM_BUFFERABLE) || defined(CONFIG_SMP)
> +#if defined(CONFIG_ARM_DMA_MEM_BUFFERABLE) || defined(CONFIG_SMP)
>  #define mb()           __arm_heavy_mb()
>  #define rmb()          dsb()
>  #define wmb()          __arm_heavy_mb(st)
> diff --git a/arch/arm/mm/Kconfig b/arch/arm/mm/Kconfig
> index cb569b6..d15a7fe 100644
> --- a/arch/arm/mm/Kconfig
> +++ b/arch/arm/mm/Kconfig
> @@ -1025,12 +1025,6 @@ config ARM_DMA_MEM_BUFFERABLE
>
>           You are recommended say 'Y' here and debug any affected drivers.
>
> -config ARCH_HAS_BARRIERS
> -       bool
> -       help
> -         This option allows the use of custom mandatory barriers
> -         included via the mach/barriers.h file.
> -
>  config ARM_HEAVY_MB
>         bool
>
> --
> 1.9.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



-- 
Best Regards
Masahiro Yamada

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

* Re: [PATCH] ARM: remove unused CONFIG_ARCH_HAS_BARRIERS
  2016-06-20  1:28 ` Masahiro Yamada
@ 2016-06-20  7:42   ` Arnd Bergmann
  0 siblings, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2016-06-20  7:42 UTC (permalink / raw)
  To: linux-arm-kernel
  Cc: Masahiro Yamada, Richard Woodruff, Kees Cook, Michael S. Tsirkin,
	Peter Zijlstra (Intel),
	Andrey Konovalov, Linus Walleij, Linux Kernel Mailing List,
	Russell King, Tony Lindgren, Zhaoxiu Zeng

On Monday, June 20, 2016 10:28:29 AM CEST Masahiro Yamada wrote:
> Hi.
> 
> I have not had any comment on this patch,
> but it looks correct to me.

Yes, I agree. Feel free to add my 

Acked-by: Arnd Bergmann <arnd@arndb.de>

> I will put it into Russell's patch tracker
> if nobody is against it.

Ok, thanks!

	Arnd

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

end of thread, other threads:[~2016-06-20  7:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-13 12:25 [PATCH] ARM: remove unused CONFIG_ARCH_HAS_BARRIERS Masahiro Yamada
2016-06-20  1:28 ` Masahiro Yamada
2016-06-20  7:42   ` Arnd Bergmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).