clocksource: time-armada-370-xp: Divorce from local timer API

Separate the armada 370xp local timers from the local timer API.
This will allow us to remove ARM local timer support in the near
future and makes this driver multi-architecture friendly.

Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Tested-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
This commit is contained in:
Stephen Boyd 2013-02-15 17:02:16 -08:00
parent faef31b482
commit 5ddb6d21c3

View File

@ -19,6 +19,7 @@
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/clk.h> #include <linux/clk.h>
#include <linux/cpu.h>
#include <linux/timer.h> #include <linux/timer.h>
#include <linux/clockchips.h> #include <linux/clockchips.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
@ -31,7 +32,6 @@
#include <linux/percpu.h> #include <linux/percpu.h>
#include <linux/time-armada-370-xp.h> #include <linux/time-armada-370-xp.h>
#include <asm/localtimer.h>
/* /*
* Timer block registers. * Timer block registers.
*/ */
@ -70,7 +70,7 @@ static bool timer25Mhz = true;
*/ */
static u32 ticks_per_jiffy; static u32 ticks_per_jiffy;
static struct clock_event_device __percpu **percpu_armada_370_xp_evt; static struct clock_event_device __percpu *armada_370_xp_evt;
static u32 notrace armada_370_xp_read_sched_clock(void) static u32 notrace armada_370_xp_read_sched_clock(void)
{ {
@ -143,21 +143,14 @@ armada_370_xp_clkevt_mode(enum clock_event_mode mode,
} }
} }
static struct clock_event_device armada_370_xp_clkevt = { static int armada_370_xp_clkevt_irq;
.name = "armada_370_xp_per_cpu_tick",
.features = CLOCK_EVT_FEAT_ONESHOT | CLOCK_EVT_FEAT_PERIODIC,
.shift = 32,
.rating = 300,
.set_next_event = armada_370_xp_clkevt_next_event,
.set_mode = armada_370_xp_clkevt_mode,
};
static irqreturn_t armada_370_xp_timer_interrupt(int irq, void *dev_id) static irqreturn_t armada_370_xp_timer_interrupt(int irq, void *dev_id)
{ {
/* /*
* ACK timer interrupt and call event handler. * ACK timer interrupt and call event handler.
*/ */
struct clock_event_device *evt = *(struct clock_event_device **)dev_id; struct clock_event_device *evt = dev_id;
writel(TIMER0_CLR_MASK, local_base + LCL_TIMER_EVENTS_STATUS); writel(TIMER0_CLR_MASK, local_base + LCL_TIMER_EVENTS_STATUS);
evt->event_handler(evt); evt->event_handler(evt);
@ -173,42 +166,55 @@ static int __cpuinit armada_370_xp_timer_setup(struct clock_event_device *evt)
u32 u; u32 u;
int cpu = smp_processor_id(); int cpu = smp_processor_id();
/* Use existing clock_event for cpu 0 */
if (!smp_processor_id())
return 0;
u = readl(local_base + TIMER_CTRL_OFF); u = readl(local_base + TIMER_CTRL_OFF);
if (timer25Mhz) if (timer25Mhz)
writel(u | TIMER0_25MHZ, local_base + TIMER_CTRL_OFF); writel(u | TIMER0_25MHZ, local_base + TIMER_CTRL_OFF);
else else
writel(u & ~TIMER0_25MHZ, local_base + TIMER_CTRL_OFF); writel(u & ~TIMER0_25MHZ, local_base + TIMER_CTRL_OFF);
evt->name = armada_370_xp_clkevt.name; evt->name = "armada_370_xp_per_cpu_tick",
evt->irq = armada_370_xp_clkevt.irq; evt->features = CLOCK_EVT_FEAT_ONESHOT |
evt->features = armada_370_xp_clkevt.features; CLOCK_EVT_FEAT_PERIODIC;
evt->shift = armada_370_xp_clkevt.shift; evt->shift = 32,
evt->rating = armada_370_xp_clkevt.rating, evt->rating = 300,
evt->set_next_event = armada_370_xp_clkevt_next_event, evt->set_next_event = armada_370_xp_clkevt_next_event,
evt->set_mode = armada_370_xp_clkevt_mode, evt->set_mode = armada_370_xp_clkevt_mode,
evt->irq = armada_370_xp_clkevt_irq;
evt->cpumask = cpumask_of(cpu); evt->cpumask = cpumask_of(cpu);
*__this_cpu_ptr(percpu_armada_370_xp_evt) = evt;
clockevents_config_and_register(evt, timer_clk, 1, 0xfffffffe); clockevents_config_and_register(evt, timer_clk, 1, 0xfffffffe);
enable_percpu_irq(evt->irq, 0); enable_percpu_irq(evt->irq, 0);
return 0; return 0;
} }
static void armada_370_xp_timer_stop(struct clock_event_device *evt) static void __cpuinit armada_370_xp_timer_stop(struct clock_event_device *evt)
{ {
evt->set_mode(CLOCK_EVT_MODE_UNUSED, evt); evt->set_mode(CLOCK_EVT_MODE_UNUSED, evt);
disable_percpu_irq(evt->irq); disable_percpu_irq(evt->irq);
} }
static struct local_timer_ops armada_370_xp_local_timer_ops __cpuinitdata = { static int __cpuinit armada_370_xp_timer_cpu_notify(struct notifier_block *self,
.setup = armada_370_xp_timer_setup, unsigned long action, void *hcpu)
.stop = armada_370_xp_timer_stop, {
/*
* Grab cpu pointer in each case to avoid spurious
* preemptible warnings
*/
switch (action & ~CPU_TASKS_FROZEN) {
case CPU_STARTING:
armada_370_xp_timer_setup(this_cpu_ptr(armada_370_xp_evt));
break;
case CPU_DYING:
armada_370_xp_timer_stop(this_cpu_ptr(armada_370_xp_evt));
break;
}
return NOTIFY_OK;
}
static struct notifier_block armada_370_xp_timer_cpu_nb __cpuinitdata = {
.notifier_call = armada_370_xp_timer_cpu_notify,
}; };
void __init armada_370_xp_timer_init(void) void __init armada_370_xp_timer_init(void)
@ -224,9 +230,6 @@ void __init armada_370_xp_timer_init(void)
if (of_find_property(np, "marvell,timer-25Mhz", NULL)) { if (of_find_property(np, "marvell,timer-25Mhz", NULL)) {
/* The fixed 25MHz timer is available so let's use it */ /* The fixed 25MHz timer is available so let's use it */
u = readl(local_base + TIMER_CTRL_OFF);
writel(u | TIMER0_25MHZ,
local_base + TIMER_CTRL_OFF);
u = readl(timer_base + TIMER_CTRL_OFF); u = readl(timer_base + TIMER_CTRL_OFF);
writel(u | TIMER0_25MHZ, writel(u | TIMER0_25MHZ,
timer_base + TIMER_CTRL_OFF); timer_base + TIMER_CTRL_OFF);
@ -236,9 +239,6 @@ void __init armada_370_xp_timer_init(void)
struct clk *clk = of_clk_get(np, 0); struct clk *clk = of_clk_get(np, 0);
WARN_ON(IS_ERR(clk)); WARN_ON(IS_ERR(clk));
rate = clk_get_rate(clk); rate = clk_get_rate(clk);
u = readl(local_base + TIMER_CTRL_OFF);
writel(u & ~(TIMER0_25MHZ),
local_base + TIMER_CTRL_OFF);
u = readl(timer_base + TIMER_CTRL_OFF); u = readl(timer_base + TIMER_CTRL_OFF);
writel(u & ~(TIMER0_25MHZ), writel(u & ~(TIMER0_25MHZ),
@ -252,7 +252,7 @@ void __init armada_370_xp_timer_init(void)
* We use timer 0 as clocksource, and private(local) timer 0 * We use timer 0 as clocksource, and private(local) timer 0
* for clockevents * for clockevents
*/ */
armada_370_xp_clkevt.irq = irq_of_parse_and_map(np, 4); armada_370_xp_clkevt_irq = irq_of_parse_and_map(np, 4);
ticks_per_jiffy = (timer_clk + HZ / 2) / HZ; ticks_per_jiffy = (timer_clk + HZ / 2) / HZ;
@ -277,26 +277,19 @@ void __init armada_370_xp_timer_init(void)
"armada_370_xp_clocksource", "armada_370_xp_clocksource",
timer_clk, 300, 32, clocksource_mmio_readl_down); timer_clk, 300, 32, clocksource_mmio_readl_down);
/* Register the clockevent on the private timer of CPU 0 */ register_cpu_notifier(&armada_370_xp_timer_cpu_nb);
armada_370_xp_clkevt.cpumask = cpumask_of(0);
clockevents_config_and_register(&armada_370_xp_clkevt,
timer_clk, 1, 0xfffffffe);
percpu_armada_370_xp_evt = alloc_percpu(struct clock_event_device *); armada_370_xp_evt = alloc_percpu(struct clock_event_device);
/* /*
* Setup clockevent timer (interrupt-driven). * Setup clockevent timer (interrupt-driven).
*/ */
*__this_cpu_ptr(percpu_armada_370_xp_evt) = &armada_370_xp_clkevt; res = request_percpu_irq(armada_370_xp_clkevt_irq,
res = request_percpu_irq(armada_370_xp_clkevt.irq,
armada_370_xp_timer_interrupt, armada_370_xp_timer_interrupt,
armada_370_xp_clkevt.name, "armada_370_xp_per_cpu_tick",
percpu_armada_370_xp_evt); armada_370_xp_evt);
if (!res) { /* Immediately configure the timer on the boot CPU */
enable_percpu_irq(armada_370_xp_clkevt.irq, 0); if (!res)
#ifdef CONFIG_LOCAL_TIMERS armada_370_xp_timer_setup(this_cpu_ptr(armada_370_xp_evt));
local_timer_register(&armada_370_xp_local_timer_ops);
#endif
}
} }