linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/4] ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static
  2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
@ 2016-02-04 14:29 ` Simon Horman
  2016-02-04 14:29 ` [PATCH 2/4] ARM: shmobile: Add includes providing forward declarations Simon Horman
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-04 14:29 UTC (permalink / raw)
  To: linux-arm-kernel

From: Geert Uytterhoeven <geert+renesas@glider.be>

make C=1:

    arch/arm/mach-shmobile/setup-rcar-gen2.c:186:12: warning: symbol 'rcar_gen2_dma_contiguous' was not declared. Should it be static?

Make it static, and move it inside the function to avoid a "defined but
not used" warning if CONFIG_DMA_CMA=n.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/setup-rcar-gen2.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/arch/arm/mach-shmobile/setup-rcar-gen2.c b/arch/arm/mach-shmobile/setup-rcar-gen2.c
index 9eccde3c7b13..6d0ebdfb03a2 100644
--- a/arch/arm/mach-shmobile/setup-rcar-gen2.c
+++ b/arch/arm/mach-shmobile/setup-rcar-gen2.c
@@ -182,8 +182,6 @@ static int __init rcar_gen2_scan_mem(unsigned long node, const char *uname,
 	return 0;
 }
 
-struct cma *rcar_gen2_dma_contiguous;
-
 void __init rcar_gen2_reserve(void)
 {
 	struct memory_reserve_config mrc;
@@ -194,8 +192,11 @@ void __init rcar_gen2_reserve(void)
 
 	of_scan_flat_dt(rcar_gen2_scan_mem, &mrc);
 #ifdef CONFIG_DMA_CMA
-	if (mrc.size && memblock_is_region_memory(mrc.base, mrc.size))
+	if (mrc.size && memblock_is_region_memory(mrc.base, mrc.size)) {
+		static struct cma *rcar_gen2_dma_contiguous;
+
 		dma_contiguous_reserve_area(mrc.size, mrc.base, 0,
 					    &rcar_gen2_dma_contiguous, true);
+	}
 #endif
 }
-- 
2.7.0.rc3.207.g0ac5344

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

* [PATCH 2/4] ARM: shmobile: Add includes providing forward declarations
  2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
  2016-02-04 14:29 ` [PATCH 1/4] ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static Simon Horman
@ 2016-02-04 14:29 ` Simon Horman
  2016-02-04 14:29 ` [PATCH 3/4] ARM: shmobile: Typo s/MIPDR/MPIDR/ Simon Horman
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-04 14:29 UTC (permalink / raw)
  To: linux-arm-kernel

From: Geert Uytterhoeven <geert+renesas@glider.be>

make C=1:

    arch/arm/mach-shmobile/timer.c:38:13: warning: symbol 'shmobile_init_delay' was not declared. Should it be static?
    arch/arm/mach-shmobile/suspend.c:37:29: warning: symbol 'shmobile_suspend_ops' was not declared. Should it be static?
    arch/arm/mach-shmobile/suspend.c:44:12: warning: symbol 'shmobile_suspend_init' was not declared. Should it be static?
    arch/arm/mach-shmobile/cpufreq.c:13:12: warning: symbol 'shmobile_cpufreq_init' was not declared. Should it be static?

Include "common.h" to fix these.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/cpufreq.c | 2 ++
 arch/arm/mach-shmobile/suspend.c | 2 ++
 arch/arm/mach-shmobile/timer.c   | 2 ++
 3 files changed, 6 insertions(+)

diff --git a/arch/arm/mach-shmobile/cpufreq.c b/arch/arm/mach-shmobile/cpufreq.c
index 57fbff024dcd..634d701c56a7 100644
--- a/arch/arm/mach-shmobile/cpufreq.c
+++ b/arch/arm/mach-shmobile/cpufreq.c
@@ -10,6 +10,8 @@
 
 #include <linux/platform_device.h>
 
+#include "common.h"
+
 int __init shmobile_cpufreq_init(void)
 {
 	platform_device_register_simple("cpufreq-dt", -1, NULL, 0);
diff --git a/arch/arm/mach-shmobile/suspend.c b/arch/arm/mach-shmobile/suspend.c
index 5d92b5dd486b..74b30bade2c1 100644
--- a/arch/arm/mach-shmobile/suspend.c
+++ b/arch/arm/mach-shmobile/suspend.c
@@ -17,6 +17,8 @@
 #include <asm/io.h>
 #include <asm/system_misc.h>
 
+#include "common.h"
+
 static int shmobile_suspend_default_enter(suspend_state_t suspend_state)
 {
 	cpu_do_idle();
diff --git a/arch/arm/mach-shmobile/timer.c b/arch/arm/mach-shmobile/timer.c
index c17d4d3881ff..ad008e4b0c49 100644
--- a/arch/arm/mach-shmobile/timer.c
+++ b/arch/arm/mach-shmobile/timer.c
@@ -18,6 +18,8 @@
 #include <linux/delay.h>
 #include <linux/of_address.h>
 
+#include "common.h"
+
 static void __init shmobile_setup_delay_hz(unsigned int max_cpu_core_hz,
 					   unsigned int mult, unsigned int div)
 {
-- 
2.7.0.rc3.207.g0ac5344

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

* [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6
@ 2016-02-04 14:29 Simon Horman
  2016-02-04 14:29 ` [PATCH 1/4] ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static Simon Horman
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-04 14:29 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Olof, Hi Kevin, Hi Arnd,

Please consider these Renesas ARM based SoC cleanup for v4.6.

These are all minor cleanups from Geert.


The following changes since commit 92e963f50fc74041b5e9e744c330dca48e04f08d:

  Linux 4.5-rc1 (2016-01-24 13:06:47 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-cleanup-for-v4.6

for you to fetch changes up to 2c7bdcfd8912ce47d8360017265713c5a110b9ed:

  ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code (2016-02-02 15:06:30 +0100)

----------------------------------------------------------------
Renesas ARM Based SoC Cleanup for v4.6

* Remove remnants of removed SCU boot setup code for r8a7779
* Correct s/MIPDR/MPIDR/ typo
* Add includes providing forward declarations
* Make rcar_gen2_dma_contiguous static

----------------------------------------------------------------
Geert Uytterhoeven (4):
      ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static
      ARM: shmobile: Add includes providing forward declarations
      ARM: shmobile: Typo s/MIPDR/MPIDR/
      ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code

 arch/arm/mach-shmobile/cpufreq.c         | 2 ++
 arch/arm/mach-shmobile/headsmp-scu.S     | 2 +-
 arch/arm/mach-shmobile/setup-rcar-gen2.c | 7 ++++---
 arch/arm/mach-shmobile/smp-r8a7779.c     | 2 --
 arch/arm/mach-shmobile/suspend.c         | 2 ++
 arch/arm/mach-shmobile/timer.c           | 2 ++
 6 files changed, 11 insertions(+), 6 deletions(-)

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

* [PATCH 3/4] ARM: shmobile: Typo s/MIPDR/MPIDR/
  2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
  2016-02-04 14:29 ` [PATCH 1/4] ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static Simon Horman
  2016-02-04 14:29 ` [PATCH 2/4] ARM: shmobile: Add includes providing forward declarations Simon Horman
@ 2016-02-04 14:29 ` Simon Horman
  2016-02-04 14:29 ` [PATCH 4/4] ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code Simon Horman
  2016-02-08 21:40 ` [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Olof Johansson
  4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-04 14:29 UTC (permalink / raw)
  To: linux-arm-kernel

From: Geert Uytterhoeven <geert+renesas@glider.be>

The ARM Multiprocessor Affinity Register is called "MPIDR".

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/headsmp-scu.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-shmobile/headsmp-scu.S b/arch/arm/mach-shmobile/headsmp-scu.S
index fa5248c52399..c0008a5aa4ca 100644
--- a/arch/arm/mach-shmobile/headsmp-scu.S
+++ b/arch/arm/mach-shmobile/headsmp-scu.S
@@ -27,7 +27,7 @@
  */
 ENTRY(shmobile_boot_scu)
 					@ r0 = SCU base address
-	mrc     p15, 0, r1, c0, c0, 5	@ read MIPDR
+	mrc     p15, 0, r1, c0, c0, 5	@ read MPIDR
 	and	r1, r1, #3		@ mask out cpu ID
 	lsl	r1, r1, #3		@ we will shift by cpu_id * 8 bits
 	ldr	r2, [r0, #8]		@ SCU Power Status Register
-- 
2.7.0.rc3.207.g0ac5344

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

* [PATCH 4/4] ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code
  2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
                   ` (2 preceding siblings ...)
  2016-02-04 14:29 ` [PATCH 3/4] ARM: shmobile: Typo s/MIPDR/MPIDR/ Simon Horman
@ 2016-02-04 14:29 ` Simon Horman
  2016-02-08 21:40 ` [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Olof Johansson
  4 siblings, 0 replies; 6+ messages in thread
From: Simon Horman @ 2016-02-04 14:29 UTC (permalink / raw)
  To: linux-arm-kernel

From: Geert Uytterhoeven <geert+renesas@glider.be>

Commit 0ca2894b5a900709 ("ARM: shmobile: Use shared SCU SMP boot code on
r8a7779") obsoleted the r8a7779-specific SCU boot code, but forgot to
remove the setup of shmobile_boot_fn and shmobile_boot_arg, which is
overwritten by shmobile_smp_scu_prepare_cpus().

Note that shmobile_scu_base wasn't initialized at that point yet anyway.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm/mach-shmobile/smp-r8a7779.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm/mach-shmobile/smp-r8a7779.c b/arch/arm/mach-shmobile/smp-r8a7779.c
index b854fe2095ad..0b024a9dbd43 100644
--- a/arch/arm/mach-shmobile/smp-r8a7779.c
+++ b/arch/arm/mach-shmobile/smp-r8a7779.c
@@ -92,8 +92,6 @@ static void __init r8a7779_smp_prepare_cpus(unsigned int max_cpus)
 {
 	/* Map the reset vector (in headsmp-scu.S, headsmp.S) */
 	__raw_writel(__pa(shmobile_boot_vector), AVECR);
-	shmobile_boot_fn = virt_to_phys(shmobile_boot_scu);
-	shmobile_boot_arg = (unsigned long)shmobile_scu_base;
 
 	/* setup r8a7779 specific SCU bits */
 	shmobile_scu_base = IOMEM(R8A7779_SCU_BASE);
-- 
2.7.0.rc3.207.g0ac5344

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

* [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6
  2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
                   ` (3 preceding siblings ...)
  2016-02-04 14:29 ` [PATCH 4/4] ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code Simon Horman
@ 2016-02-08 21:40 ` Olof Johansson
  4 siblings, 0 replies; 6+ messages in thread
From: Olof Johansson @ 2016-02-08 21:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Feb 04, 2016 at 03:29:34PM +0100, Simon Horman wrote:
> Hi Olof, Hi Kevin, Hi Arnd,
> 
> Please consider these Renesas ARM based SoC cleanup for v4.6.
> 
> These are all minor cleanups from Geert.
> 
> 
> The following changes since commit 92e963f50fc74041b5e9e744c330dca48e04f08d:
> 
>   Linux 4.5-rc1 (2016-01-24 13:06:47 -0800)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-cleanup-for-v4.6
> 
> for you to fetch changes up to 2c7bdcfd8912ce47d8360017265713c5a110b9ed:
> 
>   ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code (2016-02-02 15:06:30 +0100)


Thanks, merged into next/cleanup.


-Olof

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

end of thread, other threads:[~2016-02-08 21:40 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-04 14:29 [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Simon Horman
2016-02-04 14:29 ` [PATCH 1/4] ARM: shmobile: rcar-gen2: Make rcar_gen2_dma_contiguous static Simon Horman
2016-02-04 14:29 ` [PATCH 2/4] ARM: shmobile: Add includes providing forward declarations Simon Horman
2016-02-04 14:29 ` [PATCH 3/4] ARM: shmobile: Typo s/MIPDR/MPIDR/ Simon Horman
2016-02-04 14:29 ` [PATCH 4/4] ARM: shmobile: r8a7779: Remove remainings of removed SCU boot setup code Simon Horman
2016-02-08 21:40 ` [GIT PULL] Renesas ARM Based SoC Cleanup for v4.6 Olof Johansson

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).