All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@linaro.org>
To: Ian Campbell <ian.campbell@citrix.com>
Cc: stefano.stabellini@eu.citrix.com, tim@xen.org,
	Pranavkumar Sawargaonkar <psawargaonkar@apm.com>,
	xen-devel@lists.xen.org
Subject: Re: [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch
Date: Mon, 02 Mar 2015 14:02:53 +0000	[thread overview]
Message-ID: <54F46D8D.2000400@linaro.org> (raw)
In-Reply-To: <1425304392.21151.29.camel@citrix.com>

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.

Regards,

-- 
Julien Grall

  reply	other threads:[~2015-03-02 14:02 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-19 15:39 [PATCH 0/2] xen: arm: warn for gic and arch timer misconfiguration Ian Campbell
2015-02-19 15:24 ` [PATCH 1/2] xen: arm: log warning for interrupt configuration mismatch Ian Campbell
2015-02-19 15:45   ` Julien Grall
2015-02-28 22:12   ` Julien Grall
2015-03-02 11:12     ` Ian Campbell
2015-03-02 12:56       ` Julien Grall
2015-03-02 13:42         ` Ian Campbell
2015-03-02 13:48           ` Julien Grall
2015-03-02 13:53             ` Ian Campbell
2015-03-02 14:02               ` Julien Grall [this message]
2015-03-02 14:41                 ` Ian Campbell
2015-03-02 17:01         ` Ian Campbell
2015-03-03 12:20           ` Julien Grall
2015-02-19 15:24 ` [PATCH 2/2] xen: arm: Warn if timer interrupts are not level triggered Ian Campbell
2015-02-19 15:41   ` Julien Grall
2015-02-19 16:10     ` Ian Campbell
2015-02-19 16:20       ` Julien Grall
2015-02-25 14:36         ` Ian Campbell
2015-02-28 22:20   ` Julien Grall
2015-03-02 11:13     ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54F46D8D.2000400@linaro.org \
    --to=julien.grall@linaro.org \
    --cc=ian.campbell@citrix.com \
    --cc=psawargaonkar@apm.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.