Commit c389d701 authored by Fu Wei's avatar Fu Wei Committed by Mark Rutland

clocksource: arm_arch_timer: split MMIO timer probing.

Currently the code to probe MMIO architected timers mixes DT parsing with
actual poking of hardware. This makes the code harder than necessary to
understand, and makes it difficult to add support for probing via ACPI.

This patch splits the DT parsing from HW probing. The DT parsing now
lives in arch_timer_mem_of_init(), which fills in an arch_timer_mem
structure that it hands to probing functions that can be reused for ACPI
support.

Since the rate detection logic will be slight different when using ACPI,
the probing is performed as a number of steps. This results in more code
for the moment, and some arguably redundant work, but simplifies matters
considerably when ACPI support is added.
Signed-off-by: default avatarFu Wei <fu.wei@linaro.org>
[Mark: refactor the probing split]
Signed-off-by: default avatarMark Rutland <mark.rutland@arm.com>
parent b3251b8f
...@@ -1158,7 +1158,7 @@ static int __init arch_timer_of_init(struct device_node *np) ...@@ -1158,7 +1158,7 @@ static int __init arch_timer_of_init(struct device_node *np)
arch_timer_kvm_info.virtual_irq = arch_timer_ppi[ARCH_TIMER_VIRT_PPI]; arch_timer_kvm_info.virtual_irq = arch_timer_ppi[ARCH_TIMER_VIRT_PPI];
rate = arch_timer_get_cntfrq; rate = arch_timer_get_cntfrq();
arch_timer_of_configure_rate(rate, np); arch_timer_of_configure_rate(rate, np);
arch_timer_c3stop = !of_property_read_bool(np, "always-on"); arch_timer_c3stop = !of_property_read_bool(np, "always-on");
...@@ -1197,18 +1197,38 @@ static int __init arch_timer_of_init(struct device_node *np) ...@@ -1197,18 +1197,38 @@ static int __init arch_timer_of_init(struct device_node *np)
CLOCKSOURCE_OF_DECLARE(armv7_arch_timer, "arm,armv7-timer", arch_timer_of_init); CLOCKSOURCE_OF_DECLARE(armv7_arch_timer, "arm,armv7-timer", arch_timer_of_init);
CLOCKSOURCE_OF_DECLARE(armv8_arch_timer, "arm,armv8-timer", arch_timer_of_init); CLOCKSOURCE_OF_DECLARE(armv8_arch_timer, "arm,armv8-timer", arch_timer_of_init);
static int __init arch_timer_mem_init(struct device_node *np) static u32 __init
arch_timer_mem_frame_get_cntfrq(struct arch_timer_mem_frame *frame)
{ {
struct device_node *frame, *best_frame = NULL; void __iomem *base;
void __iomem *cntctlbase, *base; u32 rate;
unsigned int irq, ret = -EINVAL;
u32 cnttidr, rate;
arch_timers_present |= ARCH_TIMER_TYPE_MEM; base = ioremap(frame->cntbase, frame->size);
cntctlbase = of_iomap(np, 0); if (!base) {
pr_err("Unable to map frame @ %pa\n", &frame->cntbase);
return 0;
}
rate = readl_relaxed(frame + CNTFRQ);
iounmap(frame);
return rate;
}
static struct arch_timer_mem_frame * __init
arch_timer_mem_find_best_frame(struct arch_timer_mem *timer_mem)
{
struct arch_timer_mem_frame *frame, *best_frame = NULL;
void __iomem *cntctlbase;
u32 cnttidr;
int i;
cntctlbase = ioremap(timer_mem->cntctlbase, timer_mem->size);
if (!cntctlbase) { if (!cntctlbase) {
pr_err("Can't find CNTCTLBase\n"); pr_err("Can't map CNTCTLBase @ %pa\n",
return -ENXIO; &timer_mem->cntctlbase);
return NULL;
} }
cnttidr = readl_relaxed(cntctlbase + CNTTIDR); cnttidr = readl_relaxed(cntctlbase + CNTTIDR);
...@@ -1217,25 +1237,20 @@ static int __init arch_timer_mem_init(struct device_node *np) ...@@ -1217,25 +1237,20 @@ static int __init arch_timer_mem_init(struct device_node *np)
* Try to find a virtual capable frame. Otherwise fall back to a * Try to find a virtual capable frame. Otherwise fall back to a
* physical capable frame. * physical capable frame.
*/ */
for_each_available_child_of_node(np, frame) { for (i = 0; i < ARCH_TIMER_MEM_MAX_FRAMES; i++) {
int n; u32 cntacr = CNTACR_RFRQ | CNTACR_RWPT | CNTACR_RPCT |
u32 cntacr; CNTACR_RWVT | CNTACR_RVOFF | CNTACR_RVCT;
if (of_property_read_u32(frame, "frame-number", &n)) { frame = &timer_mem->frame[i];
pr_err("Missing frame-number\n"); if (!frame->valid)
of_node_put(frame); continue;
goto out;
}
/* Try enabling everything, and see what sticks */ /* Try enabling everything, and see what sticks */
cntacr = CNTACR_RFRQ | CNTACR_RWPT | CNTACR_RPCT | writel_relaxed(cntacr, cntctlbase + CNTACR(i));
CNTACR_RWVT | CNTACR_RVOFF | CNTACR_RVCT; cntacr = readl_relaxed(cntctlbase + CNTACR(i));
writel_relaxed(cntacr, cntctlbase + CNTACR(n));
cntacr = readl_relaxed(cntctlbase + CNTACR(n));
if ((cnttidr & CNTTIDR_VIRT(n)) && if ((cnttidr & CNTTIDR_VIRT(i)) &&
!(~cntacr & (CNTACR_RWVT | CNTACR_RVCT))) { !(~cntacr & (CNTACR_RWVT | CNTACR_RVCT))) {
of_node_put(best_frame);
best_frame = frame; best_frame = frame;
arch_timer_mem_use_virtual = true; arch_timer_mem_use_virtual = true;
break; break;
...@@ -1244,45 +1259,131 @@ static int __init arch_timer_mem_init(struct device_node *np) ...@@ -1244,45 +1259,131 @@ static int __init arch_timer_mem_init(struct device_node *np)
if (~cntacr & (CNTACR_RWPT | CNTACR_RPCT)) if (~cntacr & (CNTACR_RWPT | CNTACR_RPCT))
continue; continue;
of_node_put(best_frame); best_frame = frame;
best_frame = of_node_get(frame);
} }
ret= -ENXIO; iounmap(cntctlbase);
base = arch_counter_base = of_io_request_and_map(best_frame, 0,
"arch_mem_timer"); if (!best_frame)
if (IS_ERR(base)) { pr_err("Unable to find a suitable frame in timer @ %pa\n",
pr_err("Can't map frame's registers\n"); &timer_mem->cntctlbase);
goto out;
} return frame;
}
static int __init
arch_timer_mem_frame_register(struct arch_timer_mem_frame *frame)
{
void __iomem *base;
int ret, irq = 0;
if (arch_timer_mem_use_virtual) if (arch_timer_mem_use_virtual)
irq = irq_of_parse_and_map(best_frame, ARCH_TIMER_VIRT_SPI); irq = frame->virt_irq;
else else
irq = irq_of_parse_and_map(best_frame, ARCH_TIMER_PHYS_SPI); irq = frame->phys_irq;
ret = -EINVAL;
if (!irq) { if (!irq) {
pr_err("Frame missing %s irq.\n", pr_err("Frame missing %s irq.\n",
arch_timer_mem_use_virtual ? "virt" : "phys"); arch_timer_mem_use_virtual ? "virt" : "phys");
goto out; return -EINVAL;
}
if (!request_mem_region(frame->cntbase, frame->size,
"arch_mem_timer"))
return -EBUSY;
base = ioremap(frame->cntbase, frame->size);
if (!base) {
pr_err("Can't map frame's registers\n");
return -ENXIO;
} }
rate = readl(base + CNTFRQ);
arch_timer_of_configure_rate(rate, np);
ret = arch_timer_mem_register(base, irq); ret = arch_timer_mem_register(base, irq);
if (ret) if (ret) {
iounmap(base);
return ret;
}
arch_counter_base = base;
arch_timers_present |= ARCH_TIMER_TYPE_MEM;
return 0;
}
static int __init arch_timer_mem_of_init(struct device_node *np)
{
struct arch_timer_mem *timer_mem;
struct arch_timer_mem_frame *frame;
struct device_node *frame_node;
struct resource res;
int ret = -EINVAL;
u32 rate;
timer_mem = kzalloc(sizeof(*timer_mem), GFP_KERNEL);
if (!timer_mem)
return -ENOMEM;
if (of_address_to_resource(np, 0, &res))
goto out; goto out;
timer_mem->cntctlbase = res.start;
timer_mem->size = resource_size(&res);
if (!arch_timer_needs_of_probing()) for_each_available_child_of_node(np, frame_node) {
u32 n;
struct arch_timer_mem_frame *frame;
if (of_property_read_u32(frame_node, "frame-number", &n)) {
pr_err(FW_BUG "Missing frame-number.\n");
of_node_put(frame_node);
goto out;
}
if (n >= ARCH_TIMER_MEM_MAX_FRAMES) {
pr_err(FW_BUG "Wrong frame-number, only 0-%u are permitted.\n",
ARCH_TIMER_MEM_MAX_FRAMES - 1);
of_node_put(frame_node);
goto out;
}
frame = &timer_mem->frame[n];
if (frame->valid) {
pr_err(FW_BUG "Duplicated frame-number.\n");
of_node_put(frame_node);
goto out;
}
if (of_address_to_resource(frame_node, 0, &res)) {
of_node_put(frame_node);
goto out;
}
frame->cntbase = res.start;
frame->size = resource_size(&res);
frame->virt_irq = irq_of_parse_and_map(frame_node,
ARCH_TIMER_VIRT_SPI);
frame->phys_irq = irq_of_parse_and_map(frame_node,
ARCH_TIMER_PHYS_SPI);
frame->valid = true;
}
frame = arch_timer_mem_find_best_frame(timer_mem);
if (!frame) {
ret = -EINVAL;
goto out;
}
rate = arch_timer_mem_frame_get_cntfrq(frame);
arch_timer_of_configure_rate(rate, np);
ret = arch_timer_mem_frame_register(frame);
if (!ret && !arch_timer_needs_of_probing())
ret = arch_timer_common_init(); ret = arch_timer_common_init();
out: out:
iounmap(cntctlbase); kfree(timer_mem);
of_node_put(best_frame);
return ret; return ret;
} }
CLOCKSOURCE_OF_DECLARE(armv7_arch_timer_mem, "arm,armv7-timer-mem", CLOCKSOURCE_OF_DECLARE(armv7_arch_timer_mem, "arm,armv7-timer-mem",
arch_timer_mem_init); arch_timer_mem_of_init);
#ifdef CONFIG_ACPI #ifdef CONFIG_ACPI
static int __init map_generic_timer_interrupt(u32 interrupt, u32 flags) static int __init map_generic_timer_interrupt(u32 interrupt, u32 flags)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment