From: jamie@jamieiles.com (Jamie Iles)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 5/6] gpio: tegra: Dynamically allocate IRQ base, and support DT
Date: Thu, 5 Jan 2012 16:48:57 +0000 [thread overview]
Message-ID: <20120105164857.GE10530@page> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF17761F1683@HQMAIL01.nvidia.com>
On Thu, Jan 05, 2012 at 08:47:15AM -0800, Stephen Warren wrote:
> Jamie Iles wrote at Thursday, January 05, 2012 6:18 AM:
> > On Wed, Jan 04, 2012 at 11:39:37AM -0700, Stephen Warren wrote:
> > > Enhance the driver to dynamically allocate the base IRQ number, and
> > > create an IRQ domain for itself. The use of an IRQ domain ensures that
> > > any device tree node interrupts properties are correctly parsed.
> ...
> > > diff --git a/Documentation/devicetree/bindings/gpio/gpio_nvidia.txt
> ...
> > > +- #interrupt-cells : Should be 2.
> > > + The first cell is the GPIO number.
> > > + The second cell is used to specify flags:
> > > + bits[3:0] trigger type and level flags:
> > > + 1 = low-to-high edge triggered.
> > > + 2 = high-to-low edge triggered.
> > > + 4 = active high level-sensitive.
> > > + 8 = active low level-sensitive.
> > > + Valid combinations are 1, 2, 3, 4, 8.
> >
> > It looks to me like the tegra gpio driver can do IRQ_TYPE_EDGE_BOTH so I
> > would expect 12 to be a valid combination too no?
>
> I believe EDGE_BOTH is 3 (OR of edge low and edge high)
Doh! Yes, you're quite right. Sorry for the noise!
Jamie
next prev parent reply other threads:[~2012-01-05 16:48 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-04 18:39 [PATCH 1/6] ARM: tegra: Remove use of TEGRA_GPIO_TO_IRQ Stephen Warren
2012-01-04 18:39 ` [PATCH 2/6] dt: tegra gpio: Flesh out binding documentation Stephen Warren
2012-01-04 18:39 ` [PATCH 3/6] ARM: dt: tegra30.dtsi: Reformat gpio's interrupts property Stephen Warren
2012-01-04 19:50 ` Grant Likely
2012-01-04 18:39 ` [PATCH 4/6] ARM: dt: tegra30.dtsi: Add extra GPIO interrupt Stephen Warren
2012-01-04 18:39 ` [PATCH 5/6] gpio: tegra: Dynamically allocate IRQ base, and support DT Stephen Warren
2012-01-05 7:23 ` Thierry Reding
2012-01-05 17:47 ` Stephen Warren
2012-01-05 17:56 ` Grant Likely
2012-01-05 13:17 ` Jamie Iles
2012-01-05 16:47 ` Stephen Warren
2012-01-05 16:48 ` Jamie Iles [this message]
2012-01-04 18:39 ` [PATCH 6/6] gpio: tegra: Parameterize the number of banks Stephen Warren
2012-01-04 19:54 ` Rob Herring
2012-01-04 20:00 ` Stephen Warren
2012-01-04 22:00 ` Grant Likely
2012-01-05 17:24 ` Rob Herring
2012-01-13 20:55 ` Stephen Warren
2012-01-14 15:15 ` Rob Herring
2012-01-04 19:52 ` [PATCH 1/6] ARM: tegra: Remove use of TEGRA_GPIO_TO_IRQ Grant Likely
2012-01-24 8:34 ` Olof Johansson
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=20120105164857.GE10530@page \
--to=jamie@jamieiles.com \
--cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).