All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
@ 2015-06-09 21:47 ` Peter Griffin
  0 siblings, 0 replies; 8+ messages in thread
From: Peter Griffin @ 2015-06-09 21:47 UTC (permalink / raw)
  To: linux-arm-kernel, linux-kernel, srinivas.kandagatla,
	maxime.coquelin, patrice.chotard, linux
  Cc: peter.griffin, lee.jones

Calling trace_hardirqs_off() from the platform specific
secondary startup code as not been necessary since Dec 2010
when Russell King consolidated the call into the common SMP
code.

2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
---
 arch/arm/mach-sti/platsmp.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/mach-sti/platsmp.c b/arch/arm/mach-sti/platsmp.c
index d4b624f..a39f812 100644
--- a/arch/arm/mach-sti/platsmp.c
+++ b/arch/arm/mach-sti/platsmp.c
@@ -38,8 +38,6 @@ static DEFINE_SPINLOCK(boot_lock);
 
 static void sti_secondary_init(unsigned int cpu)
 {
-	trace_hardirqs_off();
-
 	/*
 	 * let the primary processor know we're out of the
 	 * pen, then head off into the C entry point
-- 
1.9.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
@ 2015-06-09 21:47 ` Peter Griffin
  0 siblings, 0 replies; 8+ messages in thread
From: Peter Griffin @ 2015-06-09 21:47 UTC (permalink / raw)
  To: linux-arm-kernel

Calling trace_hardirqs_off() from the platform specific
secondary startup code as not been necessary since Dec 2010
when Russell King consolidated the call into the common SMP
code.

2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
---
 arch/arm/mach-sti/platsmp.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/mach-sti/platsmp.c b/arch/arm/mach-sti/platsmp.c
index d4b624f..a39f812 100644
--- a/arch/arm/mach-sti/platsmp.c
+++ b/arch/arm/mach-sti/platsmp.c
@@ -38,8 +38,6 @@ static DEFINE_SPINLOCK(boot_lock);
 
 static void sti_secondary_init(unsigned int cpu)
 {
-	trace_hardirqs_off();
-
 	/*
 	 * let the primary processor know we're out of the
 	 * pen, then head off into the C entry point
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
  2015-06-09 21:47 ` Peter Griffin
@ 2015-06-10  7:40   ` Lee Jones
  -1 siblings, 0 replies; 8+ messages in thread
From: Lee Jones @ 2015-06-10  7:40 UTC (permalink / raw)
  To: Peter Griffin
  Cc: linux-arm-kernel, linux-kernel, srinivas.kandagatla,
	maxime.coquelin, patrice.chotard, linux

On Tue, 09 Jun 2015, Peter Griffin wrote:

> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
> 
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
> 
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>  arch/arm/mach-sti/platsmp.c | 2 --
>  1 file changed, 2 deletions(-)

Acked-by: Lee Jones <lee.jones@linaro.org>

> diff --git a/arch/arm/mach-sti/platsmp.c b/arch/arm/mach-sti/platsmp.c
> index d4b624f..a39f812 100644
> --- a/arch/arm/mach-sti/platsmp.c
> +++ b/arch/arm/mach-sti/platsmp.c
> @@ -38,8 +38,6 @@ static DEFINE_SPINLOCK(boot_lock);
>  
>  static void sti_secondary_init(unsigned int cpu)
>  {
> -	trace_hardirqs_off();
> -
>  	/*
>  	 * let the primary processor know we're out of the
>  	 * pen, then head off into the C entry point

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
@ 2015-06-10  7:40   ` Lee Jones
  0 siblings, 0 replies; 8+ messages in thread
From: Lee Jones @ 2015-06-10  7:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, 09 Jun 2015, Peter Griffin wrote:

> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
> 
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
> 
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>  arch/arm/mach-sti/platsmp.c | 2 --
>  1 file changed, 2 deletions(-)

Acked-by: Lee Jones <lee.jones@linaro.org>

> diff --git a/arch/arm/mach-sti/platsmp.c b/arch/arm/mach-sti/platsmp.c
> index d4b624f..a39f812 100644
> --- a/arch/arm/mach-sti/platsmp.c
> +++ b/arch/arm/mach-sti/platsmp.c
> @@ -38,8 +38,6 @@ static DEFINE_SPINLOCK(boot_lock);
>  
>  static void sti_secondary_init(unsigned int cpu)
>  {
> -	trace_hardirqs_off();
> -
>  	/*
>  	 * let the primary processor know we're out of the
>  	 * pen, then head off into the C entry point

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
  2015-06-09 21:47 ` Peter Griffin
@ 2015-06-10  8:09   ` Maxime Coquelin
  -1 siblings, 0 replies; 8+ messages in thread
From: Maxime Coquelin @ 2015-06-10  8:09 UTC (permalink / raw)
  To: Peter Griffin, linux-arm-kernel, linux-kernel,
	srinivas.kandagatla, patrice.chotard, linux
  Cc: lee.jones

Hi Peter

On 06/09/2015 11:47 PM, Peter Griffin wrote:
> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
>
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>   arch/arm/mach-sti/platsmp.c | 2 --
>   1 file changed, 2 deletions(-)
>
>

Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks!
Maxime

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
@ 2015-06-10  8:09   ` Maxime Coquelin
  0 siblings, 0 replies; 8+ messages in thread
From: Maxime Coquelin @ 2015-06-10  8:09 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Peter

On 06/09/2015 11:47 PM, Peter Griffin wrote:
> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
>
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>   arch/arm/mach-sti/platsmp.c | 2 --
>   1 file changed, 2 deletions(-)
>
>

Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

Thanks!
Maxime

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
  2015-06-09 21:47 ` Peter Griffin
@ 2015-07-22  9:17   ` Maxime Coquelin
  -1 siblings, 0 replies; 8+ messages in thread
From: Maxime Coquelin @ 2015-07-22  9:17 UTC (permalink / raw)
  To: Peter Griffin, linux-arm-kernel, linux-kernel,
	srinivas.kandagatla, patrice.chotard, linux
  Cc: lee.jones

HI Peter,

On 06/09/2015 11:47 PM, Peter Griffin wrote:
> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
>
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>   arch/arm/mach-sti/platsmp.c | 2 --
>   1 file changed, 2 deletions(-)
>

Applied to sti-soc-for-v4.3.

Thanks!
Maxime

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off()
@ 2015-07-22  9:17   ` Maxime Coquelin
  0 siblings, 0 replies; 8+ messages in thread
From: Maxime Coquelin @ 2015-07-22  9:17 UTC (permalink / raw)
  To: linux-arm-kernel

HI Peter,

On 06/09/2015 11:47 PM, Peter Griffin wrote:
> Calling trace_hardirqs_off() from the platform specific
> secondary startup code as not been necessary since Dec 2010
> when Russell King consolidated the call into the common SMP
> code.
>
> 2c0136d ARM: SMP: consolidate trace_hardirqs_off() into common SMP code
>
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>   arch/arm/mach-sti/platsmp.c | 2 --
>   1 file changed, 2 deletions(-)
>

Applied to sti-soc-for-v4.3.

Thanks!
Maxime

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-07-22  9:18 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-09 21:47 [PATCH] ARM: STi: Remove platform call to trace_hardirqs_off() Peter Griffin
2015-06-09 21:47 ` Peter Griffin
2015-06-10  7:40 ` Lee Jones
2015-06-10  7:40   ` Lee Jones
2015-06-10  8:09 ` Maxime Coquelin
2015-06-10  8:09   ` Maxime Coquelin
2015-07-22  9:17 ` Maxime Coquelin
2015-07-22  9:17   ` Maxime Coquelin

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.