* [PATCH 2/2] ASoC: tas5756m: Add DT binding document [not found] <20200118092315.10384-1-charles-antoine.couret@essensium.com> @ 2020-01-18 9:23 ` Charles-Antoine Couret 2020-01-22 16:15 ` Rob Herring 2020-01-22 16:16 ` Rob Herring 0 siblings, 2 replies; 5+ messages in thread From: Charles-Antoine Couret @ 2020-01-18 9:23 UTC (permalink / raw) To: alsa-devel; +Cc: Charles-Antoine Couret, devicetree Document the bindings for the tas5756m driver. --- .../devicetree/bindings/sound/tas5756m.txt | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.txt diff --git a/Documentation/devicetree/bindings/sound/tas5756m.txt b/Documentation/devicetree/bindings/sound/tas5756m.txt new file mode 100644 index 000000000000..293eaf20d008 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/tas5756m.txt @@ -0,0 +1,25 @@ +TAS5756M audio CODEC + +Required properties: + + - compatible: should be one of the following: + - "ti,tas5756m" + - "ti,tas5754m" + - reg: the I2C address of the device for I2C. + +Optional properties: + + - mute-gpio : GPIO wired to the mute pin. + - hybridflow : an integer between 1 and 9 to select the HybridFlow program. + if not supplied default DSP program is used. + +Example: + + tas5756m: tas5756m@4c { + compatible = "ti,tas5756m"; + reg = <0x4c>; + #sound-dai-cells = <0>; + + hybridflow = <6>; + mute-gpio = <&gpio1 11 GPIO_ACTIVE_LOW>; + }; -- 2.24.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] ASoC: tas5756m: Add DT binding document 2020-01-18 9:23 ` [PATCH 2/2] ASoC: tas5756m: Add DT binding document Charles-Antoine Couret @ 2020-01-22 16:15 ` Rob Herring 2020-01-22 21:23 ` Couret Charles-Antoine 2020-01-22 16:16 ` Rob Herring 1 sibling, 1 reply; 5+ messages in thread From: Rob Herring @ 2020-01-22 16:15 UTC (permalink / raw) To: Charles-Antoine Couret; +Cc: alsa-devel, devicetree On Sat, Jan 18, 2020 at 10:23:15AM +0100, Charles-Antoine Couret wrote: > Document the bindings for the tas5756m driver. > --- > .../devicetree/bindings/sound/tas5756m.txt | 25 +++++++++++++++++++ > 1 file changed, 25 insertions(+) > create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.txt Can you make this a DT schema. > > diff --git a/Documentation/devicetree/bindings/sound/tas5756m.txt b/Documentation/devicetree/bindings/sound/tas5756m.txt > new file mode 100644 > index 000000000000..293eaf20d008 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/tas5756m.txt > @@ -0,0 +1,25 @@ > +TAS5756M audio CODEC > + > +Required properties: > + > + - compatible: should be one of the following: > + - "ti,tas5756m" > + - "ti,tas5754m" > + - reg: the I2C address of the device for I2C. > + > +Optional properties: > + > + - mute-gpio : GPIO wired to the mute pin. mute-gpios is the preferred form. > + - hybridflow : an integer between 1 and 9 to select the HybridFlow program. > + if not supplied default DSP program is used. Needs a vendor prefix. Is this something fixed for the life of a board or something users would want to change at runtime? > + > +Example: > + > + tas5756m: tas5756m@4c { > + compatible = "ti,tas5756m"; > + reg = <0x4c>; > + #sound-dai-cells = <0>; Not documented. > + > + hybridflow = <6>; > + mute-gpio = <&gpio1 11 GPIO_ACTIVE_LOW>; > + }; > -- > 2.24.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] ASoC: tas5756m: Add DT binding document 2020-01-22 16:15 ` Rob Herring @ 2020-01-22 21:23 ` Couret Charles-Antoine 0 siblings, 0 replies; 5+ messages in thread From: Couret Charles-Antoine @ 2020-01-22 21:23 UTC (permalink / raw) To: Rob Herring; +Cc: alsa-devel, devicetree Le 22/01/2020 à 17:15, Rob Herring a écrit : > Can you make this a DT schema. Ok, thank you for the notice. >> +Optional properties: >> + >> + - mute-gpio : GPIO wired to the mute pin. > mute-gpios is the preferred form. I will fix it. >> + - hybridflow : an integer between 1 and 9 to select the HybridFlow program. >> + if not supplied default DSP program is used. > Needs a vendor prefix. > > Is this something fixed for the life of a board or something users would > want to change at runtime? This shouldn't be changed on runtime. It is possible but it does not make sense to me. Firstly we have to stop the device before switching the hybridflow. Then it changes the available ALSA commands at runtime. The hybridflow must be chosen to match requirements of the speaker behind (like 2.1, woofer or tweeter). Thank you for your advices. Regards, Charles-Antoine Couret ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] ASoC: tas5756m: Add DT binding document 2020-01-18 9:23 ` [PATCH 2/2] ASoC: tas5756m: Add DT binding document Charles-Antoine Couret 2020-01-22 16:15 ` Rob Herring @ 2020-01-22 16:16 ` Rob Herring 1 sibling, 0 replies; 5+ messages in thread From: Rob Herring @ 2020-01-22 16:16 UTC (permalink / raw) To: Charles-Antoine Couret; +Cc: alsa-devel, devicetree On Sat, Jan 18, 2020 at 10:23:15AM +0100, Charles-Antoine Couret wrote: > Document the bindings for the tas5756m driver. Also, checkpatch.pl reports your S-o-b is missing. > --- > .../devicetree/bindings/sound/tas5756m.txt | 25 +++++++++++++++++++ > 1 file changed, 25 insertions(+) > create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.txt > > diff --git a/Documentation/devicetree/bindings/sound/tas5756m.txt b/Documentation/devicetree/bindings/sound/tas5756m.txt > new file mode 100644 > index 000000000000..293eaf20d008 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/tas5756m.txt > @@ -0,0 +1,25 @@ > +TAS5756M audio CODEC > + > +Required properties: > + > + - compatible: should be one of the following: > + - "ti,tas5756m" > + - "ti,tas5754m" > + - reg: the I2C address of the device for I2C. > + > +Optional properties: > + > + - mute-gpio : GPIO wired to the mute pin. > + - hybridflow : an integer between 1 and 9 to select the HybridFlow program. > + if not supplied default DSP program is used. > + > +Example: > + > + tas5756m: tas5756m@4c { > + compatible = "ti,tas5756m"; > + reg = <0x4c>; > + #sound-dai-cells = <0>; > + > + hybridflow = <6>; > + mute-gpio = <&gpio1 11 GPIO_ACTIVE_LOW>; > + }; > -- > 2.24.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <20200117164331.87442-1-charles-antoine.couret@essensium.com>]
* [PATCH 2/2] ASoC: tas5756m: Add DT binding document [not found] <20200117164331.87442-1-charles-antoine.couret@essensium.com> @ 2020-01-17 16:43 ` Charles-Antoine Couret 0 siblings, 0 replies; 5+ messages in thread From: Charles-Antoine Couret @ 2020-01-17 16:43 UTC (permalink / raw) To: alsa-devel; +Cc: Charles-Antoine Couret, devicetree Document the bindings for the tas5756m driver. --- .../devicetree/bindings/sound/tas5756m.txt | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/tas5756m.txt diff --git a/Documentation/devicetree/bindings/sound/tas5756m.txt b/Documentation/devicetree/bindings/sound/tas5756m.txt new file mode 100644 index 000000000000..293eaf20d008 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/tas5756m.txt @@ -0,0 +1,25 @@ +TAS5756M audio CODEC + +Required properties: + + - compatible: should be one of the following: + - "ti,tas5756m" + - "ti,tas5754m" + - reg: the I2C address of the device for I2C. + +Optional properties: + + - mute-gpio : GPIO wired to the mute pin. + - hybridflow : an integer between 1 and 9 to select the HybridFlow program. + if not supplied default DSP program is used. + +Example: + + tas5756m: tas5756m@4c { + compatible = "ti,tas5756m"; + reg = <0x4c>; + #sound-dai-cells = <0>; + + hybridflow = <6>; + mute-gpio = <&gpio1 11 GPIO_ACTIVE_LOW>; + }; -- 2.24.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-01-22 21:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20200118092315.10384-1-charles-antoine.couret@essensium.com>
2020-01-18 9:23 ` [PATCH 2/2] ASoC: tas5756m: Add DT binding document Charles-Antoine Couret
2020-01-22 16:15 ` Rob Herring
2020-01-22 21:23 ` Couret Charles-Antoine
2020-01-22 16:16 ` Rob Herring
[not found] <20200117164331.87442-1-charles-antoine.couret@essensium.com>
2020-01-17 16:43 ` Charles-Antoine Couret
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).