On 24/06/14 16:44, Nicolas Pitre wrote:
On Tue, 24 Jun 2014, Daniel Thompson wrote:
Currently enable_fiq/disable_fiq use a simple offset to convert an IRQ virq into a FIQ virq. This is too inflexible for multi-platform kernels and makes runtime error checking impossible.
We solve this by introducing a flexible mapping that allows interrupt controllers that support FIQ to register those mappings. This, in turn, makes it much possible for drivers in DT kernels to install FIQ handlers without knowing anything about the interrupt controller.
Signed-off-by: Daniel Thompson daniel.thompson@linaro.org Cc: Russell King linux@arm.linux.org.uk Cc: Fabio Estevam festevam@gmail.com Cc: Nicolas Pitre nico@linaro.org
arch/arm/include/asm/fiq.h | 7 +++ arch/arm/kernel/fiq.c | 103 ++++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 108 insertions(+), 2 deletions(-)
[...]
+bool has_fiq(int fiq) +{
- struct fiq_data *data = lookup_fiq_data(fiq);
- if (data)
return true;- if (fiq_start == -1)
return false;- return fiq > fiq_start;
Shouldn't this be fiq >= fiq_start ?
Absolutely! Will fix that shortly.
Thanks
Daniel.