|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [RFC 10/29] xen/arm: Add helpers to retrieve an interrupt description from the device tree
On Mon, 2013-04-29 at 00:01 +0100, Julien Grall wrote:
> diff --git a/xen/include/xen/irq.h b/xen/include/xen/irq.h
> index 7386358..42dc172 100644
> --- a/xen/include/xen/irq.h
> +++ b/xen/include/xen/irq.h
> @@ -33,6 +33,31 @@ struct irqaction {
> #define NEVER_ASSIGN_IRQ (-2)
> #define FREE_TO_ASSIGN_IRQ (-3)
>
> +/**
> + * IRQ line type.
> + *
> + * IRQ_TYPE_NONE - default, unspecified type
> + * IRQ_TYPE_EDGE_RISING - rising edge triggered
> + * IRQ_TYPE_EDGE_FALLING - falling edge triggered
> + * IRQ_TYPE_EDGE_BOTH - rising and falling edge triggered
> + * IRQ_TYPE_LEVEL_HIGH - high level triggered
> + * IRQ_TYPE_LEVEL_LOW - low level triggered
> + * IRQ_TYPE_LEVEL_MASK - Mask to filter out the level bits
> + * IRQ_TYPE_SENSE_MASK - Mask for all the above bits
> + */
> +#define IRQ_TYPE_NONE 0x00000000
> +#define IRQ_TYPE_EDGE_RISING 0x00000001
> +#define IRQ_TYPE_EDGE_FALLING 0x00000002
> +#define IRQ_TYPE_EDGE_BOTH (IRQ_TYPE_EDGE_FALLING |
> IRQ_TYPE_EDGE_RISING)
> +#define IRQ_TYPE_LEVEL_HIGH 0x00000004
> +#define IRQ_TYPE_LEVEL_LOW 0x00000008
> +#define IRQ_TYPE_LEVEL_MASK (IRQ_TYPE_LEVEL_LOW | IRQ_TYPE_LEVEL_HIGH)
> +#define IRQ_TYPE_SENSE_MASK 0x0000000f
> +
> +/* If type == IRQ_TYPE_NONE, assume we use level triggered */
> +#define irq_is_level_trigger(irq) \
> + (((irq)->type & IRQ_TYPE_LEVEL_MASK) || ((irq)->type == IRQ_TYPE_NONE))
What is the type of irq here? None of the structs in this irq.h have a
type member, so I must be looking in the wrong place.
Or is it struct dt_irq? If so then can we put these in a DT (or ARM)
specific header and add a DT_ prefix, rather than pollute the non-DT
headers, e.g. these defines have no meaning on x86 AFAICT.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |