Linux Sound subsystem development
 help / color / mirror / Atom feed
* [PATCH] ASoC: dt-bindings: tse850: Convert to DT schema
@ 2026-08-12 23:33 Bhargav Joshi
  2026-08-13 10:51 ` Peter Rosin
  0 siblings, 1 reply; 3+ messages in thread
From: Bhargav Joshi @ 2026-08-12 23:33 UTC (permalink / raw)
  To: Claudiu Beznea, Andrei Simion, Liam Girdwood, Mark Brown,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley, Peter Rosin
  Cc: linux-sound, devicetree, linux-kernel, goledhruva, m-chawdhry,
	daniel.baluta, simona.toaca, j.bhargav.u

Convert Axentia TSE-850 audio complex from text to DT schema. No
funcational changes.

Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
---
 .../bindings/sound/axentia,tse850-pcm5142.txt      |  92 ------------------
 .../bindings/sound/axentia,tse850-pcm5142.yaml     | 104 +++++++++++++++++++++
 2 files changed, 104 insertions(+), 92 deletions(-)

diff --git a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt
deleted file mode 100644
index b6cc5f6f78c2..000000000000
--- a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt
+++ /dev/null
@@ -1,92 +0,0 @@
-Devicetree bindings for the Axentia TSE-850 audio complex
-
-Required properties:
-  - compatible: "axentia,tse850-pcm5142"
-  - axentia,cpu-dai: The phandle of the cpu dai.
-  - axentia,audio-codec: The phandle of the PCM5142 codec.
-  - axentia,add-gpios: gpio specifier that controls the mixer.
-  - axentia,loop1-gpios: gpio specifier that controls loop relays on channel 1.
-  - axentia,loop2-gpios: gpio specifier that controls loop relays on channel 2.
-  - axentia,ana-supply: Regulator that supplies the output amplifier. Must
-    support voltages in the 2V - 20V range, in 1V steps.
-
-The schematics explaining the gpios are as follows:
-
-               loop1 relays
-   IN1 +---o  +------------+  o---+ OUT1
-            \                /
-             +              +
-             |   /          |
-             +--o  +--.     |
-             |  add   |     |
-             |        V     |
-             |      .---.   |
-   DAC +----------->|Sum|---+
-             |      '---'   |
-             |              |
-             +              +
-
-   IN2 +---o--+------------+--o---+ OUT2
-               loop2 relays
-
-The 'loop1' gpio pin controls two relays, which are either in loop position,
-meaning that input and output are directly connected, or they are in mixer
-position, meaning that the signal is passed through the 'Sum' mixer. Similarly
-for 'loop2'.
-
-In the above, the 'loop1' relays are inactive, thus feeding IN1 to the mixer
-(if 'add' is active) and feeding the mixer output to OUT1. The 'loop2' relays
-are active, short-cutting the TSE-850 from channel 2. IN1, IN2, OUT1 and OUT2
-are TSE-850 connectors and DAC is the PCB name of the (filtered) output from
-the PCM5142 codec.
-
-Example:
-
-	&ssc0 {
-		#sound-dai-cells = <0>;
-
-	};
-
-	&i2c {
-		codec: pcm5142@4c {
-			compatible = "ti,pcm5142";
-
-			reg = <0x4c>;
-
-			AVDD-supply = <&reg_3v3>;
-			DVDD-supply = <&reg_3v3>;
-			CPVDD-supply = <&reg_3v3>;
-
-			clocks = <&sck>;
-
-			pll-in = <3>;
-			pll-out = <6>;
-		};
-	};
-
-	ana: ana-reg {
-		compatible = "pwm-regulator";
-
-		regulator-name = "ANA";
-
-		pwms = <&pwm0 2 1000 PWM_POLARITY_INVERTED>;
-		pwm-dutycycle-unit = <1000>;
-		pwm-dutycycle-range = <100 1000>;
-
-		regulator-min-microvolt = <2000000>;
-		regulator-max-microvolt = <20000000>;
-		regulator-ramp-delay = <1000>;
-	};
-
-	sound {
-		compatible = "axentia,tse850-pcm5142";
-
-		axentia,cpu-dai = <&ssc0>;
-		axentia,audio-codec = <&codec>;
-
-		axentia,add-gpios = <&pioA 8 GPIO_ACTIVE_LOW>;
-		axentia,loop1-gpios = <&pioA 10 GPIO_ACTIVE_LOW>;
-		axentia,loop2-gpios = <&pioA 11 GPIO_ACTIVE_LOW>;
-
-		axentia,ana-supply = <&ana>;
-	};
diff --git a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml
new file mode 100644
index 000000000000..0ab4b697d28e
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml
@@ -0,0 +1,104 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/sound/axentia,tse850-pcm5142.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Axentia TSE-850 audio complex
+
+maintainers:
+  - Peter Rosin <peda@axentia.se>
+
+description: |
+  The TSE-850 is an FM Transmitter Station Equipment, designed to generate
+  baseband signals for FM, mainly the DARC subcarrier, but other signals
+  are also possible.
+
+  The schematics explaining the gpios are as follows::
+
+                 loop1 relays
+     IN1 +---o  +------------+  o---+ OUT1
+              \                /
+               +              +
+               |   /          |
+               +--o  +--.     |
+               |  add   |     |
+               |        V     |
+               |      .---.   |
+     DAC +----------->|Sum|---+
+               |      '---'   |
+               |              |
+               +              +
+     IN2 +---o--+------------+--o---+ OUT2
+                 loop2 relays
+
+  The 'loop1' gpio pin controls two relays, which are either in loop
+  position, meaning that input and output are directly connected, or
+  they are in mixer position, meaning that the signal is passed through
+  the 'Sum' mixer. Similarly for 'loop2'.
+
+  In the above, the 'loop1' relays are inactive, thus feeding IN1 to the
+  mixer (if 'add' is active) and feeding the mixer output to OUT1. The
+  'loop2' relays are active, short-cutting the TSE-850 from channel 2.
+  IN1, IN2, OUT1 and OUT2 are TSE-850 connectors and DAC is the PCB name
+  of the (filtered) output from the PCM5142 codec.
+
+properties:
+  compatible:
+    const: axentia,tse850-pcm5142
+
+  axentia,cpu-dai:
+    $ref: /schemas/types.yaml#/definitions/phandle
+    description:
+      The phandle of the cpu dai.
+
+  axentia,audio-codec:
+    $ref: /schemas/types.yaml#/definitions/phandle
+    description:
+      The phandle of the PCM5142 codec.
+
+  axentia,add-gpios:
+    maxItems: 1
+    description:
+      gpio specifier that controls the mixer.
+
+  axentia,loop1-gpios:
+    maxItems: 1
+    description:
+      gpio specifier that controls loop relays on channel 1.
+
+  axentia,loop2-gpios:
+    maxItems: 1
+    description:
+      gpio specifier that controls loop relays on channel 2.
+
+  axentia,ana-supply:
+    description:
+      Regulator that supplies the output amplifier. Must support voltages in
+      the 2V - 20V range, in 1V steps.
+
+required:
+  - compatible
+  - axentia,cpu-dai
+  - axentia,audio-codec
+  - axentia,add-gpios
+  - axentia,loop1-gpios
+  - axentia,loop2-gpios
+  - axentia,ana-supply
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/gpio/gpio.h>
+    #include <dt-bindings/pwm/pwm.h>
+
+    sound {
+        compatible = "axentia,tse850-pcm5142";
+        axentia,cpu-dai = <&ssc0>;
+        axentia,audio-codec = <&codec>;
+        axentia,add-gpios = <&pioA 8 GPIO_ACTIVE_LOW>;
+        axentia,loop1-gpios = <&pioA 10 GPIO_ACTIVE_LOW>;
+        axentia,loop2-gpios = <&pioA 11 GPIO_ACTIVE_LOW>;
+        axentia,ana-supply = <&regulator_ana>;
+    };

---
base-commit: 3d6d817622b0a9721e3cc404df3469171582be13
change-id: 20260813-ti-axtensia-59c88b5f0f17

Best regards,
-- 
Bhargav


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: tse850: Convert to DT schema
  2026-08-12 23:33 [PATCH] ASoC: dt-bindings: tse850: Convert to DT schema Bhargav Joshi
@ 2026-08-13 10:51 ` Peter Rosin
  2026-08-14 20:18   ` Rob Herring
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Rosin @ 2026-08-13 10:51 UTC (permalink / raw)
  To: Bhargav Joshi, Claudiu Beznea, Andrei Simion, Liam Girdwood,
	Mark Brown, Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-sound, devicetree, linux-kernel, goledhruva, m-chawdhry,
	daniel.baluta, simona.toaca

On 2026-08-13 01:33, Bhargav Joshi wrote:
> Convert Axentia TSE-850 audio complex from text to DT schema. No
> funcational changes.
> 
> Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
> ---
>  .../bindings/sound/axentia,tse850-pcm5142.txt      |  92 ------------------
>  .../bindings/sound/axentia,tse850-pcm5142.yaml     | 104 +++++++++++++++++++++
>  2 files changed, 104 insertions(+), 92 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt
> deleted file mode 100644
> index b6cc5f6f78c2..000000000000
> --- a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.txt
> +++ /dev/null
> @@ -1,92 +0,0 @@
> -Devicetree bindings for the Axentia TSE-850 audio complex
> -
> -Required properties:
> -  - compatible: "axentia,tse850-pcm5142"
> -  - axentia,cpu-dai: The phandle of the cpu dai.
> -  - axentia,audio-codec: The phandle of the PCM5142 codec.
> -  - axentia,add-gpios: gpio specifier that controls the mixer.
> -  - axentia,loop1-gpios: gpio specifier that controls loop relays on channel 1.
> -  - axentia,loop2-gpios: gpio specifier that controls loop relays on channel 2.
> -  - axentia,ana-supply: Regulator that supplies the output amplifier. Must
> -    support voltages in the 2V - 20V range, in 1V steps.
> -
> -The schematics explaining the gpios are as follows:
> -
> -               loop1 relays
> -   IN1 +---o  +------------+  o---+ OUT1
> -            \                /
> -             +              +
> -             |   /          |
> -             +--o  +--.     |
> -             |  add   |     |
> -             |        V     |
> -             |      .---.   |
> -   DAC +----------->|Sum|---+
> -             |      '---'   |
> -             |              |
> -             +              +
> -
> -   IN2 +---o--+------------+--o---+ OUT2
> -               loop2 relays
> -
> -The 'loop1' gpio pin controls two relays, which are either in loop position,
> -meaning that input and output are directly connected, or they are in mixer
> -position, meaning that the signal is passed through the 'Sum' mixer. Similarly
> -for 'loop2'.
> -
> -In the above, the 'loop1' relays are inactive, thus feeding IN1 to the mixer
> -(if 'add' is active) and feeding the mixer output to OUT1. The 'loop2' relays
> -are active, short-cutting the TSE-850 from channel 2. IN1, IN2, OUT1 and OUT2
> -are TSE-850 connectors and DAC is the PCB name of the (filtered) output from
> -the PCM5142 codec.
> -
> -Example:
> -
> -	&ssc0 {
> -		#sound-dai-cells = <0>;
> -
> -	};
> -
> -	&i2c {
> -		codec: pcm5142@4c {
> -			compatible = "ti,pcm5142";
> -
> -			reg = <0x4c>;
> -
> -			AVDD-supply = <&reg_3v3>;
> -			DVDD-supply = <&reg_3v3>;
> -			CPVDD-supply = <&reg_3v3>;
> -
> -			clocks = <&sck>;
> -
> -			pll-in = <3>;
> -			pll-out = <6>;
> -		};
> -	};
> -
> -	ana: ana-reg {
> -		compatible = "pwm-regulator";
> -
> -		regulator-name = "ANA";
> -
> -		pwms = <&pwm0 2 1000 PWM_POLARITY_INVERTED>;
> -		pwm-dutycycle-unit = <1000>;
> -		pwm-dutycycle-range = <100 1000>;
> -
> -		regulator-min-microvolt = <2000000>;
> -		regulator-max-microvolt = <20000000>;
> -		regulator-ramp-delay = <1000>;
> -	};
> -
> -	sound {
> -		compatible = "axentia,tse850-pcm5142";
> -
> -		axentia,cpu-dai = <&ssc0>;
> -		axentia,audio-codec = <&codec>;
> -
> -		axentia,add-gpios = <&pioA 8 GPIO_ACTIVE_LOW>;
> -		axentia,loop1-gpios = <&pioA 10 GPIO_ACTIVE_LOW>;
> -		axentia,loop2-gpios = <&pioA 11 GPIO_ACTIVE_LOW>;
> -
> -		axentia,ana-supply = <&ana>;
> -	};
> diff --git a/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml
> new file mode 100644
> index 000000000000..0ab4b697d28e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/sound/axentia,tse850-pcm5142.yaml
> @@ -0,0 +1,104 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)

Where did BSD-2-Clause come from?

> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/sound/axentia,tse850-pcm5142.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Axentia TSE-850 audio complex
> +
> +maintainers:
> +  - Peter Rosin <peda@axentia.se>

Bzzzt, this is no longer the case. This binding is orphaned (as is its
corresponding driver).

Cheers,
Peter


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ASoC: dt-bindings: tse850: Convert to DT schema
  2026-08-13 10:51 ` Peter Rosin
