* [PATCH v2 0/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation @ 2020-08-27 0:24 Kuninori Morimoto 2020-08-27 0:24 ` [PATCH v2 1/2] " Kuninori Morimoto 2020-08-27 0:25 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto 0 siblings, 2 replies; 9+ messages in thread From: Kuninori Morimoto @ 2020-08-27 0:24 UTC (permalink / raw) To: Rob Herring, Mark Brown; +Cc: devicetree, alsa-devel Hi Rob, Mark These are v2 of switch to yaml base Documentation for audio-graph-card and renesas,rsnd which uses audio-graph-card. v1 -> v2 - upgrade dt-schema, and fixup errors for [2/2] patch Kuninori Morimoto (2): ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation ASoC: dt-bindings: renesas,rsnd: switch to yaml base Documentation .../bindings/sound/audio-graph-card.txt | 337 ------------ .../bindings/sound/audio-graph-card.yaml | 135 +++++ .../bindings/sound/renesas,rsnd.txt | 520 ------------------ .../bindings/sound/renesas,rsnd.yaml | 454 +++++++++++++++ .../devicetree/bindings/vendor-prefixes.yaml | 2 +- 5 files changed, 590 insertions(+), 858 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.txt create mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.yaml create mode 100644 Documentation/devicetree/bindings/sound/renesas,rsnd.yaml -- 2.25.1 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation 2020-08-27 0:24 [PATCH v2 0/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation Kuninori Morimoto @ 2020-08-27 0:24 ` Kuninori Morimoto 2020-09-08 23:14 ` Rob Herring 2020-08-27 0:25 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto 1 sibling, 1 reply; 9+ messages in thread From: Kuninori Morimoto @ 2020-08-27 0:24 UTC (permalink / raw) To: Rob Herring, Mark Brown; +Cc: devicetree, alsa-devel From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> This patch switches from .txt base to .yaml base Document. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> --- .../bindings/sound/audio-graph-card.txt | 337 ------------------ .../bindings/sound/audio-graph-card.yaml | 135 +++++++ .../devicetree/bindings/vendor-prefixes.yaml | 2 +- 3 files changed, 136 insertions(+), 338 deletions(-) delete mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.txt create mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.yaml diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.txt b/Documentation/devicetree/bindings/sound/audio-graph-card.txt deleted file mode 100644 index d5f6919a2d69..000000000000 --- a/Documentation/devicetree/bindings/sound/audio-graph-card.txt +++ /dev/null @@ -1,337 +0,0 @@ -Audio Graph Card: - -Audio Graph Card specifies audio DAI connections of SoC <-> codec. -It is based on common bindings for device graphs. -see ${LINUX}/Documentation/devicetree/bindings/graph.txt - -Basically, Audio Graph Card property is same as Simple Card. -see ${LINUX}/Documentation/devicetree/bindings/sound/simple-card.yaml - -Below are same as Simple-Card. - -- label -- widgets -- routing -- dai-format -- frame-master -- bitclock-master -- bitclock-inversion -- frame-inversion -- mclk-fs -- hp-det-gpio -- mic-det-gpio -- dai-tdm-slot-num -- dai-tdm-slot-width -- clocks / system-clock-frequency - -Required properties: - -- compatible : "audio-graph-card"; -- dais : list of CPU DAI port{s} - -Optional properties: -- pa-gpios: GPIO used to control external amplifier. - ------------------------ -Example: Single DAI case ------------------------ - - sound_card { - compatible = "audio-graph-card"; - - dais = <&cpu_port>; - }; - - dai-controller { - ... - cpu_port: port { - cpu_endpoint: endpoint { - remote-endpoint = <&codec_endpoint>; - - dai-format = "left_j"; - ... - }; - }; - }; - - audio-codec { - ... - port { - codec_endpoint: endpoint { - remote-endpoint = <&cpu_endpoint>; - }; - }; - }; - ------------------------ -Example: Multi DAI case ------------------------ - - sound-card { - compatible = "audio-graph-card"; - - label = "sound-card"; - - dais = <&cpu_port0 - &cpu_port1 - &cpu_port2>; - }; - - audio-codec@0 { - ... - port { - codec0_endpoint: endpoint { - remote-endpoint = <&cpu_endpoint0>; - }; - }; - }; - - audio-codec@1 { - ... - port { - codec1_endpoint: endpoint { - remote-endpoint = <&cpu_endpoint1>; - }; - }; - }; - - audio-codec@2 { - ... - port { - codec2_endpoint: endpoint { - remote-endpoint = <&cpu_endpoint2>; - }; - }; - }; - - dai-controller { - ... - ports { - cpu_port0: port@0 { - cpu_endpoint0: endpoint { - remote-endpoint = <&codec0_endpoint>; - - dai-format = "left_j"; - ... - }; - }; - cpu_port1: port@1 { - cpu_endpoint1: endpoint { - remote-endpoint = <&codec1_endpoint>; - - dai-format = "i2s"; - ... - }; - }; - cpu_port2: port@2 { - cpu_endpoint2: endpoint { - remote-endpoint = <&codec2_endpoint>; - - dai-format = "i2s"; - ... - }; - }; - }; - }; - - ------------------------ -Example: Sampling Rate Conversion ------------------------ - - sound_card { - compatible = "audio-graph-card"; - - label = "sound-card"; - prefix = "codec"; - routing = "codec Playback", "DAI0 Playback", - "DAI0 Capture", "codec Capture"; - convert-rate = <48000>; - - dais = <&cpu_port>; - }; - - audio-codec { - ... - port { - codec_endpoint: endpoint { - remote-endpoint = <&cpu_endpoint>; - }; - }; - }; - - dai-controller { - ... - cpu_port: port { - cpu_endpoint: endpoint { - remote-endpoint = <&codec_endpoint>; - - dai-format = "left_j"; - ... - }; - }; - }; - ------------------------ -Example: 2 CPU 1 Codec (Mixing) ------------------------ - - sound_card { - compatible = "audio-graph-card"; - - label = "sound-card"; - routing = "codec Playback", "DAI0 Playback", - "codec Playback", "DAI1 Playback", - "DAI0 Capture", "codec Capture"; - - dais = <&cpu_port>; - }; - - audio-codec { - ... - - audio-graph-card,prefix = "codec"; - audio-graph-card,convert-rate = <48000>; - port { - reg = <0>; - codec_endpoint0: endpoint@0 { - remote-endpoint = <&cpu_endpoint0>; - }; - codec_endpoint1: endpoint@1 { - remote-endpoint = <&cpu_endpoint1>; - }; - }; - }; - - dai-controller { - ... - cpu_port: port { - cpu_endpoint0: endpoint@0 { - remote-endpoint = <&codec_endpoint0>; - - dai-format = "left_j"; - ... - }; - cpu_endpoint1: endpoint@1 { - remote-endpoint = <&codec_endpoint1>; - - dai-format = "left_j"; - ... - }; - }; - }; - ------------------------ -Example: Multi DAI with DPCM ------------------------ - - CPU0 ------ ak4613 - CPU1 ------ HDMI - CPU2 ------ PCM3168A-p /* DPCM 1ch/2ch */ - CPU3 --/ /* DPCM 3ch/4ch */ - CPU4 --/ /* DPCM 5ch/6ch */ - CPU5 --/ /* DPCM 7ch/8ch */ - CPU6 ------ PCM3168A-c - - sound_card: sound { - compatible = "audio-graph-card"; - - label = "sound-card"; - - routing = "pcm3168a Playback", "DAI2 Playback", - "pcm3168a Playback", "DAI3 Playback", - "pcm3168a Playback", "DAI4 Playback", - "pcm3168a Playback", "DAI5 Playback"; - - dais = <&snd_port0 /* ak4613 */ - &snd_port1 /* HDMI0 */ - &snd_port2 /* pcm3168a playback */ - &snd_port3 /* pcm3168a capture */ - >; - }; - - ak4613: codec@10 { - ... - port { - ak4613_endpoint: endpoint { - remote-endpoint = <&rsnd_endpoint0>; - }; - }; - }; - - pcm3168a: audio-codec@44 { - ... - audio-graph-card,prefix = "pcm3168a"; - audio-graph-card,convert-channels = <8>; /* TDM Split */ - ports { - port@0 { - reg = <0>; - pcm3168a_endpoint_p1: endpoint@1 { - remote-endpoint = <&rsnd_endpoint2>; - ... - }; - pcm3168a_endpoint_p2: endpoint@2 { - remote-endpoint = <&rsnd_endpoint3>; - ... - }; - pcm3168a_endpoint_p3: endpoint@3 { - remote-endpoint = <&rsnd_endpoint4>; - ... - }; - pcm3168a_endpoint_p4: endpoint@4 { - remote-endpoint = <&rsnd_endpoint5>; - ... - }; - }; - port@1 { - reg = <1>; - pcm3168a_endpoint_c: endpoint { - remote-endpoint = <&rsnd_endpoint6>; - ... - }; - }; - }; - }; - - &sound { - ports { - snd_port0: port@0 { - rsnd_endpoint0: endpoint { - remote-endpoint = <&ak4613_endpoint>; - ... - }; - }; - snd_port1: port@1 { - rsnd_endpoint1: endpoint { - remote-endpoint = <&dw_hdmi0_snd_in>; - ... - }; - }; - snd_port2: port@2 { - #address-cells = <1>; - #size-cells = <0>; - rsnd_endpoint2: endpoint@2 { - remote-endpoint = <&pcm3168a_endpoint_p1>; - ... - }; - rsnd_endpoint3: endpoint@3 { - remote-endpoint = <&pcm3168a_endpoint_p2>; - ... - }; - rsnd_endpoint4: endpoint@4 { - remote-endpoint = <&pcm3168a_endpoint_p3>; - ... - }; - rsnd_endpoint5: endpoint@5 { - remote-endpoint = <&pcm3168a_endpoint_p4>; - ... - }; - }; - snd_port3: port@6 { - rsnd_endpoint6: endpoint { - remote-endpoint = <&pcm3168a_endpoint_c>; - ... - }; - }; - }; - }; diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.yaml b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml new file mode 100644 index 000000000000..60d7b1504128 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml @@ -0,0 +1,135 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/audio-graph-card.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Audio Graph Card Driver Device Tree Bindings + +maintainers: + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> + +definitions: + endpoint: + type: object + properties: + remote-endpoint: + maxItems: 1 + mclk-fs: + $ref: "simple-card.yaml#/definitions/mclk-fs" + frame-inversion: + $ref: "simple-card.yaml#/definitions/frame-inversion" + bitclock-inversion: + $ref: "simple-card.yaml#/definitions/bitclock-inversion" + frame-master: + $ref: "simple-card.yaml#/definitions/frame-master" + bitclock-master: + $ref: "simple-card.yaml#/definitions/bitclock-master" + dai-format: + $ref: "simple-card.yaml#/definitions/format" + convert-rate: + $ref: "simple-card.yaml#/definitions/convert-rate" + convert-channels: + $ref: "simple-card.yaml#/definitions/convert-channels" + required: + - remote-endpoint + + port: + description: single OF-Graph subnode + type: object + properties: + reg: + maxItems: 1 + prefix: + $ref: "simple-card.yaml#/definitions/prefix" + convert-rate: + $ref: "simple-card.yaml#/definitions/convert-rate" + convert-channels: + $ref: "simple-card.yaml#/definitions/convert-channels" + patternProperties: + "^endpoint(@[0-9a-f]+)?": + $ref: "#/definitions/endpoint" + + ports: + description: multi OF-Graph subnode + type: object + properties: + reg: + maxItems: 1 + prefix: + $ref: "simple-card.yaml#/definitions/prefix" + convert-rate: + $ref: "simple-card.yaml#/definitions/convert-rate" + convert-channels: + $ref: "simple-card.yaml#/definitions/convert-channels" + patternProperties: + "^port(@[0-9a-f]+)?": + $ref: "#/definitions/port" + +properties: + compatible: + contains: + enum: + - audio-graph-card + - audio-graph-scu-card + + dais: + $ref: /schemas/types.yaml#/definitions/phandle-array + label: + maxItems: 1 + prefix: + $ref: "simple-card.yaml#/definitions/prefix" + routing: + $ref: "simple-card.yaml#/definitions/routing" + widgets: + $ref: "simple-card.yaml#/definitions/widgets" + convert-rate: + $ref: "simple-card.yaml#/definitions/convert-rate" + convert-channels: + $ref: "simple-card.yaml#/definitions/convert-channels" + pa-gpios: + maxItems: 1 + hp-det-gpio: + maxItems: 1 + mic-det-gpio: + maxItems: 1 + +required: + - compatible + - dais + +additionalProperties: false + +examples: + - | + sound { + compatible = "audio-graph-card"; + + dais = <&cpu_port_a>; + }; + + cpu { + /* + * dai-controller own settings + */ + + port { + cpu_endpoint: endpoint { + remote-endpoint = <&codec_endpoint>; + dai-format = "left_j"; + /* ... */ + }; + }; + }; + + codec { + /* + * codec own settings + */ + + port { + codec_endpoint: endpoint { + remote-endpoint = <&cpu_endpoint>; + }; + }; + }; diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 63996ab03521..5ada35ddac7e 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -20,7 +20,7 @@ patternProperties: "^(keypad|m25p|max8952|max8997|max8998|mpmc),.*": true "^(pinctrl-single|#pinctrl-single|PowerPC),.*": true "^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*": true - "^(simple-audio-card|st-plgpio|st-spics|ts),.*": true + "^(simple-audio-card|audio-graph-card|st-plgpio|st-spics|ts),.*": true # Keep list in alphabetical order. "^70mai,.*": -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation 2020-08-27 0:24 ` [PATCH v2 1/2] " Kuninori Morimoto @ 2020-09-08 23:14 ` Rob Herring 2020-09-10 1:17 ` Kuninori Morimoto 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2020-09-08 23:14 UTC (permalink / raw) To: Kuninori Morimoto; +Cc: devicetree, alsa-devel, Mark Brown On Thu, Aug 27, 2020 at 09:24:54AM +0900, Kuninori Morimoto wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > This patch switches from .txt base to .yaml base Document. > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > .../bindings/sound/audio-graph-card.txt | 337 ------------------ > .../bindings/sound/audio-graph-card.yaml | 135 +++++++ > .../devicetree/bindings/vendor-prefixes.yaml | 2 +- > 3 files changed, 136 insertions(+), 338 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.txt > create mode 100644 Documentation/devicetree/bindings/sound/audio-graph-card.yaml > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.txt b/Documentation/devicetree/bindings/sound/audio-graph-card.txt > deleted file mode 100644 > index d5f6919a2d69..000000000000 > --- a/Documentation/devicetree/bindings/sound/audio-graph-card.txt > +++ /dev/null > @@ -1,337 +0,0 @@ > -Audio Graph Card: > - > -Audio Graph Card specifies audio DAI connections of SoC <-> codec. > -It is based on common bindings for device graphs. > -see ${LINUX}/Documentation/devicetree/bindings/graph.txt > - > -Basically, Audio Graph Card property is same as Simple Card. > -see ${LINUX}/Documentation/devicetree/bindings/sound/simple-card.yaml > - > -Below are same as Simple-Card. > - > -- label > -- widgets > -- routing > -- dai-format > -- frame-master > -- bitclock-master > -- bitclock-inversion > -- frame-inversion > -- mclk-fs > -- hp-det-gpio > -- mic-det-gpio > -- dai-tdm-slot-num > -- dai-tdm-slot-width > -- clocks / system-clock-frequency > - > -Required properties: > - > -- compatible : "audio-graph-card"; > -- dais : list of CPU DAI port{s} > - > -Optional properties: > -- pa-gpios: GPIO used to control external amplifier. > - > ------------------------ > -Example: Single DAI case > ------------------------ > - > - sound_card { > - compatible = "audio-graph-card"; > - > - dais = <&cpu_port>; > - }; > - > - dai-controller { > - ... > - cpu_port: port { > - cpu_endpoint: endpoint { > - remote-endpoint = <&codec_endpoint>; > - > - dai-format = "left_j"; > - ... > - }; > - }; > - }; > - > - audio-codec { > - ... > - port { > - codec_endpoint: endpoint { > - remote-endpoint = <&cpu_endpoint>; > - }; > - }; > - }; > - > ------------------------ > -Example: Multi DAI case > ------------------------ > - > - sound-card { > - compatible = "audio-graph-card"; > - > - label = "sound-card"; > - > - dais = <&cpu_port0 > - &cpu_port1 > - &cpu_port2>; > - }; > - > - audio-codec@0 { > - ... > - port { > - codec0_endpoint: endpoint { > - remote-endpoint = <&cpu_endpoint0>; > - }; > - }; > - }; > - > - audio-codec@1 { > - ... > - port { > - codec1_endpoint: endpoint { > - remote-endpoint = <&cpu_endpoint1>; > - }; > - }; > - }; > - > - audio-codec@2 { > - ... > - port { > - codec2_endpoint: endpoint { > - remote-endpoint = <&cpu_endpoint2>; > - }; > - }; > - }; > - > - dai-controller { > - ... > - ports { > - cpu_port0: port@0 { > - cpu_endpoint0: endpoint { > - remote-endpoint = <&codec0_endpoint>; > - > - dai-format = "left_j"; > - ... > - }; > - }; > - cpu_port1: port@1 { > - cpu_endpoint1: endpoint { > - remote-endpoint = <&codec1_endpoint>; > - > - dai-format = "i2s"; > - ... > - }; > - }; > - cpu_port2: port@2 { > - cpu_endpoint2: endpoint { > - remote-endpoint = <&codec2_endpoint>; > - > - dai-format = "i2s"; > - ... > - }; > - }; > - }; > - }; > - > - > ------------------------ > -Example: Sampling Rate Conversion > ------------------------ > - > - sound_card { > - compatible = "audio-graph-card"; > - > - label = "sound-card"; > - prefix = "codec"; > - routing = "codec Playback", "DAI0 Playback", > - "DAI0 Capture", "codec Capture"; > - convert-rate = <48000>; > - > - dais = <&cpu_port>; > - }; > - > - audio-codec { > - ... > - port { > - codec_endpoint: endpoint { > - remote-endpoint = <&cpu_endpoint>; > - }; > - }; > - }; > - > - dai-controller { > - ... > - cpu_port: port { > - cpu_endpoint: endpoint { > - remote-endpoint = <&codec_endpoint>; > - > - dai-format = "left_j"; > - ... > - }; > - }; > - }; > - > ------------------------ > -Example: 2 CPU 1 Codec (Mixing) > ------------------------ > - > - sound_card { > - compatible = "audio-graph-card"; > - > - label = "sound-card"; > - routing = "codec Playback", "DAI0 Playback", > - "codec Playback", "DAI1 Playback", > - "DAI0 Capture", "codec Capture"; > - > - dais = <&cpu_port>; > - }; > - > - audio-codec { > - ... > - > - audio-graph-card,prefix = "codec"; > - audio-graph-card,convert-rate = <48000>; > - port { > - reg = <0>; > - codec_endpoint0: endpoint@0 { > - remote-endpoint = <&cpu_endpoint0>; > - }; > - codec_endpoint1: endpoint@1 { > - remote-endpoint = <&cpu_endpoint1>; > - }; > - }; > - }; > - > - dai-controller { > - ... > - cpu_port: port { > - cpu_endpoint0: endpoint@0 { > - remote-endpoint = <&codec_endpoint0>; > - > - dai-format = "left_j"; > - ... > - }; > - cpu_endpoint1: endpoint@1 { > - remote-endpoint = <&codec_endpoint1>; > - > - dai-format = "left_j"; > - ... > - }; > - }; > - }; > - > ------------------------ > -Example: Multi DAI with DPCM > ------------------------ > - > - CPU0 ------ ak4613 > - CPU1 ------ HDMI > - CPU2 ------ PCM3168A-p /* DPCM 1ch/2ch */ > - CPU3 --/ /* DPCM 3ch/4ch */ > - CPU4 --/ /* DPCM 5ch/6ch */ > - CPU5 --/ /* DPCM 7ch/8ch */ > - CPU6 ------ PCM3168A-c > - > - sound_card: sound { > - compatible = "audio-graph-card"; > - > - label = "sound-card"; > - > - routing = "pcm3168a Playback", "DAI2 Playback", > - "pcm3168a Playback", "DAI3 Playback", > - "pcm3168a Playback", "DAI4 Playback", > - "pcm3168a Playback", "DAI5 Playback"; > - > - dais = <&snd_port0 /* ak4613 */ > - &snd_port1 /* HDMI0 */ > - &snd_port2 /* pcm3168a playback */ > - &snd_port3 /* pcm3168a capture */ > - >; > - }; > - > - ak4613: codec@10 { > - ... > - port { > - ak4613_endpoint: endpoint { > - remote-endpoint = <&rsnd_endpoint0>; > - }; > - }; > - }; > - > - pcm3168a: audio-codec@44 { > - ... > - audio-graph-card,prefix = "pcm3168a"; > - audio-graph-card,convert-channels = <8>; /* TDM Split */ > - ports { > - port@0 { > - reg = <0>; > - pcm3168a_endpoint_p1: endpoint@1 { > - remote-endpoint = <&rsnd_endpoint2>; > - ... > - }; > - pcm3168a_endpoint_p2: endpoint@2 { > - remote-endpoint = <&rsnd_endpoint3>; > - ... > - }; > - pcm3168a_endpoint_p3: endpoint@3 { > - remote-endpoint = <&rsnd_endpoint4>; > - ... > - }; > - pcm3168a_endpoint_p4: endpoint@4 { > - remote-endpoint = <&rsnd_endpoint5>; > - ... > - }; > - }; > - port@1 { > - reg = <1>; > - pcm3168a_endpoint_c: endpoint { > - remote-endpoint = <&rsnd_endpoint6>; > - ... > - }; > - }; > - }; > - }; > - > - &sound { > - ports { > - snd_port0: port@0 { > - rsnd_endpoint0: endpoint { > - remote-endpoint = <&ak4613_endpoint>; > - ... > - }; > - }; > - snd_port1: port@1 { > - rsnd_endpoint1: endpoint { > - remote-endpoint = <&dw_hdmi0_snd_in>; > - ... > - }; > - }; > - snd_port2: port@2 { > - #address-cells = <1>; > - #size-cells = <0>; > - rsnd_endpoint2: endpoint@2 { > - remote-endpoint = <&pcm3168a_endpoint_p1>; > - ... > - }; > - rsnd_endpoint3: endpoint@3 { > - remote-endpoint = <&pcm3168a_endpoint_p2>; > - ... > - }; > - rsnd_endpoint4: endpoint@4 { > - remote-endpoint = <&pcm3168a_endpoint_p3>; > - ... > - }; > - rsnd_endpoint5: endpoint@5 { > - remote-endpoint = <&pcm3168a_endpoint_p4>; > - ... > - }; > - }; > - snd_port3: port@6 { > - rsnd_endpoint6: endpoint { > - remote-endpoint = <&pcm3168a_endpoint_c>; > - ... > - }; > - }; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.yaml b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > new file mode 100644 > index 000000000000..60d7b1504128 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > @@ -0,0 +1,135 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sound/audio-graph-card.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Audio Graph Card Driver Device Tree Bindings > + > +maintainers: > + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > + > +definitions: Please don't use 'definitions' unless really needed. There's only 1 reference to 'endpoint' and 'port' so you don't here. > + endpoint: > + type: object > + properties: > + remote-endpoint: > + maxItems: 1 > + mclk-fs: > + $ref: "simple-card.yaml#/definitions/mclk-fs" > + frame-inversion: > + $ref: "simple-card.yaml#/definitions/frame-inversion" > + bitclock-inversion: > + $ref: "simple-card.yaml#/definitions/bitclock-inversion" > + frame-master: > + $ref: "simple-card.yaml#/definitions/frame-master" > + bitclock-master: > + $ref: "simple-card.yaml#/definitions/bitclock-master" > + dai-format: > + $ref: "simple-card.yaml#/definitions/format" > + convert-rate: > + $ref: "simple-card.yaml#/definitions/convert-rate" > + convert-channels: > + $ref: "simple-card.yaml#/definitions/convert-channels" > + required: > + - remote-endpoint > + > + port: > + description: single OF-Graph subnode > + type: object > + properties: > + reg: > + maxItems: 1 > + prefix: > + $ref: "simple-card.yaml#/definitions/prefix" > + convert-rate: > + $ref: "simple-card.yaml#/definitions/convert-rate" > + convert-channels: > + $ref: "simple-card.yaml#/definitions/convert-channels" > + patternProperties: > + "^endpoint(@[0-9a-f]+)?": > + $ref: "#/definitions/endpoint" > + > + ports: > + description: multi OF-Graph subnode > + type: object > + properties: > + reg: > + maxItems: 1 'ports' never has a 'reg' property. > + prefix: > + $ref: "simple-card.yaml#/definitions/prefix" > + convert-rate: > + $ref: "simple-card.yaml#/definitions/convert-rate" > + convert-channels: > + $ref: "simple-card.yaml#/definitions/convert-channels" These shouldn't be valid under 'ports' either? > + patternProperties: > + "^port(@[0-9a-f]+)?": > + $ref: "#/definitions/port" > + > +properties: > + compatible: > + contains: Any other additional string is valid? > + enum: > + - audio-graph-card > + - audio-graph-scu-card > + > + dais: > + $ref: /schemas/types.yaml#/definitions/phandle-array > + label: > + maxItems: 1 > + prefix: > + $ref: "simple-card.yaml#/definitions/prefix" > + routing: > + $ref: "simple-card.yaml#/definitions/routing" > + widgets: > + $ref: "simple-card.yaml#/definitions/widgets" > + convert-rate: > + $ref: "simple-card.yaml#/definitions/convert-rate" > + convert-channels: > + $ref: "simple-card.yaml#/definitions/convert-channels" > + pa-gpios: > + maxItems: 1 > + hp-det-gpio: > + maxItems: 1 > + mic-det-gpio: > + maxItems: 1 > + > +required: > + - compatible > + - dais > + > +additionalProperties: false > + > +examples: > + - | > + sound { > + compatible = "audio-graph-card"; > + > + dais = <&cpu_port_a>; > + }; > + > + cpu { > + /* > + * dai-controller own settings > + */ > + > + port { > + cpu_endpoint: endpoint { > + remote-endpoint = <&codec_endpoint>; > + dai-format = "left_j"; > + /* ... */ Would be good if the example used all/most of the possible properties to have at least some testing of the schema. > + }; > + }; > + }; > + > + codec { > + /* > + * codec own settings > + */ > + > + port { > + codec_endpoint: endpoint { > + remote-endpoint = <&cpu_endpoint>; > + }; > + }; > + }; > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > index 63996ab03521..5ada35ddac7e 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -20,7 +20,7 @@ patternProperties: > "^(keypad|m25p|max8952|max8997|max8998|mpmc),.*": true > "^(pinctrl-single|#pinctrl-single|PowerPC),.*": true > "^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*": true > - "^(simple-audio-card|st-plgpio|st-spics|ts),.*": true > + "^(simple-audio-card|audio-graph-card|st-plgpio|st-spics|ts),.*": true This isn't needed as we got rid of the silly 'audio-graph-card,' prefix. > > # Keep list in alphabetical order. > "^70mai,.*": > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation 2020-09-08 23:14 ` Rob Herring @ 2020-09-10 1:17 ` Kuninori Morimoto 2020-09-10 18:57 ` Rob Herring 0 siblings, 1 reply; 9+ messages in thread From: Kuninori Morimoto @ 2020-09-10 1:17 UTC (permalink / raw) To: Rob Herring; +Cc: devicetree, alsa-devel, Mark Brown Hi Rob Thank you for your feedback > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.yaml b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > > new file mode 100644 > > index 000000000000..60d7b1504128 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > > @@ -0,0 +1,135 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/sound/audio-graph-card.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Audio Graph Card Driver Device Tree Bindings > > + > > +maintainers: > > + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > + > > +definitions: > > Please don't use 'definitions' unless really needed. There's only 1 > reference to 'endpoint' and 'port' so you don't here. If this was applied, all audio-graph-card user driver want to use it. > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > index 63996ab03521..5ada35ddac7e 100644 > > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > @@ -20,7 +20,7 @@ patternProperties: > > "^(keypad|m25p|max8952|max8997|max8998|mpmc),.*": true > > "^(pinctrl-single|#pinctrl-single|PowerPC),.*": true > > "^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*": true > > - "^(simple-audio-card|st-plgpio|st-spics|ts),.*": true > > + "^(simple-audio-card|audio-graph-card|st-plgpio|st-spics|ts),.*": true > > This isn't needed as we got rid of the silly 'audio-graph-card,' prefix. Oops ? silly "audio-graph-card,xxx" is needed, but was missing on this patch. Will care about in v2. Thank you for your help !! Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation 2020-09-10 1:17 ` Kuninori Morimoto @ 2020-09-10 18:57 ` Rob Herring 0 siblings, 0 replies; 9+ messages in thread From: Rob Herring @ 2020-09-10 18:57 UTC (permalink / raw) To: Kuninori Morimoto; +Cc: devicetree, Linux-ALSA, Mark Brown On Wed, Sep 9, 2020 at 7:17 PM Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: > > > Hi Rob > > Thank you for your feedback > > > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-card.yaml b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > > > new file mode 100644 > > > index 000000000000..60d7b1504128 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/sound/audio-graph-card.yaml > > > @@ -0,0 +1,135 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/sound/audio-graph-card.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: Audio Graph Card Driver Device Tree Bindings > > > + > > > +maintainers: > > > + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > + > > > +definitions: > > > > Please don't use 'definitions' unless really needed. There's only 1 > > reference to 'endpoint' and 'port' so you don't here. > > If this was applied, all audio-graph-card user driver want to use it. They should reference this schema at the top level, not down in it. Rob ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: switch to yaml base Documentation 2020-08-27 0:24 [PATCH v2 0/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation Kuninori Morimoto 2020-08-27 0:24 ` [PATCH v2 1/2] " Kuninori Morimoto @ 2020-08-27 0:25 ` Kuninori Morimoto 2020-09-08 23:22 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas,rsnd: " Rob Herring 1 sibling, 1 reply; 9+ messages in thread From: Kuninori Morimoto @ 2020-08-27 0:25 UTC (permalink / raw) To: Rob Herring, Mark Brown; +Cc: devicetree, alsa-devel From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> This patch switches from .txt base to .yaml base Document. It is still keeping detail explanations at .txt Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> --- .../bindings/sound/renesas,rsnd.txt | 520 ------------------ .../bindings/sound/renesas,rsnd.yaml | 454 +++++++++++++++ 2 files changed, 454 insertions(+), 520 deletions(-) create mode 100644 Documentation/devicetree/bindings/sound/renesas,rsnd.yaml diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.txt b/Documentation/devicetree/bindings/sound/renesas,rsnd.txt index b39743d3f7c4..b731f16aea84 100644 --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.txt +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.txt @@ -253,523 +253,3 @@ This is example of TDM 6ch. Driver can automatically switches TDM <-> stereo mode in this case. see "Example: simple sound card for TDM" - -============================================= -Required properties: -============================================= - -- compatible : "renesas,rcar_sound-<soctype>", fallbacks - "renesas,rcar_sound-gen1" if generation1, and - "renesas,rcar_sound-gen2" if generation2 (or RZ/G1) - "renesas,rcar_sound-gen3" if generation3 (or RZ/G2) - Examples with soctypes are: - - "renesas,rcar_sound-r8a7742" (RZ/G1H) - - "renesas,rcar_sound-r8a7743" (RZ/G1M) - - "renesas,rcar_sound-r8a7744" (RZ/G1N) - - "renesas,rcar_sound-r8a7745" (RZ/G1E) - - "renesas,rcar_sound-r8a77470" (RZ/G1C) - - "renesas,rcar_sound-r8a774a1" (RZ/G2M) - - "renesas,rcar_sound-r8a774b1" (RZ/G2N) - - "renesas,rcar_sound-r8a774c0" (RZ/G2E) - - "renesas,rcar_sound-r8a774e1" (RZ/G2H) - - "renesas,rcar_sound-r8a7778" (R-Car M1A) - - "renesas,rcar_sound-r8a7779" (R-Car H1) - - "renesas,rcar_sound-r8a7790" (R-Car H2) - - "renesas,rcar_sound-r8a7791" (R-Car M2-W) - - "renesas,rcar_sound-r8a7793" (R-Car M2-N) - - "renesas,rcar_sound-r8a7794" (R-Car E2) - - "renesas,rcar_sound-r8a7795" (R-Car H3) - - "renesas,rcar_sound-r8a7796" (R-Car M3-W) - - "renesas,rcar_sound-r8a77965" (R-Car M3-N) - - "renesas,rcar_sound-r8a77990" (R-Car E3) - - "renesas,rcar_sound-r8a77995" (R-Car D3) -- reg : Should contain the register physical address. - required register is - SRU/ADG/SSI if generation1 - SRU/ADG/SSIU/SSI/AUDIO-DMAC-periperi if generation2/generation3 - Select extended AUDIO-DMAC-periperi address if SoC has it, - otherwise select normal AUDIO-DMAC-periperi address. -- reg-names : Should contain the register names. - scu/adg/ssi if generation1 - scu/adg/ssiu/ssi/audmapp if generation2/generation3 -- rcar_sound,ssi : Should contain SSI feature. - The number of SSI subnode should be same as HW. - see below for detail. -- rcar_sound,ssiu : Should contain SSIU feature. - The number of SSIU subnode should be same as HW. - see below for detail. -- rcar_sound,src : Should contain SRC feature. - The number of SRC subnode should be same as HW. - see below for detail. -- rcar_sound,ctu : Should contain CTU feature. - The number of CTU subnode should be same as HW. - see below for detail. -- rcar_sound,mix : Should contain MIX feature. - The number of MIX subnode should be same as HW. - see below for detail. -- rcar_sound,dvc : Should contain DVC feature. - The number of DVC subnode should be same as HW. - see below for detail. -- rcar_sound,dai : DAI contents. - The number of DAI subnode should be same as HW. - see below for detail. -- #sound-dai-cells : it must be 0 if your system is using single DAI - it must be 1 if your system is using multi DAI -- clocks : References to SSI/SRC/MIX/CTU/DVC/AUDIO_CLK clocks. -- clock-names : List of necessary clock names. - "ssi-all", "ssi.X", "src.X", "mix.X", "ctu.X", - "dvc.X", "clk_a", "clk_b", "clk_c", "clk_i" - -Optional properties: -- #clock-cells : it must be 0 if your system has audio_clkout - it must be 1 if your system has audio_clkout0/1/2/3 -- clock-frequency : for all audio_clkout0/1/2/3 -- clkout-lr-asynchronous : boolean property. it indicates that audio_clkoutn - is asynchronizes with lr-clock. -- resets : References to SSI resets. -- reset-names : List of valid reset names. - "ssi-all", "ssi.X" - -SSI subnode properties: -- interrupts : Should contain SSI interrupt for PIO transfer -- shared-pin : if shared clock pin -- pio-transfer : use PIO transfer mode -- no-busif : BUSIF is not ussed when [mem -> SSI] via DMA case -- dma : Should contain Audio DMAC entry -- dma-names : SSI case "rx" (=playback), "tx" (=capture) - Deprecated: see SSIU subnode properties - SSIU case "rxu" (=playback), "txu" (=capture) - -SSIU subnode properties: -- dma : Should contain Audio DMAC entry -- dma-names : "rx" (=playback), "tx" (=capture) - -SRC subnode properties: -- dma : Should contain Audio DMAC entry -- dma-names : "rx" (=playback), "tx" (=capture) - -DVC subnode properties: -- dma : Should contain Audio DMAC entry -- dma-names : "tx" (=playback/capture) - -DAI subnode properties: -- playback : list of playback modules -- capture : list of capture modules - - -============================================= -Example: -============================================= - -rcar_sound: sound@ec500000 { - #sound-dai-cells = <1>; - compatible = "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2"; - reg = <0 0xec500000 0 0x1000>, /* SCU */ - <0 0xec5a0000 0 0x100>, /* ADG */ - <0 0xec540000 0 0x1000>, /* SSIU */ - <0 0xec541000 0 0x1280>, /* SSI */ - <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ - reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; - - clocks = <&mstp10_clks R8A7790_CLK_SSI_ALL>, - <&mstp10_clks R8A7790_CLK_SSI9>, <&mstp10_clks R8A7790_CLK_SSI8>, - <&mstp10_clks R8A7790_CLK_SSI7>, <&mstp10_clks R8A7790_CLK_SSI6>, - <&mstp10_clks R8A7790_CLK_SSI5>, <&mstp10_clks R8A7790_CLK_SSI4>, - <&mstp10_clks R8A7790_CLK_SSI3>, <&mstp10_clks R8A7790_CLK_SSI2>, - <&mstp10_clks R8A7790_CLK_SSI1>, <&mstp10_clks R8A7790_CLK_SSI0>, - <&mstp10_clks R8A7790_CLK_SCU_SRC9>, <&mstp10_clks R8A7790_CLK_SCU_SRC8>, - <&mstp10_clks R8A7790_CLK_SCU_SRC7>, <&mstp10_clks R8A7790_CLK_SCU_SRC6>, - <&mstp10_clks R8A7790_CLK_SCU_SRC5>, <&mstp10_clks R8A7790_CLK_SCU_SRC4>, - <&mstp10_clks R8A7790_CLK_SCU_SRC3>, <&mstp10_clks R8A7790_CLK_SCU_SRC2>, - <&mstp10_clks R8A7790_CLK_SCU_SRC1>, <&mstp10_clks R8A7790_CLK_SCU_SRC0>, - <&mstp10_clks R8A7790_CLK_SCU_DVC0>, <&mstp10_clks R8A7790_CLK_SCU_DVC1>, - <&audio_clk_a>, <&audio_clk_b>, <&audio_clk_c>, <&m2_clk>; - clock-names = "ssi-all", - "ssi.9", "ssi.8", "ssi.7", "ssi.6", "ssi.5", - "ssi.4", "ssi.3", "ssi.2", "ssi.1", "ssi.0", - "src.9", "src.8", "src.7", "src.6", "src.5", - "src.4", "src.3", "src.2", "src.1", "src.0", - "dvc.0", "dvc.1", - "clk_a", "clk_b", "clk_c", "clk_i"; - - rcar_sound,dvc { - dvc0: dvc-0 { - dmas = <&audma0 0xbc>; - dma-names = "tx"; - }; - dvc1: dvc-1 { - dmas = <&audma0 0xbe>; - dma-names = "tx"; - }; - }; - - rcar_sound,mix { - mix0: mix-0 { }; - mix1: mix-1 { }; - }; - - rcar_sound,ctu { - ctu00: ctu-0 { }; - ctu01: ctu-1 { }; - ctu02: ctu-2 { }; - ctu03: ctu-3 { }; - ctu10: ctu-4 { }; - ctu11: ctu-5 { }; - ctu12: ctu-6 { }; - ctu13: ctu-7 { }; - }; - - rcar_sound,src { - src0: src-0 { - interrupts = <0 352 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x85>, <&audma1 0x9a>; - dma-names = "rx", "tx"; - }; - src1: src-1 { - interrupts = <0 353 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x87>, <&audma1 0x9c>; - dma-names = "rx", "tx"; - }; - src2: src-2 { - interrupts = <0 354 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x89>, <&audma1 0x9e>; - dma-names = "rx", "tx"; - }; - src3: src-3 { - interrupts = <0 355 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x8b>, <&audma1 0xa0>; - dma-names = "rx", "tx"; - }; - src4: src-4 { - interrupts = <0 356 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x8d>, <&audma1 0xb0>; - dma-names = "rx", "tx"; - }; - src5: src-5 { - interrupts = <0 357 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x8f>, <&audma1 0xb2>; - dma-names = "rx", "tx"; - }; - src6: src-6 { - interrupts = <0 358 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x91>, <&audma1 0xb4>; - dma-names = "rx", "tx"; - }; - src7: src-7 { - interrupts = <0 359 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x93>, <&audma1 0xb6>; - dma-names = "rx", "tx"; - }; - src8: src-8 { - interrupts = <0 360 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x95>, <&audma1 0xb8>; - dma-names = "rx", "tx"; - }; - src9: src-9 { - interrupts = <0 361 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x97>, <&audma1 0xba>; - dma-names = "rx", "tx"; - }; - }; - - rcar_sound,ssiu { - ssiu00: ssiu-0 { - dmas = <&audma0 0x15>, <&audma1 0x16>; - dma-names = "rx", "tx"; - }; - ssiu01: ssiu-1 { - dmas = <&audma0 0x35>, <&audma1 0x36>; - dma-names = "rx", "tx"; - }; - - ... - - ssiu95: ssiu-49 { - dmas = <&audma0 0xA5>, <&audma1 0xA6>; - dma-names = "rx", "tx"; - }; - ssiu96: ssiu-50 { - dmas = <&audma0 0xA7>, <&audma1 0xA8>; - dma-names = "rx", "tx"; - }; - ssiu97: ssiu-51 { - dmas = <&audma0 0xA9>, <&audma1 0xAA>; - dma-names = "rx", "tx"; - }; - }; - - rcar_sound,ssi { - ssi0: ssi-0 { - interrupts = <0 370 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x01>, <&audma1 0x02>; - dma-names = "rx", "tx"; - }; - ssi1: ssi-1 { - interrupts = <0 371 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x03>, <&audma1 0x04>; - dma-names = "rx", "tx"; - }; - - ... - - ssi8: ssi-8 { - interrupts = <0 378 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x11>, <&audma1 0x12>; - dma-names = "rx", "tx"; - }; - ssi9: ssi-9 { - interrupts = <0 379 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x13>, <&audma1 0x14>; - dma-names = "rx", "tx"; - }; - }; - - rcar_sound,dai { - dai0 { - playback = <&ssi5 &src5>; - capture = <&ssi6>; - }; - dai1 { - playback = <&ssi3>; - }; - dai2 { - capture = <&ssi4>; - }; - dai3 { - playback = <&ssi7>; - }; - dai4 { - capture = <&ssi8>; - }; - }; -}; - -============================================= -Example: simple sound card -============================================= - - rsnd_ak4643: sound { - compatible = "simple-audio-card"; - - simple-audio-card,format = "left_j"; - simple-audio-card,bitclock-master = <&sndcodec>; - simple-audio-card,frame-master = <&sndcodec>; - - sndcpu: simple-audio-card,cpu { - sound-dai = <&rcar_sound>; - }; - - sndcodec: simple-audio-card,codec { - sound-dai = <&ak4643>; - clocks = <&audio_clock>; - }; - }; - -&rcar_sound { - pinctrl-0 = <&sound_pins &sound_clk_pins>; - pinctrl-names = "default"; - - /* Single DAI */ - #sound-dai-cells = <0>; - - - rcar_sound,dai { - dai0 { - playback = <&ssi0 &src2 &dvc0>; - capture = <&ssi1 &src3 &dvc1>; - }; - }; -}; - -&ssi1 { - shared-pin; -}; - -============================================= -Example: simple sound card for Asynchronous mode -============================================= - -sound { - compatible = "simple-scu-audio-card"; - ... - /* - * SRC Asynchronous mode setting - * Playback: - * All input data will be converted to 48kHz - * Capture: - * Inputed 48kHz data will be converted to - * system specified Hz - */ - simple-audio-card,convert-rate = <48000>; - ... - simple-audio-card,cpu { - sound-dai = <&rcar_sound>; - }; - simple-audio-card,codec { - ... - }; -}; - -============================================= -Example: simple sound card for channel convert -============================================= - -sound { - compatible = "simple-scu-audio-card"; - ... - /* - * CTU setting - * All input data will be converted to 2ch - * as output data - */ - simple-audio-card,convert-channels = <2>; - ... - simple-audio-card,cpu { - sound-dai = <&rcar_sound>; - }; - simple-audio-card,codec { - ... - }; -}; - -============================================= -Example: simple sound card for MIXer -============================================= - -sound { - compatible = "simple-scu-audio-card"; - ... - simple-audio-card,cpu@0 { - sound-dai = <&rcar_sound 0>; - }; - simple-audio-card,cpu@1 { - sound-dai = <&rcar_sound 1>; - }; - simple-audio-card,codec { - ... - }; -}; - -&rcar_sound { - ... - rcar_sound,dai { - dai0 { - playback = <&src1 &ctu02 &mix0 &dvc0 &ssi0>; - }; - dai1 { - playback = <&src2 &ctu03 &mix0 &dvc0 &ssi0>; - }; - }; -}; - -============================================= -Example: simple sound card for TDM -============================================= - -rsnd_tdm: sound { - compatible = "simple-audio-card"; - - simple-audio-card,format = "left_j"; - simple-audio-card,bitclock-master = <&sndcodec>; - simple-audio-card,frame-master = <&sndcodec>; - - sndcpu: simple-audio-card,cpu { - sound-dai = <&rcar_sound>; - dai-tdm-slot-num = <6>; - }; - - sndcodec: simple-audio-card,codec { - sound-dai = <&xxx>; - }; -}; - -============================================= -Example: simple sound card for TDM Split -============================================= - -sound_card: sound { - compatible = "audio-graph-scu-card"; - prefix = "xxxx"; - routing = "xxxx Playback", "DAI0 Playback", - "xxxx Playback", "DAI1 Playback", - "xxxx Playback", "DAI2 Playback", - "xxxx Playback", "DAI3 Playback"; - convert-channels = <8>; /* TDM Split */ - - dais = <&rsnd_port0 /* playback ch1/ch2 */ - &rsnd_port1 /* playback ch3/ch4 */ - &rsnd_port2 /* playback ch5/ch6 */ - &rsnd_port3 /* playback ch7/ch8 */ - >; -}; - -audio-codec { - ... - port { - codec_0: endpoint@1 { - remote-endpoint = <&rsnd_ep0>; - }; - codec_1: endpoint@2 { - remote-endpoint = <&rsnd_ep1>; - }; - codec_2: endpoint@3 { - remote-endpoint = <&rsnd_ep2>; - }; - codec_3: endpoint@4 { - remote-endpoint = <&rsnd_ep3>; - }; - }; -}; - -&rcar_sound { - ... - ports { - rsnd_port0: port@0 { - rsnd_ep0: endpoint { - remote-endpoint = <&codec_0>; - ... - playback = <&ssiu30 &ssi3>; - }; - }; - rsnd_port1: port@1 { - rsnd_ep1: endpoint { - remote-endpoint = <&codec_1>; - ... - playback = <&ssiu31 &ssi3>; - }; - }; - rsnd_port2: port@2 { - rsnd_ep2: endpoint { - remote-endpoint = <&codec_2>; - ... - playback = <&ssiu32 &ssi3>; - }; - }; - rsnd_port3: port@3 { - rsnd_ep3: endpoint { - remote-endpoint = <&codec_3>; - ... - playback = <&ssiu33 &ssi3>; - }; - }; - }; -}; - -============================================= -Example: simple sound card for Multi channel -============================================= - -&rcar_sound { - pinctrl-0 = <&sound_pins &sound_clk_pins>; - pinctrl-names = "default"; - - /* Single DAI */ - #sound-dai-cells = <0>; - - - rcar_sound,dai { - dai0 { - playback = <&ssi0 &ssi1 &ssi2 &src0 &dvc0>; - }; - }; -}; diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml new file mode 100644 index 000000000000..3f37f314b580 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml @@ -0,0 +1,454 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/renesas,rsnd.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Renesas R-Car Sound Driver Device Tree Bindings + +maintainers: + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> + +properties: + + compatible: + oneOf: + # for Gen1 SoC + - items: + - enum: + - renesas,rcar_sound-r8a7778 # R-Car M1A + - renesas,rcar_sound-r8a7779 # R-Car H1 + - enum: + - renesas,rcar_sound-gen1 + # for Gen2 SoC + - items: + - enum: + - renesas,rcar_sound-r8a7742 # RZ/G1H + - renesas,rcar_sound-r8a7743 # RZ/G1M + - renesas,rcar_sound-r8a7744 # RZ/G1N + - renesas,rcar_sound-r8a7745 # RZ/G1E + - renesas,rcar_sound-r8a77470 # RZ/G1C + - renesas,rcar_sound-r8a7790 # R-Car H2 + - renesas,rcar_sound-r8a7791 # R-Car M2-W + - renesas,rcar_sound-r8a7793 # R-Car M2-N + - renesas,rcar_sound-r8a7794 # R-Car E2 + - enum: + - renesas,rcar_sound-gen2 + # for Gen3 SoC + - items: + - enum: + - renesas,rcar_sound-r8a774a1 # RZ/G2M + - renesas,rcar_sound-r8a774b1 # RZ/G2N + - renesas,rcar_sound-r8a774c0 # RZ/G2E + - renesas,rcar_sound-r8a774e1 # RZ/G2H + - renesas,rcar_sound-r8a7795 # R-Car H3 + - renesas,rcar_sound-r8a7796 # R-Car M3-W + - renesas,rcar_sound-r8a77965 # R-Car M3-N + - renesas,rcar_sound-r8a77990 # R-Car E3 + - renesas,rcar_sound-r8a77995 # R-Car D3 + - enum: + - renesas,rcar_sound-gen3 + # for Generic + - items: + - enum: + - renesas,rcar_sound-gen1 + - renesas,rcar_sound-gen2 + - renesas,rcar_sound-gen3 + + reg: + minItems: 3 # Gen1 + maxItems: 5 # Gen2/Gen3 + + reg-names: + description: | + Should contain the register names. + scu/adg/ssi if generation1 + scu/adg/ssiu/ssi/audmapp if generation2/generation3 + minItems: 3 # Gen1 + maxItems: 5 # Gen2/Gen3 + oneOf: + # for Gen1 + - allOf: + - items: + enum: + - scu + - adg + - ssi + # for Gen2/Gen3 + - allOf: + - items: + enum: + - scu + - adg + - ssiu + - ssi + - audmapp + + "#sound-dai-cells": + description: | + it must be 0 if your system is using single DAI + it must be 1 if your system is using multi DAIs + enum: [0, 1] + + "#clock-cells": + description: | + it must be 0 if your system has audio_clkout + it must be 1 if your system has audio_clkout0/1/2/3 + enum: [0, 1] + + clock-frequency: + description: for audio_clkout0/1/2/3 + $ref: /schemas/types.yaml#/definitions/uint32-array + + clkout-lr-asynchronous: + description: audio_clkoutn is asynchronizes with lr-clock. + $ref: /schemas/types.yaml#/definitions/flag + + power-domains: true + + resets: + description: References to SSI resets + $ref: /schemas/types.yaml#/definitions/phandle-array + + reset-names: + description: List of valid reset names. + allOf: + - items: + oneOf: + - pattern: '^ssi(-all|\.[0-9])$' + + clocks: + description: References to SSI/SRC/MIX/CTU/DVC/AUDIO_CLK clocks. + minItems: 1 + maxItems: 31 + + clock-names: + description: List of necessary clock names. + minItems: 1 + maxItems: 31 + allOf: + - items: + oneOf: + - const: ssi-all + - pattern: '^ssi\.[0-9]$' + - pattern: '^src\.[0-9]$' + - pattern: '^mix\.[0-1]$' + - pattern: '^ctu\.[0-1]$' + - pattern: '^dvc\.[0-1]$' + - pattern: '^clk_(a|b|c|i)$' + + # For OF-graph + port: + description: OF-Graph subnode + $ref: "audio-graph-card.yaml#/definitions/port" + + # For multi OF-graph + ports: + description: multi OF-Graph subnode + $ref: "audio-graph-card.yaml#/definitions/ports" + +# use patternProperties to avoid naming "xxx,yyy" +patternProperties: + "^rcar_sound,dvc$": + description: DVC subnode. + type: object + patternProperties: + "^dvc-[0-1]$": + type: object + properties: + dmas: + maxItems: 1 + dma-names: + const: "tx" + required: + - dmas + - dma-names + additionalProperties: false + + "^rcar_sound,mix$": + description: MIX subnode. + type: object + patternProperties: + "^mix-[0-1]$": + type: object + # no propaties + additionalProperties: false + + "^rcar_sound,ctu$": + description: CTU subnode. + type: object + patternProperties: + "^ctu-[0-7]$": + type: object + # no propaties + additionalProperties: false + + "^rcar_sound,src$": + description: SRC subnode. + type: object + patternProperties: + "^src-[0-9]$": + type: object + properties: + interrupts: + maxItems: 1 + dmas: + maxItems: 2 + dma-names: + allOf: + - items: + enum: + - tx + - rx + allOf: + - if: + properties: + status: + const: "ok" + then: + required: + - interrupts + - dmas + - dma-names + additionalProperties: false + + "^rcar_sound,ssiu$": + description: SSIU subnode. + type: object + patternProperties: + "^ssiu-[0-9]+$": + type: object + properties: + dmas: + maxItems: 2 + dma-names: + allOf: + - items: + enum: + - tx + - rx + required: + - dmas + - dma-names + additionalProperties: false + + "^rcar_sound,ssi$": + description: SSI subnode. + type: object + patternProperties: + "^ssi-[0-9]$": + type: object + properties: + interrupts: + maxItems: 1 + dmas: + minItems: 2 + maxItems: 4 + dma-names: + allOf: + - items: + enum: + - tx + - rx + - txu # if no ssiu node + - rxu # if no ssiu node + + shared-pin: + description: shared clock pin + $ref: /schemas/types.yaml#/definitions/flag + pio-transfer: + description: PIO transfer mode + $ref: /schemas/types.yaml#/definitions/flag + no-busif: + description: BUSIF is not used when [mem -> SSI] via DMA case + $ref: /schemas/types.yaml#/definitions/flag + required: + - interrupts + - dmas + - dma-names + additionalProperties: false + + # For DAI base + "^rcar_sound,dai$": + description: DAI subnode. + type: object + patternProperties: + "^dai([0-9]+)?$": + type: object + properties: + playback: + $ref: /schemas/types.yaml#/definitions/phandle-array + capture: + $ref: /schemas/types.yaml#/definitions/phandle-array + anyOf: + - required: + - playback + - required: + - capture + additionalProperties: false + +required: + - compatible + - reg + - reg-names + - clocks + - clock-names + - "#sound-dai-cells" + +additionalProperties: false + +examples: + - | + rcar_sound: sound@ec500000 { + #sound-dai-cells = <1>; + compatible = "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2"; + reg = <0xec500000 0x1000>, /* SCU */ + <0xec5a0000 0x100>, /* ADG */ + <0xec540000 0x1000>, /* SSIU */ + <0xec541000 0x1280>, /* SSI */ + <0xec740000 0x200>; /* Audio DMAC peri peri*/ + reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; + + clocks = <&mstp10_clks 1005>, /* SSI-ALL */ + <&mstp10_clks 1006>, <&mstp10_clks 1007>, /* SSI9, SSI8 */ + <&mstp10_clks 1008>, <&mstp10_clks 1009>, /* SSI7, SSI6 */ + <&mstp10_clks 1010>, <&mstp10_clks 1011>, /* SSI5, SSI4 */ + <&mstp10_clks 1012>, <&mstp10_clks 1013>, /* SSI3, SSI2 */ + <&mstp10_clks 1014>, <&mstp10_clks 1015>, /* SSI1, SSI0 */ + <&mstp10_clks 1022>, <&mstp10_clks 1023>, /* SRC9, SRC8 */ + <&mstp10_clks 1024>, <&mstp10_clks 1025>, /* SRC7, SRC6 */ + <&mstp10_clks 1026>, <&mstp10_clks 1027>, /* SRC5, SRC4 */ + <&mstp10_clks 1028>, <&mstp10_clks 1029>, /* SRC3, SRC2 */ + <&mstp10_clks 1030>, <&mstp10_clks 1031>, /* SRC1, SRC0 */ + <&mstp10_clks 1020>, <&mstp10_clks 1021>, /* MIX1, MIX0 */ + <&mstp10_clks 1020>, <&mstp10_clks 1021>, /* CTU1, CTU0 */ + <&mstp10_clks 1019>, <&mstp10_clks 1018>, /* DVC0, DVC1 */ + <&audio_clk_a>, <&audio_clk_b>, /* CLKA, CLKB */ + <&audio_clk_c>, <&audio_clk_i>; /* CLKC, CLKI */ + + clock-names = "ssi-all", + "ssi.9", "ssi.8", + "ssi.7", "ssi.6", + "ssi.5", "ssi.4", + "ssi.3", "ssi.2", + "ssi.1", "ssi.0", + "src.9", "src.8", + "src.7", "src.6", + "src.5", "src.4", + "src.3", "src.2", + "src.1", "src.0", + "mix.1", "mix.0", + "ctu.1", "ctu.0", + "dvc.0", "dvc.1", + "clk_a", "clk_b", + "clk_c", "clk_i"; + + rcar_sound,dvc { + dvc0: dvc-0 { + dmas = <&audma0 0xbc>; + dma-names = "tx"; + }; + dvc1: dvc-1 { + dmas = <&audma0 0xbe>; + dma-names = "tx"; + }; + }; + + rcar_sound,mix { + mix0: mix-0 { }; + mix1: mix-1 { }; + }; + + rcar_sound,ctu { + ctu00: ctu-0 { }; + ctu01: ctu-1 { }; + ctu02: ctu-2 { }; + ctu03: ctu-3 { }; + ctu10: ctu-4 { }; + ctu11: ctu-5 { }; + ctu12: ctu-6 { }; + ctu13: ctu-7 { }; + }; + + rcar_sound,src { + src0: src-0 { + status = "disabled"; + }; + src1: src-1 { + interrupts = <0 353 0>; + dmas = <&audma0 0x87>, <&audma1 0x9c>; + dma-names = "rx", "tx"; + }; + /* skip after src-2 */ + }; + + rcar_sound,ssiu { + ssiu00: ssiu-0 { + dmas = <&audma0 0x15>, <&audma1 0x16>; + dma-names = "rx", "tx"; + }; + ssiu01: ssiu-1 { + dmas = <&audma0 0x35>, <&audma1 0x36>; + dma-names = "rx", "tx"; + }; + /* skip after ssiu-2 */ + }; + + rcar_sound,ssi { + ssi0: ssi-0 { + interrupts = <0 370 1>; + dmas = <&audma0 0x01>, <&audma1 0x02>; + dma-names = "rx", "tx"; + }; + ssi1: ssi-1 { + interrupts = <0 371 1>; + dmas = <&audma0 0x03>, <&audma1 0x04>; + dma-names = "rx", "tx"; + }; + /* skip other ssi-2 */ + }; + + /* DAI base */ + rcar_sound,dai { + dai0 { + playback = <&ssi5 &src5>; + capture = <&ssi6>; + }; + dai1 { + playback = <&ssi3>; + }; + dai2 { + capture = <&ssi4>; + }; + dai3 { + playback = <&ssi7>; + }; + dai4 { + capture = <&ssi8>; + }; + }; + + /* assume audio-graph */ + port { + rsnd_endpoint: endpoint { + remote-endpoint = <&codec_endpoint>; + + dai-format = "left_j"; + bitclock-master = <&rsnd_endpoint0>; + frame-master = <&rsnd_endpoint0>; + + playback = <&ssi0 &src0 &dvc0>; + capture = <&ssi1 &src1 &dvc1>; + }; + }; + }; + + + /* assume audio-graph */ + codec { + port { + codec_endpoint: endpoint { + remote-endpoint = <&rsnd_endpoint>; + }; + }; + }; -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] ASoC: dt-bindings: renesas,rsnd: switch to yaml base Documentation 2020-08-27 0:25 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto @ 2020-09-08 23:22 ` Rob Herring [not found] ` <87eemrlx15.wl-kuninori.morimoto.gx@renesas.com> 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2020-09-08 23:22 UTC (permalink / raw) To: Kuninori Morimoto; +Cc: devicetree, alsa-devel, Mark Brown On Thu, Aug 27, 2020 at 09:25:12AM +0900, Kuninori Morimoto wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > This patch switches from .txt base to .yaml base Document. > It is still keeping detail explanations at .txt > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > .../bindings/sound/renesas,rsnd.txt | 520 ------------------ > .../bindings/sound/renesas,rsnd.yaml | 454 +++++++++++++++ > 2 files changed, 454 insertions(+), 520 deletions(-) > create mode 100644 Documentation/devicetree/bindings/sound/renesas,rsnd.yaml > > diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.txt b/Documentation/devicetree/bindings/sound/renesas,rsnd.txt > index b39743d3f7c4..b731f16aea84 100644 > --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.txt > +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.txt > @@ -253,523 +253,3 @@ This is example of TDM 6ch. > Driver can automatically switches TDM <-> stereo mode in this case. > > see "Example: simple sound card for TDM" > - > -============================================= > -Required properties: > -============================================= > - > -- compatible : "renesas,rcar_sound-<soctype>", fallbacks > - "renesas,rcar_sound-gen1" if generation1, and > - "renesas,rcar_sound-gen2" if generation2 (or RZ/G1) > - "renesas,rcar_sound-gen3" if generation3 (or RZ/G2) > - Examples with soctypes are: > - - "renesas,rcar_sound-r8a7742" (RZ/G1H) > - - "renesas,rcar_sound-r8a7743" (RZ/G1M) > - - "renesas,rcar_sound-r8a7744" (RZ/G1N) > - - "renesas,rcar_sound-r8a7745" (RZ/G1E) > - - "renesas,rcar_sound-r8a77470" (RZ/G1C) > - - "renesas,rcar_sound-r8a774a1" (RZ/G2M) > - - "renesas,rcar_sound-r8a774b1" (RZ/G2N) > - - "renesas,rcar_sound-r8a774c0" (RZ/G2E) > - - "renesas,rcar_sound-r8a774e1" (RZ/G2H) > - - "renesas,rcar_sound-r8a7778" (R-Car M1A) > - - "renesas,rcar_sound-r8a7779" (R-Car H1) > - - "renesas,rcar_sound-r8a7790" (R-Car H2) > - - "renesas,rcar_sound-r8a7791" (R-Car M2-W) > - - "renesas,rcar_sound-r8a7793" (R-Car M2-N) > - - "renesas,rcar_sound-r8a7794" (R-Car E2) > - - "renesas,rcar_sound-r8a7795" (R-Car H3) > - - "renesas,rcar_sound-r8a7796" (R-Car M3-W) > - - "renesas,rcar_sound-r8a77965" (R-Car M3-N) > - - "renesas,rcar_sound-r8a77990" (R-Car E3) > - - "renesas,rcar_sound-r8a77995" (R-Car D3) > -- reg : Should contain the register physical address. > - required register is > - SRU/ADG/SSI if generation1 > - SRU/ADG/SSIU/SSI/AUDIO-DMAC-periperi if generation2/generation3 > - Select extended AUDIO-DMAC-periperi address if SoC has it, > - otherwise select normal AUDIO-DMAC-periperi address. > -- reg-names : Should contain the register names. > - scu/adg/ssi if generation1 > - scu/adg/ssiu/ssi/audmapp if generation2/generation3 > -- rcar_sound,ssi : Should contain SSI feature. > - The number of SSI subnode should be same as HW. > - see below for detail. > -- rcar_sound,ssiu : Should contain SSIU feature. > - The number of SSIU subnode should be same as HW. > - see below for detail. > -- rcar_sound,src : Should contain SRC feature. > - The number of SRC subnode should be same as HW. > - see below for detail. > -- rcar_sound,ctu : Should contain CTU feature. > - The number of CTU subnode should be same as HW. > - see below for detail. > -- rcar_sound,mix : Should contain MIX feature. > - The number of MIX subnode should be same as HW. > - see below for detail. > -- rcar_sound,dvc : Should contain DVC feature. > - The number of DVC subnode should be same as HW. > - see below for detail. > -- rcar_sound,dai : DAI contents. > - The number of DAI subnode should be same as HW. > - see below for detail. > -- #sound-dai-cells : it must be 0 if your system is using single DAI > - it must be 1 if your system is using multi DAI > -- clocks : References to SSI/SRC/MIX/CTU/DVC/AUDIO_CLK clocks. > -- clock-names : List of necessary clock names. > - "ssi-all", "ssi.X", "src.X", "mix.X", "ctu.X", > - "dvc.X", "clk_a", "clk_b", "clk_c", "clk_i" > - > -Optional properties: > -- #clock-cells : it must be 0 if your system has audio_clkout > - it must be 1 if your system has audio_clkout0/1/2/3 > -- clock-frequency : for all audio_clkout0/1/2/3 > -- clkout-lr-asynchronous : boolean property. it indicates that audio_clkoutn > - is asynchronizes with lr-clock. > -- resets : References to SSI resets. > -- reset-names : List of valid reset names. > - "ssi-all", "ssi.X" > - > -SSI subnode properties: > -- interrupts : Should contain SSI interrupt for PIO transfer > -- shared-pin : if shared clock pin > -- pio-transfer : use PIO transfer mode > -- no-busif : BUSIF is not ussed when [mem -> SSI] via DMA case > -- dma : Should contain Audio DMAC entry > -- dma-names : SSI case "rx" (=playback), "tx" (=capture) > - Deprecated: see SSIU subnode properties > - SSIU case "rxu" (=playback), "txu" (=capture) > - > -SSIU subnode properties: > -- dma : Should contain Audio DMAC entry > -- dma-names : "rx" (=playback), "tx" (=capture) > - > -SRC subnode properties: > -- dma : Should contain Audio DMAC entry > -- dma-names : "rx" (=playback), "tx" (=capture) > - > -DVC subnode properties: > -- dma : Should contain Audio DMAC entry > -- dma-names : "tx" (=playback/capture) > - > -DAI subnode properties: > -- playback : list of playback modules > -- capture : list of capture modules > - > - > -============================================= > -Example: > -============================================= > - > -rcar_sound: sound@ec500000 { > - #sound-dai-cells = <1>; > - compatible = "renesas,rcar_sound-r8a7791", "renesas,rcar_sound-gen2"; > - reg = <0 0xec500000 0 0x1000>, /* SCU */ > - <0 0xec5a0000 0 0x100>, /* ADG */ > - <0 0xec540000 0 0x1000>, /* SSIU */ > - <0 0xec541000 0 0x1280>, /* SSI */ > - <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ > - reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; > - > - clocks = <&mstp10_clks R8A7790_CLK_SSI_ALL>, > - <&mstp10_clks R8A7790_CLK_SSI9>, <&mstp10_clks R8A7790_CLK_SSI8>, > - <&mstp10_clks R8A7790_CLK_SSI7>, <&mstp10_clks R8A7790_CLK_SSI6>, > - <&mstp10_clks R8A7790_CLK_SSI5>, <&mstp10_clks R8A7790_CLK_SSI4>, > - <&mstp10_clks R8A7790_CLK_SSI3>, <&mstp10_clks R8A7790_CLK_SSI2>, > - <&mstp10_clks R8A7790_CLK_SSI1>, <&mstp10_clks R8A7790_CLK_SSI0>, > - <&mstp10_clks R8A7790_CLK_SCU_SRC9>, <&mstp10_clks R8A7790_CLK_SCU_SRC8>, > - <&mstp10_clks R8A7790_CLK_SCU_SRC7>, <&mstp10_clks R8A7790_CLK_SCU_SRC6>, > - <&mstp10_clks R8A7790_CLK_SCU_SRC5>, <&mstp10_clks R8A7790_CLK_SCU_SRC4>, > - <&mstp10_clks R8A7790_CLK_SCU_SRC3>, <&mstp10_clks R8A7790_CLK_SCU_SRC2>, > - <&mstp10_clks R8A7790_CLK_SCU_SRC1>, <&mstp10_clks R8A7790_CLK_SCU_SRC0>, > - <&mstp10_clks R8A7790_CLK_SCU_DVC0>, <&mstp10_clks R8A7790_CLK_SCU_DVC1>, > - <&audio_clk_a>, <&audio_clk_b>, <&audio_clk_c>, <&m2_clk>; > - clock-names = "ssi-all", > - "ssi.9", "ssi.8", "ssi.7", "ssi.6", "ssi.5", > - "ssi.4", "ssi.3", "ssi.2", "ssi.1", "ssi.0", > - "src.9", "src.8", "src.7", "src.6", "src.5", > - "src.4", "src.3", "src.2", "src.1", "src.0", > - "dvc.0", "dvc.1", > - "clk_a", "clk_b", "clk_c", "clk_i"; > - > - rcar_sound,dvc { > - dvc0: dvc-0 { > - dmas = <&audma0 0xbc>; > - dma-names = "tx"; > - }; > - dvc1: dvc-1 { > - dmas = <&audma0 0xbe>; > - dma-names = "tx"; > - }; > - }; > - > - rcar_sound,mix { > - mix0: mix-0 { }; > - mix1: mix-1 { }; > - }; > - > - rcar_sound,ctu { > - ctu00: ctu-0 { }; > - ctu01: ctu-1 { }; > - ctu02: ctu-2 { }; > - ctu03: ctu-3 { }; > - ctu10: ctu-4 { }; > - ctu11: ctu-5 { }; > - ctu12: ctu-6 { }; > - ctu13: ctu-7 { }; > - }; > - > - rcar_sound,src { > - src0: src-0 { > - interrupts = <0 352 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x85>, <&audma1 0x9a>; > - dma-names = "rx", "tx"; > - }; > - src1: src-1 { > - interrupts = <0 353 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x87>, <&audma1 0x9c>; > - dma-names = "rx", "tx"; > - }; > - src2: src-2 { > - interrupts = <0 354 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x89>, <&audma1 0x9e>; > - dma-names = "rx", "tx"; > - }; > - src3: src-3 { > - interrupts = <0 355 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x8b>, <&audma1 0xa0>; > - dma-names = "rx", "tx"; > - }; > - src4: src-4 { > - interrupts = <0 356 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x8d>, <&audma1 0xb0>; > - dma-names = "rx", "tx"; > - }; > - src5: src-5 { > - interrupts = <0 357 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x8f>, <&audma1 0xb2>; > - dma-names = "rx", "tx"; > - }; > - src6: src-6 { > - interrupts = <0 358 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x91>, <&audma1 0xb4>; > - dma-names = "rx", "tx"; > - }; > - src7: src-7 { > - interrupts = <0 359 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x93>, <&audma1 0xb6>; > - dma-names = "rx", "tx"; > - }; > - src8: src-8 { > - interrupts = <0 360 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x95>, <&audma1 0xb8>; > - dma-names = "rx", "tx"; > - }; > - src9: src-9 { > - interrupts = <0 361 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x97>, <&audma1 0xba>; > - dma-names = "rx", "tx"; > - }; > - }; > - > - rcar_sound,ssiu { > - ssiu00: ssiu-0 { > - dmas = <&audma0 0x15>, <&audma1 0x16>; > - dma-names = "rx", "tx"; > - }; > - ssiu01: ssiu-1 { > - dmas = <&audma0 0x35>, <&audma1 0x36>; > - dma-names = "rx", "tx"; > - }; > - > - ... > - > - ssiu95: ssiu-49 { > - dmas = <&audma0 0xA5>, <&audma1 0xA6>; > - dma-names = "rx", "tx"; > - }; > - ssiu96: ssiu-50 { > - dmas = <&audma0 0xA7>, <&audma1 0xA8>; > - dma-names = "rx", "tx"; > - }; > - ssiu97: ssiu-51 { > - dmas = <&audma0 0xA9>, <&audma1 0xAA>; > - dma-names = "rx", "tx"; > - }; > - }; > - > - rcar_sound,ssi { > - ssi0: ssi-0 { > - interrupts = <0 370 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x01>, <&audma1 0x02>; > - dma-names = "rx", "tx"; > - }; > - ssi1: ssi-1 { > - interrupts = <0 371 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x03>, <&audma1 0x04>; > - dma-names = "rx", "tx"; > - }; > - > - ... > - > - ssi8: ssi-8 { > - interrupts = <0 378 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x11>, <&audma1 0x12>; > - dma-names = "rx", "tx"; > - }; > - ssi9: ssi-9 { > - interrupts = <0 379 IRQ_TYPE_LEVEL_HIGH>; > - dmas = <&audma0 0x13>, <&audma1 0x14>; > - dma-names = "rx", "tx"; > - }; > - }; > - > - rcar_sound,dai { > - dai0 { > - playback = <&ssi5 &src5>; > - capture = <&ssi6>; > - }; > - dai1 { > - playback = <&ssi3>; > - }; > - dai2 { > - capture = <&ssi4>; > - }; > - dai3 { > - playback = <&ssi7>; > - }; > - dai4 { > - capture = <&ssi8>; > - }; > - }; > -}; > - > -============================================= > -Example: simple sound card > -============================================= > - > - rsnd_ak4643: sound { > - compatible = "simple-audio-card"; > - > - simple-audio-card,format = "left_j"; > - simple-audio-card,bitclock-master = <&sndcodec>; > - simple-audio-card,frame-master = <&sndcodec>; > - > - sndcpu: simple-audio-card,cpu { > - sound-dai = <&rcar_sound>; > - }; > - > - sndcodec: simple-audio-card,codec { > - sound-dai = <&ak4643>; > - clocks = <&audio_clock>; > - }; > - }; > - > -&rcar_sound { > - pinctrl-0 = <&sound_pins &sound_clk_pins>; > - pinctrl-names = "default"; > - > - /* Single DAI */ > - #sound-dai-cells = <0>; > - > - > - rcar_sound,dai { > - dai0 { > - playback = <&ssi0 &src2 &dvc0>; > - capture = <&ssi1 &src3 &dvc1>; > - }; > - }; > -}; > - > -&ssi1 { > - shared-pin; > -}; > - > -============================================= > -Example: simple sound card for Asynchronous mode > -============================================= > - > -sound { > - compatible = "simple-scu-audio-card"; > - ... > - /* > - * SRC Asynchronous mode setting > - * Playback: > - * All input data will be converted to 48kHz > - * Capture: > - * Inputed 48kHz data will be converted to > - * system specified Hz > - */ > - simple-audio-card,convert-rate = <48000>; > - ... > - simple-audio-card,cpu { > - sound-dai = <&rcar_sound>; > - }; > - simple-audio-card,codec { > - ... > - }; > -}; > - > -============================================= > -Example: simple sound card for channel convert > -============================================= > - > -sound { > - compatible = "simple-scu-audio-card"; > - ... > - /* > - * CTU setting > - * All input data will be converted to 2ch > - * as output data > - */ > - simple-audio-card,convert-channels = <2>; > - ... > - simple-audio-card,cpu { > - sound-dai = <&rcar_sound>; > - }; > - simple-audio-card,codec { > - ... > - }; > -}; > - > -============================================= > -Example: simple sound card for MIXer > -============================================= > - > -sound { > - compatible = "simple-scu-audio-card"; > - ... > - simple-audio-card,cpu@0 { > - sound-dai = <&rcar_sound 0>; > - }; > - simple-audio-card,cpu@1 { > - sound-dai = <&rcar_sound 1>; > - }; > - simple-audio-card,codec { > - ... > - }; > -}; > - > -&rcar_sound { > - ... > - rcar_sound,dai { > - dai0 { > - playback = <&src1 &ctu02 &mix0 &dvc0 &ssi0>; > - }; > - dai1 { > - playback = <&src2 &ctu03 &mix0 &dvc0 &ssi0>; > - }; > - }; > -}; > - > -============================================= > -Example: simple sound card for TDM > -============================================= > - > -rsnd_tdm: sound { > - compatible = "simple-audio-card"; > - > - simple-audio-card,format = "left_j"; > - simple-audio-card,bitclock-master = <&sndcodec>; > - simple-audio-card,frame-master = <&sndcodec>; > - > - sndcpu: simple-audio-card,cpu { > - sound-dai = <&rcar_sound>; > - dai-tdm-slot-num = <6>; > - }; > - > - sndcodec: simple-audio-card,codec { > - sound-dai = <&xxx>; > - }; > -}; > - > -============================================= > -Example: simple sound card for TDM Split > -============================================= > - > -sound_card: sound { > - compatible = "audio-graph-scu-card"; > - prefix = "xxxx"; > - routing = "xxxx Playback", "DAI0 Playback", > - "xxxx Playback", "DAI1 Playback", > - "xxxx Playback", "DAI2 Playback", > - "xxxx Playback", "DAI3 Playback"; > - convert-channels = <8>; /* TDM Split */ > - > - dais = <&rsnd_port0 /* playback ch1/ch2 */ > - &rsnd_port1 /* playback ch3/ch4 */ > - &rsnd_port2 /* playback ch5/ch6 */ > - &rsnd_port3 /* playback ch7/ch8 */ > - >; > -}; > - > -audio-codec { > - ... > - port { > - codec_0: endpoint@1 { > - remote-endpoint = <&rsnd_ep0>; > - }; > - codec_1: endpoint@2 { > - remote-endpoint = <&rsnd_ep1>; > - }; > - codec_2: endpoint@3 { > - remote-endpoint = <&rsnd_ep2>; > - }; > - codec_3: endpoint@4 { > - remote-endpoint = <&rsnd_ep3>; > - }; > - }; > -}; > - > -&rcar_sound { > - ... > - ports { > - rsnd_port0: port@0 { > - rsnd_ep0: endpoint { > - remote-endpoint = <&codec_0>; > - ... > - playback = <&ssiu30 &ssi3>; > - }; > - }; > - rsnd_port1: port@1 { > - rsnd_ep1: endpoint { > - remote-endpoint = <&codec_1>; > - ... > - playback = <&ssiu31 &ssi3>; > - }; > - }; > - rsnd_port2: port@2 { > - rsnd_ep2: endpoint { > - remote-endpoint = <&codec_2>; > - ... > - playback = <&ssiu32 &ssi3>; > - }; > - }; > - rsnd_port3: port@3 { > - rsnd_ep3: endpoint { > - remote-endpoint = <&codec_3>; > - ... > - playback = <&ssiu33 &ssi3>; > - }; > - }; > - }; > -}; > - > -============================================= > -Example: simple sound card for Multi channel > -============================================= > - > -&rcar_sound { > - pinctrl-0 = <&sound_pins &sound_clk_pins>; > - pinctrl-names = "default"; > - > - /* Single DAI */ > - #sound-dai-cells = <0>; > - > - > - rcar_sound,dai { > - dai0 { > - playback = <&ssi0 &ssi1 &ssi2 &src0 &dvc0>; > - }; > - }; > -}; > diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml > new file mode 100644 > index 000000000000..3f37f314b580 > --- /dev/null > +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml > @@ -0,0 +1,454 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/sound/renesas,rsnd.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas R-Car Sound Driver Device Tree Bindings > + > +maintainers: > + - Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > + > +properties: > + > + compatible: > + oneOf: > + # for Gen1 SoC > + - items: > + - enum: > + - renesas,rcar_sound-r8a7778 # R-Car M1A > + - renesas,rcar_sound-r8a7779 # R-Car H1 > + - enum: > + - renesas,rcar_sound-gen1 > + # for Gen2 SoC > + - items: > + - enum: > + - renesas,rcar_sound-r8a7742 # RZ/G1H > + - renesas,rcar_sound-r8a7743 # RZ/G1M > + - renesas,rcar_sound-r8a7744 # RZ/G1N > + - renesas,rcar_sound-r8a7745 # RZ/G1E > + - renesas,rcar_sound-r8a77470 # RZ/G1C > + - renesas,rcar_sound-r8a7790 # R-Car H2 > + - renesas,rcar_sound-r8a7791 # R-Car M2-W > + - renesas,rcar_sound-r8a7793 # R-Car M2-N > + - renesas,rcar_sound-r8a7794 # R-Car E2 > + - enum: > + - renesas,rcar_sound-gen2 > + # for Gen3 SoC > + - items: > + - enum: > + - renesas,rcar_sound-r8a774a1 # RZ/G2M > + - renesas,rcar_sound-r8a774b1 # RZ/G2N > + - renesas,rcar_sound-r8a774c0 # RZ/G2E > + - renesas,rcar_sound-r8a774e1 # RZ/G2H > + - renesas,rcar_sound-r8a7795 # R-Car H3 > + - renesas,rcar_sound-r8a7796 # R-Car M3-W > + - renesas,rcar_sound-r8a77965 # R-Car M3-N > + - renesas,rcar_sound-r8a77990 # R-Car E3 > + - renesas,rcar_sound-r8a77995 # R-Car D3 > + - enum: > + - renesas,rcar_sound-gen3 > + # for Generic > + - items: > + - enum: > + - renesas,rcar_sound-gen1 > + - renesas,rcar_sound-gen2 > + - renesas,rcar_sound-gen3 > + > + reg: > + minItems: 3 # Gen1 > + maxItems: 5 # Gen2/Gen3 > + > + reg-names: > + description: | > + Should contain the register names. > + scu/adg/ssi if generation1 > + scu/adg/ssiu/ssi/audmapp if generation2/generation3 The schema describes this, no need to duplicate here. > + minItems: 3 # Gen1 > + maxItems: 5 # Gen2/Gen3 > + oneOf: > + # for Gen1 > + - allOf: > + - items: Don't need 'allOf' if only 1 entry. > + enum: > + - scu > + - adg > + - ssi > + # for Gen2/Gen3 > + - allOf: > + - items: > + enum: > + - scu > + - adg > + - ssiu > + - ssi > + - audmapp We'd normally define these with an if/then schema based on the compatibles. > + > + "#sound-dai-cells": > + description: | > + it must be 0 if your system is using single DAI > + it must be 1 if your system is using multi DAIs > + enum: [0, 1] > + > + "#clock-cells": > + description: | > + it must be 0 if your system has audio_clkout > + it must be 1 if your system has audio_clkout0/1/2/3 > + enum: [0, 1] > + > + clock-frequency: > + description: for audio_clkout0/1/2/3 > + $ref: /schemas/types.yaml#/definitions/uint32-array > + > + clkout-lr-asynchronous: > + description: audio_clkoutn is asynchronizes with lr-clock. > + $ref: /schemas/types.yaml#/definitions/flag > + > + power-domains: true > + > + resets: > + description: References to SSI resets > + $ref: /schemas/types.yaml#/definitions/phandle-array 'resets' already has a type. Just need to know how many and what each one is. > + > + reset-names: > + description: List of valid reset names. > + allOf: > + - items: > + oneOf: > + - pattern: '^ssi(-all|\.[0-9])$' Just: reset-names: maxItems: ?? items: pattern: '^ssi(-all|\.[0-9])$' > + > + clocks: > + description: References to SSI/SRC/MIX/CTU/DVC/AUDIO_CLK clocks. > + minItems: 1 > + maxItems: 31 > + > + clock-names: > + description: List of necessary clock names. > + minItems: 1 > + maxItems: 31 > + allOf: > + - items: Don't need 'allOf'. > + oneOf: > + - const: ssi-all > + - pattern: '^ssi\.[0-9]$' > + - pattern: '^src\.[0-9]$' > + - pattern: '^mix\.[0-1]$' > + - pattern: '^ctu\.[0-1]$' > + - pattern: '^dvc\.[0-1]$' > + - pattern: '^clk_(a|b|c|i)$' > + > + # For OF-graph > + port: > + description: OF-Graph subnode > + $ref: "audio-graph-card.yaml#/definitions/port" Something like this should work (at the top-level): allOf: - $ref: "audio-graph-card.yaml# > + > + # For multi OF-graph > + ports: > + description: multi OF-Graph subnode > + $ref: "audio-graph-card.yaml#/definitions/ports" > + > +# use patternProperties to avoid naming "xxx,yyy" > +patternProperties: > + "^rcar_sound,dvc$": None of these are patterns. > + description: DVC subnode. > + type: object > + patternProperties: > + "^dvc-[0-1]$": > + type: object > + properties: > + dmas: > + maxItems: 1 > + dma-names: > + const: "tx" > + required: > + - dmas > + - dma-names > + additionalProperties: false > + > + "^rcar_sound,mix$": > + description: MIX subnode. > + type: object > + patternProperties: > + "^mix-[0-1]$": > + type: object > + # no propaties > + additionalProperties: false > + > + "^rcar_sound,ctu$": > + description: CTU subnode. > + type: object > + patternProperties: > + "^ctu-[0-7]$": > + type: object > + # no propaties > + additionalProperties: false > + > + "^rcar_sound,src$": > + description: SRC subnode. > + type: object > + patternProperties: > + "^src-[0-9]$": > + type: object > + properties: > + interrupts: > + maxItems: 1 > + dmas: > + maxItems: 2 > + dma-names: > + allOf: > + - items: > + enum: > + - tx > + - rx > + allOf: > + - if: > + properties: > + status: > + const: "ok" > + then: > + required: > + - interrupts > + - dmas > + - dma-names > + additionalProperties: false > + > + "^rcar_sound,ssiu$": > + description: SSIU subnode. > + type: object > + patternProperties: > + "^ssiu-[0-9]+$": > + type: object > + properties: > + dmas: > + maxItems: 2 > + dma-names: > + allOf: > + - items: > + enum: > + - tx > + - rx > + required: > + - dmas > + - dma-names > + additionalProperties: false > + > + "^rcar_sound,ssi$": > + description: SSI subnode. > + type: object > + patternProperties: > + "^ssi-[0-9]$": > + type: object > + properties: > + interrupts: > + maxItems: 1 > + dmas: > + minItems: 2 > + maxItems: 4 > + dma-names: > + allOf: > + - items: > + enum: > + - tx > + - rx > + - txu # if no ssiu node > + - rxu # if no ssiu node > + > + shared-pin: > + description: shared clock pin > + $ref: /schemas/types.yaml#/definitions/flag > + pio-transfer: > + description: PIO transfer mode > + $ref: /schemas/types.yaml#/definitions/flag > + no-busif: > + description: BUSIF is not used when [mem -> SSI] via DMA case > + $ref: /schemas/types.yaml#/definitions/flag > + required: > + - interrupts > + - dmas > + - dma-names > + additionalProperties: false > + > + # For DAI base > + "^rcar_sound,dai$": > + description: DAI subnode. > + type: object > + patternProperties: > + "^dai([0-9]+)?$": > + type: object > + properties: > + playback: > + $ref: /schemas/types.yaml#/definitions/phandle-array > + capture: > + $ref: /schemas/types.yaml#/definitions/phandle-array > + anyOf: > + - required: > + - playback > + - required: > + - capture > + additionalProperties: false > + > +required: > + - compatible > + - reg > + - reg-names > + - clocks > + - clock-names > + - "#sound-dai-cells" > + > +additionalProperties: false > + > +examples: > + - | > + rcar_sound: sound@ec500000 { > + #sound-dai-cells = <1>; > + compatible = "renesas,rcar_sound-r8a7790", "renesas,rcar_sound-gen2"; > + reg = <0xec500000 0x1000>, /* SCU */ > + <0xec5a0000 0x100>, /* ADG */ > + <0xec540000 0x1000>, /* SSIU */ > + <0xec541000 0x1280>, /* SSI */ > + <0xec740000 0x200>; /* Audio DMAC peri peri*/ > + reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; > + > + clocks = <&mstp10_clks 1005>, /* SSI-ALL */ > + <&mstp10_clks 1006>, <&mstp10_clks 1007>, /* SSI9, SSI8 */ > + <&mstp10_clks 1008>, <&mstp10_clks 1009>, /* SSI7, SSI6 */ > + <&mstp10_clks 1010>, <&mstp10_clks 1011>, /* SSI5, SSI4 */ > + <&mstp10_clks 1012>, <&mstp10_clks 1013>, /* SSI3, SSI2 */ > + <&mstp10_clks 1014>, <&mstp10_clks 1015>, /* SSI1, SSI0 */ > + <&mstp10_clks 1022>, <&mstp10_clks 1023>, /* SRC9, SRC8 */ > + <&mstp10_clks 1024>, <&mstp10_clks 1025>, /* SRC7, SRC6 */ > + <&mstp10_clks 1026>, <&mstp10_clks 1027>, /* SRC5, SRC4 */ > + <&mstp10_clks 1028>, <&mstp10_clks 1029>, /* SRC3, SRC2 */ > + <&mstp10_clks 1030>, <&mstp10_clks 1031>, /* SRC1, SRC0 */ > + <&mstp10_clks 1020>, <&mstp10_clks 1021>, /* MIX1, MIX0 */ > + <&mstp10_clks 1020>, <&mstp10_clks 1021>, /* CTU1, CTU0 */ > + <&mstp10_clks 1019>, <&mstp10_clks 1018>, /* DVC0, DVC1 */ > + <&audio_clk_a>, <&audio_clk_b>, /* CLKA, CLKB */ > + <&audio_clk_c>, <&audio_clk_i>; /* CLKC, CLKI */ > + > + clock-names = "ssi-all", > + "ssi.9", "ssi.8", > + "ssi.7", "ssi.6", > + "ssi.5", "ssi.4", > + "ssi.3", "ssi.2", > + "ssi.1", "ssi.0", > + "src.9", "src.8", > + "src.7", "src.6", > + "src.5", "src.4", > + "src.3", "src.2", > + "src.1", "src.0", > + "mix.1", "mix.0", > + "ctu.1", "ctu.0", > + "dvc.0", "dvc.1", > + "clk_a", "clk_b", > + "clk_c", "clk_i"; > + > + rcar_sound,dvc { > + dvc0: dvc-0 { > + dmas = <&audma0 0xbc>; > + dma-names = "tx"; > + }; > + dvc1: dvc-1 { > + dmas = <&audma0 0xbe>; > + dma-names = "tx"; > + }; > + }; > + > + rcar_sound,mix { > + mix0: mix-0 { }; > + mix1: mix-1 { }; > + }; > + > + rcar_sound,ctu { > + ctu00: ctu-0 { }; > + ctu01: ctu-1 { }; > + ctu02: ctu-2 { }; > + ctu03: ctu-3 { }; > + ctu10: ctu-4 { }; > + ctu11: ctu-5 { }; > + ctu12: ctu-6 { }; > + ctu13: ctu-7 { }; > + }; > + > + rcar_sound,src { > + src0: src-0 { > + status = "disabled"; > + }; > + src1: src-1 { > + interrupts = <0 353 0>; > + dmas = <&audma0 0x87>, <&audma1 0x9c>; > + dma-names = "rx", "tx"; > + }; > + /* skip after src-2 */ > + }; > + > + rcar_sound,ssiu { > + ssiu00: ssiu-0 { > + dmas = <&audma0 0x15>, <&audma1 0x16>; > + dma-names = "rx", "tx"; > + }; > + ssiu01: ssiu-1 { > + dmas = <&audma0 0x35>, <&audma1 0x36>; > + dma-names = "rx", "tx"; > + }; > + /* skip after ssiu-2 */ > + }; > + > + rcar_sound,ssi { > + ssi0: ssi-0 { > + interrupts = <0 370 1>; > + dmas = <&audma0 0x01>, <&audma1 0x02>; > + dma-names = "rx", "tx"; > + }; > + ssi1: ssi-1 { > + interrupts = <0 371 1>; > + dmas = <&audma0 0x03>, <&audma1 0x04>; > + dma-names = "rx", "tx"; > + }; > + /* skip other ssi-2 */ > + }; > + > + /* DAI base */ > + rcar_sound,dai { > + dai0 { > + playback = <&ssi5 &src5>; > + capture = <&ssi6>; > + }; > + dai1 { > + playback = <&ssi3>; > + }; > + dai2 { > + capture = <&ssi4>; > + }; > + dai3 { > + playback = <&ssi7>; > + }; > + dai4 { > + capture = <&ssi8>; > + }; > + }; > + > + /* assume audio-graph */ > + port { > + rsnd_endpoint: endpoint { > + remote-endpoint = <&codec_endpoint>; > + > + dai-format = "left_j"; > + bitclock-master = <&rsnd_endpoint0>; > + frame-master = <&rsnd_endpoint0>; > + > + playback = <&ssi0 &src0 &dvc0>; > + capture = <&ssi1 &src1 &dvc1>; > + }; > + }; > + }; > + > + > + /* assume audio-graph */ > + codec { > + port { > + codec_endpoint: endpoint { > + remote-endpoint = <&rsnd_endpoint>; > + }; > + }; > + }; > -- > 2.25.1 > ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <87eemrlx15.wl-kuninori.morimoto.gx@renesas.com>]
* Re: [PATCH v2 2/2] ASoC: dt-bindings: renesas,rsnd: switch to yaml base Documentation [not found] ` <87eemrlx15.wl-kuninori.morimoto.gx@renesas.com> @ 2020-10-19 22:03 ` Rob Herring 2020-10-19 23:07 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto 0 siblings, 1 reply; 9+ messages in thread From: Rob Herring @ 2020-10-19 22:03 UTC (permalink / raw) To: Kuninori Morimoto; +Cc: devicetree, Linux-ALSA, Mark Brown On Wed, Sep 23, 2020 at 9:40 PM Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> wrote: > > > Hi Rob > > > > From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > > > > This patch switches from .txt base to .yaml base Document. > > > It is still keeping detail explanations at .txt > > > > > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > > > --- > (snip) > > > + # For OF-graph > > > + port: > > > + description: OF-Graph subnode > > > + $ref: "audio-graph-card.yaml#/definitions/port" > > > > Something like this should work (at the top-level): > > > > allOf: > > - $ref: "audio-graph-card.yaml# > > Hmm.. it will be error, see below (A). > How to solve it ?? All the 'allOf's have to be grouped. > (I will keep to use above style) > > > > +# use patternProperties to avoid naming "xxx,yyy" > > > +patternProperties: > > > + "^rcar_sound,dvc$": > > > > None of these are patterns. > > As always happen, "xxx,yyy" style will be error for me without patternProperties. > What is wrong ?? Node names shouldn't really have vendor prefixes, but since you already depend on this we shouldn't change that. We'll need to relax the meta-schema, but for now it's fine as-is. Rob ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: switch to yaml base Documentation 2020-10-19 22:03 ` Rob Herring @ 2020-10-19 23:07 ` Kuninori Morimoto 0 siblings, 0 replies; 9+ messages in thread From: Kuninori Morimoto @ 2020-10-19 23:07 UTC (permalink / raw) To: Rob Herring; +Cc: devicetree, Linux-ALSA, Mark Brown Hi Rob > > As always happen, "xxx,yyy" style will be error for me without patternProperties. > > What is wrong ?? > > Node names shouldn't really have vendor prefixes, but since you > already depend on this we shouldn't change that. We'll need to relax > the meta-schema, but for now it's fine as-is. OK, thanks. The original code didn't have vendor prefixes, but I was ordered to have it from reviewer. I don't remember from whom though... Thank you for your help !! Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-10-19 23:08 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-08-27 0:24 [PATCH v2 0/2] ASoC: dt-bindings: audio-graph-card: switch to yaml base Documentation Kuninori Morimoto 2020-08-27 0:24 ` [PATCH v2 1/2] " Kuninori Morimoto 2020-09-08 23:14 ` Rob Herring 2020-09-10 1:17 ` Kuninori Morimoto 2020-09-10 18:57 ` Rob Herring 2020-08-27 0:25 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto 2020-09-08 23:22 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas,rsnd: " Rob Herring [not found] ` <87eemrlx15.wl-kuninori.morimoto.gx@renesas.com> 2020-10-19 22:03 ` Rob Herring 2020-10-19 23:07 ` [PATCH v2 2/2] ASoC: dt-bindings: renesas, rsnd: " Kuninori Morimoto
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).