All of lore.kernel.org
 help / color / mirror / Atom feed
From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: linux-omap@vger.kernel.org
Cc: khilman@deeprootsystems.com, tony@atomide.com,
	linux-arm-kernel@lists.infradead.org,
	Santosh Shilimkar <santosh.shilimkar@ti.com>
Subject: [PATCH 5/9] ARM: OMAP4+: Remove un-necessary cacheflush in secondary CPU boot path
Date: Wed, 20 Feb 2013 20:48:43 +0530	[thread overview]
Message-ID: <1361373527-21695-6-git-send-email-santosh.shilimkar@ti.com> (raw)
In-Reply-To: <1361373527-21695-1-git-send-email-santosh.shilimkar@ti.com>

This was borrowed from ARM versatile code with pen_release mechanism but since
OMAP uses hardware register based synchronisation, pen_release stuff was
dropped. Unfortunately the cacheflush wasn't dropped along with it.

Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
---
 arch/arm/mach-omap2/omap-smp.c |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/arm/mach-omap2/omap-smp.c b/arch/arm/mach-omap2/omap-smp.c
index cd42d92..81fd489 100644
--- a/arch/arm/mach-omap2/omap-smp.c
+++ b/arch/arm/mach-omap2/omap-smp.c
@@ -20,7 +20,6 @@
 #include <linux/smp.h>
 #include <linux/io.h>
 
-#include <asm/cacheflush.h>
 #include <asm/hardware/gic.h>
 #include <asm/smp_scu.h>
 
@@ -103,9 +102,6 @@ static int __cpuinit omap4_boot_secondary(unsigned int cpu, struct task_struct *
 	else
 		__raw_writel(0x20, base + OMAP_AUX_CORE_BOOT_0);
 
-	flush_cache_all();
-	smp_wmb();
-
 	if (!cpu1_clkdm)
 		cpu1_clkdm = clkdm_lookup("mpu1_clkdm");
 
-- 
1.7.9.5


WARNING: multiple messages have this Message-ID (diff)
From: santosh.shilimkar@ti.com (Santosh Shilimkar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 5/9] ARM: OMAP4+: Remove un-necessary cacheflush in secondary CPU boot path
Date: Wed, 20 Feb 2013 20:48:43 +0530	[thread overview]
Message-ID: <1361373527-21695-6-git-send-email-santosh.shilimkar@ti.com> (raw)
In-Reply-To: <1361373527-21695-1-git-send-email-santosh.shilimkar@ti.com>

This was borrowed from ARM versatile code with pen_release mechanism but since
OMAP uses hardware register based synchronisation, pen_release stuff was
dropped. Unfortunately the cacheflush wasn't dropped along with it.

Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
---
 arch/arm/mach-omap2/omap-smp.c |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/arm/mach-omap2/omap-smp.c b/arch/arm/mach-omap2/omap-smp.c
index cd42d92..81fd489 100644
--- a/arch/arm/mach-omap2/omap-smp.c
+++ b/arch/arm/mach-omap2/omap-smp.c
@@ -20,7 +20,6 @@
 #include <linux/smp.h>
 #include <linux/io.h>
 
-#include <asm/cacheflush.h>
 #include <asm/hardware/gic.h>
 #include <asm/smp_scu.h>
 
@@ -103,9 +102,6 @@ static int __cpuinit omap4_boot_secondary(unsigned int cpu, struct task_struct *
 	else
 		__raw_writel(0x20, base + OMAP_AUX_CORE_BOOT_0);
 
-	flush_cache_all();
-	smp_wmb();
-
 	if (!cpu1_clkdm)
 		cpu1_clkdm = clkdm_lookup("mpu1_clkdm");
 
-- 
1.7.9.5

  parent reply	other threads:[~2013-02-20 15:18 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-20 15:18 [PATCH 0/9] ARM: OMAP: Static deps, fiq, omap-smp cleanup Santosh Shilimkar
2013-02-20 15:18 ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 1/9] ARM: OMAP4+: Use common scratchpad SAR RAM offsets for all architectures Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:41   ` Kevin Hilman
2013-03-27 18:41     ` Kevin Hilman
2013-03-27 20:49     ` Santosh Shilimkar
2013-03-27 20:49       ` Santosh Shilimkar
2013-03-27 20:49       ` Tony Lindgren
2013-03-27 20:49         ` Tony Lindgren
2013-03-27 20:52         ` Santosh Shilimkar
2013-03-27 20:52           ` Santosh Shilimkar
2013-03-28  7:32           ` Santosh Shilimkar
2013-03-28  7:32             ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 2/9] ARM: OMAP1: PM: Remove bogus fiq_[enable/disable] tuple Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-02-20 16:09   ` Tony Lindgren
2013-02-20 16:09     ` Tony Lindgren
2013-02-20 16:14     ` Santosh Shilimkar
2013-02-20 16:14       ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 3/9] ARM: OMAP2+: " Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:36   ` Kevin Hilman
2013-03-27 18:36     ` Kevin Hilman
2013-03-27 19:02     ` Santosh Shilimkar
2013-03-27 19:02       ` Santosh Shilimkar
2013-03-28  7:37       ` Santosh Shilimkar
2013-03-28  7:37         ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 4/9] ARM: OMAP4+: Remove the un-necessary cache flush from hotplug code Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:43   ` Kevin Hilman
2013-03-27 18:43     ` Kevin Hilman
2013-03-27 20:47     ` Santosh Shilimkar
2013-03-27 20:47       ` Santosh Shilimkar
2013-03-28  7:29       ` Santosh Shilimkar
2013-03-28  7:29         ` Santosh Shilimkar
2013-02-20 15:18 ` Santosh Shilimkar [this message]
2013-02-20 15:18   ` [PATCH 5/9] ARM: OMAP4+: Remove un-necessary cacheflush in secondary CPU boot path Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 6/9] ARM: OMAP4+: Remove out of placed smp_wmb() in secondary wakeup code Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-02-21 12:55   ` Sergei Shtylyov
2013-02-21 12:55     ` Sergei Shtylyov
2013-02-21 12:59     ` Santosh Shilimkar
2013-02-21 12:59       ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 7/9] ARM: OMAP4+: Move the CPU wakeup prepare code under smp_prepare_cpus() Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:45   ` Kevin Hilman
2013-03-27 18:45     ` Kevin Hilman
2013-03-27 19:04     ` Santosh Shilimkar
2013-03-27 19:04       ` Santosh Shilimkar
2013-03-27 19:54       ` Kevin Hilman
2013-03-27 19:54         ` Kevin Hilman
2013-03-27 20:50         ` Santosh Shilimkar
2013-03-27 20:50           ` Santosh Shilimkar
2013-03-28  7:35           ` Santosh Shilimkar
2013-03-28  7:35             ` Santosh Shilimkar
2013-03-28  9:46       ` Russell King - ARM Linux
2013-03-28  9:46         ` Russell King - ARM Linux
2013-03-28  9:58         ` Santosh Shilimkar
2013-03-28  9:58           ` Santosh Shilimkar
2013-03-28 12:04           ` Russell King - ARM Linux
2013-03-28 12:04             ` Russell King - ARM Linux
2013-03-28 12:09             ` Santosh Shilimkar
2013-03-28 12:09               ` Santosh Shilimkar
2013-02-20 15:18 ` [PATCH 8/9] ARM: OMAP4: PM: Remove L4 wakeup depedency with MPU since errata fix exist now Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:46   ` Kevin Hilman
2013-03-27 18:46     ` Kevin Hilman
2013-03-27 19:01     ` Peter Korsgaard
2013-03-27 19:01       ` Peter Korsgaard
2013-02-20 15:18 ` [PATCH 9/9] ARM: OMAP4: PM: Now remove L4 per clockdomain static depedency with MPU Santosh Shilimkar
2013-02-20 15:18   ` Santosh Shilimkar
2013-03-27 18:46   ` Kevin Hilman
2013-03-27 18:46     ` Kevin Hilman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1361373527-21695-6-git-send-email-santosh.shilimkar@ti.com \
    --to=santosh.shilimkar@ti.com \
    --cc=khilman@deeprootsystems.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.