@ 2026-08-14 20:18   ` Rob Herring
  0 siblings, 0 replies; 3+ messages in thread
From: Rob Herring @ 2026-08-14 20:18 UTC (permalink / raw)
  To: Peter Rosin
  Cc: Bhargav Joshi, Claudiu Beznea, Andrei Simion, Liam Girdwood,
	Mark Brown, Krzysztof Kozlowski, Conor Dooley, linux-sound,
	devicetree, linux-kernel, goledhruva, m-chawdhry, daniel.baluta,
	simona.toaca

On Thu, Aug 13, 2026 at 12:51:31PM +0200, Peter Rosin wrote:
> On 2026-08-13 01:33, Bhargav Joshi wrote:
> > Convert Axentia TSE-850 audio complex from text to DT schema. No
> > funcational changes.
> > 
> > Signed-off-by: Bhargav Joshi <j.bhargav.u@gmail.com>
> > ---
> >  .../bindings/sound/axentia,tse850-pcm5142.txt      |  92 ------------------
> >  .../bindings/sound/axentia,tse850-pcm5142.yaml     | 104 +++++++++++++++++++++
> >  2 files changed, 104 insertions(+), 92 deletions(-)


> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/sound/axentia,tse850-pcm5142.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Axentia TSE-850 audio complex
> > +
> > +maintainers:
> > +  - Peter Rosin <peda@axentia.se>
> 
> Bzzzt, this is no longer the case. This binding is orphaned (as is its
> corresponding driver).

So we can remove them?

Rob

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-08-14 20:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-12 23:33 [PATCH] ASoC: dt-bindings: tse850: Convert to DT schema Bhargav Joshi
2026-08-13 10:51 ` Peter Rosin
2026-08-14 20:18   ` Rob Herring

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox