From: Elinor Montmasson <elinor.montmasson@savoirfairelinux.com>
To: Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>, Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Conor Dooley <conor+dt@kernel.org>,
Shengjiu Wang <shengjiu.wang@gmail.com>,
Xiubo Li <Xiubo.Lee@gmail.com>,
Fabio Estevam <festevam@gmail.com>,
Nicolin Chen <nicoleotsuka@gmail.com>,
Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>
Cc: linux-sound@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org,
linuxppc-dev@lists.ozlabs.org,
Elinor Montmasson <elinor.montmasson@savoirfairelinux.com>
Subject: [PATCHv4 9/9] ASoC: dt-bindings: fsl-asoc-card: add compatible for generic codec
Date: Wed, 15 May 2024 15:54:11 +0200 [thread overview]
Message-ID: <20240515135411.343333-10-elinor.montmasson@savoirfairelinux.com> (raw)
In-Reply-To: <20240515135411.343333-1-elinor.montmasson@savoirfairelinux.com>
Add documentation about new dts bindings following new support
for compatible "fsl,imx-audio-generic".
Some CPU DAI don't require a real audio codec. The new compatible
"fsl,imx-audio-generic" allows using the driver with codec drivers
SPDIF DIT and SPDIF DIR as dummy codecs.
It also allows using not pre-configured audio codecs which
don't require specific control through a codec driver.
The new dts properties give the possibility to set some parameters
about the CPU DAI usually set through the codec configuration.
Signed-off-by: Elinor Montmasson <elinor.montmasson@savoirfairelinux.com>
---
.../bindings/sound/fsl-asoc-card.yaml | 96 ++++++++++++++++++-
1 file changed, 92 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/sound/fsl-asoc-card.yaml b/Documentation/devicetree/bindings/sound/fsl-asoc-card.yaml
index 9922664d5ccc..332d8bf96e06 100644
--- a/Documentation/devicetree/bindings/sound/fsl-asoc-card.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl-asoc-card.yaml
@@ -23,6 +23,16 @@ description:
and PCM DAI formats. However, it'll be also possible to support those non
AC'97/I2S/PCM type sound cards, such as S/PDIF audio and HDMI audio, as
long as the driver has been properly upgraded.
+ To use CPU DAIs that do not require a codec such as an S/PDIF controller,
+ or to use a DAI to output or capture raw I2S/TDM data, you can
+ use the compatible "fsl,imx-audio-generic".
+
+definitions:
+ imx-audio-generic-dependency:
+ properties:
+ compatible:
+ contains:
+ const: fsl,imx-audio-generic
maintainers:
- Shengjiu Wang <shengjiu.wang@nxp.com>
@@ -81,6 +91,7 @@ properties:
- fsl,imx-audio-wm8960
- fsl,imx-audio-wm8962
- fsl,imx-audio-wm8958
+ - fsl,imx-audio-generic
model:
$ref: /schemas/types.yaml#/definitions/string
@@ -93,8 +104,14 @@ properties:
need to add ASRC support via DPCM.
audio-codec:
- $ref: /schemas/types.yaml#/definitions/phandle
- description: The phandle of an audio codec
+ $ref: /schemas/types.yaml#/definitions/phandle-array
+ description: |
+ The phandle of an audio codec.
+ If using the "fsl,imx-audio-generic" compatible, give instead a pair of
+ phandles with the spdif_transmitter first (driver SPDIF DIT) and the
+ spdif_receiver second (driver SPDIF DIR).
+ items:
+ maxItems: 1
audio-cpu:
$ref: /schemas/types.yaml#/definitions/phandle
@@ -150,8 +167,8 @@ properties:
description: dai-link uses bit clock inversion.
mclk-id:
- $ref: /schemas/types.yaml#/definitions/uint32
- description: main clock id, specific for each card configuration.
+ $ref: /schemas/types.yaml#/definitions/uint32-array
+ description: Main clock id for each codec, specific for each card configuration.
mux-int-port:
$ref: /schemas/types.yaml#/definitions/uint32
@@ -167,10 +184,68 @@ properties:
$ref: /schemas/types.yaml#/definitions/phandle
description: The phandle of an CPU DAI controller
+ # Properties relevant only with "fsl,imx-audio-generic" compatible
+ dai-tdm-slot-width:
+ description: See tdm-slot.txt.
+ $ref: /schemas/types.yaml#/definitions/uint32
+
+ dai-tdm-slot-num:
+ description: See tdm-slot.txt.
+ $ref: /schemas/types.yaml#/definitions/uint32
+
+ clocks:
+ description: |
+ The CPU DAI system clock, used to retrieve
+ the CPU DAI system clock frequency with the generic codec.
+ maxItems: 1
+
+ clock-names:
+ items:
+ - const: cpu_sysclk
+
+ cpu-system-clock-direction-out:
+ description: |
+ Specifies cpu system clock direction as 'out' on initialization.
+ If not set, direction is 'in'.
+ $ref: /schemas/types.yaml#/definitions/flag
+
+dependencies:
+ dai-tdm-slot-width:
+ $ref: "#/definitions/imx-audio-generic-dependency"
+ dai-tdm-slot-num:
+ $ref: "#/definitions/imx-audio-generic-dependency"
+ clocks:
+ $ref: "#/definitions/imx-audio-generic-dependency"
+ cpu-system-clock-direction-out:
+ $ref: "#/definitions/imx-audio-generic-dependency"
+
required:
- compatible
- model
+allOf:
+ - if:
+ $ref: "#/definitions/imx-audio-generic-dependency"
+ then:
+ properties:
+ audio-codec:
+ items:
+ - description: SPDIF DIT phandle
+ - description: SPDIF DIR phandle
+ mclk-id:
+ maxItems: 1
+ items:
+ minItems: 1
+ maxItems: 2
+ else:
+ properties:
+ audio-codec:
+ maxItems: 1
+ mclk-id:
+ maxItems: 1
+ items:
+ maxItems: 1
+
unevaluatedProperties: false
examples:
@@ -195,3 +270,16 @@ examples:
"AIN2L", "Line In Jack",
"AIN2R", "Line In Jack";
};
+
+ - |
+ #include <dt-bindings/clock/imx8mn-clock.h>
+ sound-spdif-asrc {
+ compatible = "fsl,imx-audio-generic";
+ model = "spdif-asrc-audio";
+ audio-cpu = <&spdif>;
+ audio-asrc = <&easrc>;
+ audio-codec = <&spdifdit>, <&spdifdir>;
+ clocks = <&clk IMX8MN_CLK_SAI5_ROOT>;
+ clock-names = "cpu_sysclk";
+ cpu-system-clock-direction-out;
+ };
--
2.34.1
next prev parent reply other threads:[~2024-05-15 13:54 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-15 13:54 [PATCHv4 0/9] ASoC: fsl-asoc-card: compatibility integration of a generic codec use case for use with S/PDIF controller Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 1/9] ASoC: fsl-asoc-card: add support for dai links with multiple codecs Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 2/9] ASoC: fsl-asoc-card: add second dai link component for codecs Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 3/9] ASoC: fsl-asoc-card: add compatibility to use 2 codecs in dai-links Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 4/9] ASoC: fsl-asoc-card: add new compatible for a generic codec use case Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 5/9] ASoC: fsl-asoc-card: set generic codec as clock provider Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 6/9] ASoC: fsl-asoc-card: add use of devicetree TDM slot properties Elinor Montmasson
2024-05-15 13:54 ` [PATCHv4 7/9] ASoC: fsl-asoc-card: add DT clock "cpu_sysclk" with generic codec Elinor Montmasson
2024-05-16 12:13 ` Mark Brown
2024-05-17 9:05 ` Elinor Montmasson
2024-05-17 11:17 ` Mark Brown
2024-05-31 12:46 ` Elinor Montmasson
2024-05-31 13:05 ` Mark Brown
2024-05-31 14:48 ` Elinor Montmasson
2024-05-31 16:55 ` Mark Brown
2024-05-15 13:54 ` [PATCHv4 8/9] ASoC: fsl-asoc-card: add DT property "cpu-system-clock-direction-out" Elinor Montmasson
2024-05-16 12:18 ` Mark Brown
2024-05-17 9:05 ` Elinor Montmasson
2024-05-17 11:06 ` Mark Brown
2024-05-31 12:47 ` Elinor Montmasson
2024-05-31 13:09 ` Mark Brown
2024-05-15 13:54 ` Elinor Montmasson [this message]
2024-05-16 12:11 ` [PATCHv4 9/9] ASoC: dt-bindings: fsl-asoc-card: add compatible for generic codec Mark Brown
2024-05-17 9:05 ` Elinor Montmasson
2024-05-17 11:11 ` Mark Brown
2024-05-31 12:48 ` Elinor Montmasson
2024-05-31 13:14 ` Mark Brown
2024-05-31 14:48 ` Elinor Montmasson
2024-05-31 16:06 ` Mark Brown
2024-05-20 18:31 ` Rob Herring
2024-05-31 12:48 ` Elinor Montmasson
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=20240515135411.343333-10-elinor.montmasson@savoirfairelinux.com \
--to=elinor.montmasson@savoirfairelinux.com \
--cc=Xiubo.Lee@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=nicoleotsuka@gmail.com \
--cc=perex@perex.cz \
--cc=robh+dt@kernel.org \
--cc=shengjiu.wang@gmail.com \
--cc=tiwai@suse.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 read-only IMAP folder(s) and NNTP newsgroup(s).