Skip to content
Snippets Groups Projects
Commit be076b83 authored by Artur Rojek's avatar Artur Rojek Committed by Frieder Schrempf
Browse files

irqchip/jcore-aic, clocksource/drivers/jcore: Fix jcore-pit interrupt request


[ Upstream commit d7e3fd658248f257006227285095d190e70ee73a ]

The jcore-aic irqchip does not have separate interrupt numbers reserved for
cpu-local vs global interrupts. Therefore the device drivers need to
request the given interrupt as per CPU interrupt.

69a9dcbd ("clocksource/drivers/jcore: Use request_percpu_irq()")
converted the clocksource driver over to request_percpu_irq(), but failed
to do add all the required changes, resulting in a failure to register PIT
interrupts.

Fix this by:

 1) Explicitly mark the interrupt via irq_set_percpu_devid() in
    jcore_pit_init().

 2) Enable and disable the per CPU interrupt in the CPU hotplug callbacks.

 3) Pass the correct per-cpu cookie to the irq handler by using
    handle_percpu_devid_irq() instead of handle_percpu_irq() in
    handle_jcore_irq().

[ tglx: Massage change log ]

Fixes: 69a9dcbd ("clocksource/drivers/jcore: Use request_percpu_irq()")
Signed-off-by: default avatarArtur Rojek <contact@artur-rojek.eu>
Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Acked-by: default avatarUros Bizjak <ubizjak@gmail.com>
Link: https://lore.kernel.org/all/20250216175545.35079-3-contact@artur-rojek.eu


Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 2fab6eb0
No related branches found
No related tags found
1 merge request!192🤖 Sync Bot: Update v6.12-ktn to Latest Stable Kernel (v6.12.17)
...@@ -114,6 +114,18 @@ static int jcore_pit_local_init(unsigned cpu) ...@@ -114,6 +114,18 @@ static int jcore_pit_local_init(unsigned cpu)
pit->periodic_delta = DIV_ROUND_CLOSEST(NSEC_PER_SEC, HZ * buspd); pit->periodic_delta = DIV_ROUND_CLOSEST(NSEC_PER_SEC, HZ * buspd);
clockevents_config_and_register(&pit->ced, freq, 1, ULONG_MAX); clockevents_config_and_register(&pit->ced, freq, 1, ULONG_MAX);
enable_percpu_irq(pit->ced.irq, IRQ_TYPE_NONE);
return 0;
}
static int jcore_pit_local_teardown(unsigned cpu)
{
struct jcore_pit *pit = this_cpu_ptr(jcore_pit_percpu);
pr_info("Local J-Core PIT teardown on cpu %u\n", cpu);
disable_percpu_irq(pit->ced.irq);
return 0; return 0;
} }
...@@ -168,6 +180,7 @@ static int __init jcore_pit_init(struct device_node *node) ...@@ -168,6 +180,7 @@ static int __init jcore_pit_init(struct device_node *node)
return -ENOMEM; return -ENOMEM;
} }
irq_set_percpu_devid(pit_irq);
err = request_percpu_irq(pit_irq, jcore_timer_interrupt, err = request_percpu_irq(pit_irq, jcore_timer_interrupt,
"jcore_pit", jcore_pit_percpu); "jcore_pit", jcore_pit_percpu);
if (err) { if (err) {
...@@ -237,7 +250,7 @@ static int __init jcore_pit_init(struct device_node *node) ...@@ -237,7 +250,7 @@ static int __init jcore_pit_init(struct device_node *node)
cpuhp_setup_state(CPUHP_AP_JCORE_TIMER_STARTING, cpuhp_setup_state(CPUHP_AP_JCORE_TIMER_STARTING,
"clockevents/jcore:starting", "clockevents/jcore:starting",
jcore_pit_local_init, NULL); jcore_pit_local_init, jcore_pit_local_teardown);
return 0; return 0;
} }
......
...@@ -38,7 +38,7 @@ static struct irq_chip jcore_aic; ...@@ -38,7 +38,7 @@ static struct irq_chip jcore_aic;
static void handle_jcore_irq(struct irq_desc *desc) static void handle_jcore_irq(struct irq_desc *desc)
{ {
if (irqd_is_per_cpu(irq_desc_get_irq_data(desc))) if (irqd_is_per_cpu(irq_desc_get_irq_data(desc)))
handle_percpu_irq(desc); handle_percpu_devid_irq(desc);
else else
handle_simple_irq(desc); handle_simple_irq(desc);
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment