[PATCH 06/15] ARM: plat-versatile: convert to twd_local_timer_register() interface

On 12/01/12 09:14, Russell King - ARM Linux wrote:

On Wed, Jan 11, 2012 at 01:08:45PM +0000, Marc Zyngier wrote:

quoted

Add support for the new smp_twd runtime registration interface
to the RealView/VE platforms, and remove the old compile-time support.
Tested on EB11MP.

Signed-off-by: Marc Zyngier <redacted>
---
 arch/arm/mach-realview/realview_eb.c     |   34 ++++++++++++++++++++++++++---
 arch/arm/mach-realview/realview_pb11mp.c |   32 ++++++++++++++++++++++++---
 arch/arm/mach-realview/realview_pbx.c    |   31 ++++++++++++++++++++++++--
 arch/arm/mach-vexpress/ct-ca9x4.c        |   30 +++++++++++++++++++++++--
 arch/arm/plat-versatile/Makefile         |    1 -
 arch/arm/plat-versatile/localtimer.c     |   27 -----------------------
 6 files changed, 113 insertions(+), 42 deletions(-)
 delete mode 100644 arch/arm/plat-versatile/localtimer.c

diff --git a/arch/arm/mach-realview/realview_eb.c b/arch/arm/mach-realview/realview_eb.c
index e629621..6c5e61d 100644
--- a/arch/arm/mach-realview/realview_eb.c
+++ b/arch/arm/mach-realview/realview_eb.c

@@ -36,7 +36,7 @@
 #include <asm/pgtable.h>
 #include <asm/hardware/gic.h>
 #include <asm/hardware/cache-l2x0.h>
-#include <asm/localtimer.h>
+#include <asm/smp_twd.h>
 
 #include <asm/mach/arch.h>
 #include <asm/mach/map.h>

@@ -388,6 +388,34 @@ static void realview_eb11mp_fixup(void)
 	realview_eb_isp1761_resources[1].end	= IRQ_EB11MP_USB;
 }
 
+#ifdef CONFIG_HAVE_ARM_TWD
+static struct twd_local_timer twd_local_timer __initdata = {
+	.res =	{
+		{
+			.start	= REALVIEW_EB11MP_TWD_BASE,
+			.end	= REALVIEW_EB11MP_TWD_BASE + 0xff,
+			.flags	= IORESOURCE_MEM,
+		},

DEFINE_RES_MEM() ?

quoted

+		{
+			.start	= IRQ_LOCALTIMER,
+			.end	= IRQ_LOCALTIMER,
+			.flags	= IORESOURCE_IRQ,
+		},

DEFINE_RES_IRQ() ?

Indeed. Actually, I'd like to have the following in smp_twd.h:
#define DEFINE_TWD_LOCAL_TIMER(name,base,irq)	\
struct twd_local_timer name __initdata = {	\
	.res	= {				\
		DEFINE_RES_MEM(base, 0x10),	\
		DEFINE_RES_IRQ(irq),		\
	},					\
};

	M.
-- 
Jazz is not dead. It just smells funny...