From: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
To: Mark Brown <broonie@kernel.org>
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
Samuel Ortiz <sameo@linux.intel.com>,
Sebastien Jan <sjan@baylibre.com>,
patches@opensource.wolfsonmicro.com,
Lee Jones <lee.jones@linaro.org>
Subject: Re: [PATCH v2] mfd: arizona: Add DT binding for irq-gpios property
Date: Tue, 3 Nov 2015 16:52:47 +0000 [thread overview]
Message-ID: <20151103165247.GC2549@ck-lbox> (raw)
In-Reply-To: <20151103150450.GP20228@sirena.org.uk>
On Tue, Nov 03, 2015 at 03:04:50PM +0000, Mark Brown wrote:
> On Tue, Nov 03, 2015 at 03:10:06PM +0100, Sebastien Jan wrote:
>
> > + - irq-gpios : GPIO specifier for the GPIO connected to /IRQ
>
> The platform data this is providing is essentially a workaround for the
> lack of an irq_to_gpio() function and the use of it is a workaround for
> the lack of level triggered IRQ emulation in genirq... do we really
> want this in the ABI (especially as a generic thing)?
Personally I would rather not add this to the DT unless we really
had to.
Also are you sure this is necessary on your system? Specifically
you only need this entry if the CODEC is connected to an IRQ line
that only supports edge triggered IRQs.
Thanks,
Charles
next prev parent reply other threads:[~2015-11-03 16:52 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-03 14:10 [PATCH v2] mfd: arizona: Add DT binding for irq-gpios property Sebastien Jan
[not found] ` <1446559806-3315-1-git-send-email-sjan-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2015-11-03 15:04 ` Mark Brown
2015-11-03 16:52 ` Charles Keepax [this message]
2015-11-05 6:08 ` Sebastien Jan
[not found] ` <563AF241.5030404-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
2015-11-06 9:49 ` Lee Jones
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=20151103165247.GC2549@ck-lbox \
--to=ckeepax@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=lee.jones@linaro.org \
--cc=patches@opensource.wolfsonmicro.com \
--cc=sameo@linux.intel.com \
--cc=sjan@baylibre.com \
/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).