Re: [PATCH 1/2] ASoC: codecs: add TA5720 digital amplifier DT bindings

From	Andreas Dannenberg <dannenberg@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/2] ASoC: codecs: add TA5720 digital amplifier DT bindings
Date	2016-03-21 15:10 +0100
Message-ID	<rf8GT-6NS-9@gated-at.bofh.it> (permalink)
References	<rec0a-3uD-7@gated-at.bofh.it> <rec0a-3uD-5@gated-at.bofh.it> <rf7rs-5J5-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, Mar 21, 2016 at 07:45:55AM -0500, Rob Herring wrote:
> > +Optional properties:
> > +
> > +- dvdd-supply : phandle to a 3.3-V supply for the digital circuitry
> > +- pvdd-supply : phandle to a supply used for the Class-D amp and the analog
> > +- interrupts-extended : reference to a GPIO pin connected to the TAS5720
> > +                        FAULTZ pin for error reporting purposes
> 
> There's no point in having interrupts-extended here with a single 
> interrupt. It is fine to use, but really that's outside the scope of 
> this binding. So I'd just document "interrupts" here as that is what 
> most docs do, and that implies interrupts-extended could be used.

Hi Rob,
Good point. Will remove it for v2.

Regards,

--
Andreas Dannenberg
Texas Instruments Inc

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] ASoC: codecs: add TA5720 digital amplifier DT bindings Andreas Dannenberg <dannenberg@ti.com> - 2016-03-19 00:30 +0100
  Re: [PATCH 1/2] ASoC: codecs: add TA5720 digital amplifier DT  bindings Rob Herring <robh@kernel.org> - 2016-03-21 13:50 +0100
    Re: [PATCH 1/2] ASoC: codecs: add TA5720 digital amplifier DT  bindings Andreas Dannenberg <dannenberg@ti.com> - 2016-03-21 15:10 +0100

csiph-web