Re: [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch

[Ian Campbell <ian.campbell@citrix.com>]

On Mon, 2015-03-02 at 14:02 +0000, Julien Grall wrote:
> On 02/03/15 13:53, Ian Campbell wrote:
> > On Mon, 2015-03-02 at 13:48 +0000, Julien Grall wrote:
> >> Given that, I was wondering if it would be useful to only store
> >> EDGE/LEVEL in the desc->arch.type and, therefore, ignoring
> >> rising/falling and low/high.
> > 
> > If we don't ever need to know about the polarity then sure. It would
> > also get a DT-ism out of a non-DT specific place, which will make the
> > ACPI folks happy I bet.
> 
> That would be required if we implement an interrupt controller or UART
> driver which need to know if it's a rising/falling edge or low/high
> level interrupt.
> 
> Although I don't see any UART in Linux using the IRQ type. So it would
> only be an interrupt controller.
>
> I guess we could go on this way (i.e only keeping track of level/edge)
> for now.
> 
> I will give a look after my device passthrough series.

Sounds good, thanks.

> 
> Regards,
>