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 <[email protected]>
> Cc: Russell King <[email protected]>
> Cc: Fabio Estevam <[email protected]>
> Cc: Nicolas Pitre <[email protected]>
> ---
> 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 ?
Other than that...
Acked-by: Nicolas Pitre <[email protected]>
Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
