devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: Mark Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Linux-ALSA <alsa-devel@alsa-project.org>,
	devicetree@vger.kernel.org,
	Geert Uytterhoeven <geert+renesas@glider.be>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Sameer Pujar <spujar@nvidia.com>,
	Mohan Kumar <mkumard@nvidia.com>
Subject: [PATCH 09/11] ASoC: dt-bindings: renesas,rsnd: add missing playback/capture
Date: Wed, 14 Dec 2022 01:24:21 +0000	[thread overview]
Message-ID: <87iliepykb.wl-kuninori.morimoto.gx@renesas.com> (raw)
In-Reply-To: <87v8mepyoy.wl-kuninori.morimoto.gx@renesas.com>

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

renesas,rsnd.yaml is possible to use ports/port/endpoint if it is using
Audio Graph Card/Card2 for sound. The schema is defined under
audio-graph-port.yaml.

rsnd driver needs "playback/capture" property under endpoint, but it is not
defined in audio-graph-port.yaml. This patch adds missing "playback/capture"
properties under endpoint.

Without this patch, we will get below warning

${LINUX}/arch/arm64/boot/dts/renesas/r8a77950-salvator-x.dtb: sound@ec500000: ports:port@0:endpoint: Unevaluated properties are not allowed ('playback', 'capture' were unexpected)
	From schema: ${LINUX}/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 .../bindings/sound/renesas,rsnd.yaml          | 27 ++++++++++++++++---
 1 file changed, 23 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
index 679a246dd666..637130b7c307 100644
--- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
+++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
@@ -113,15 +113,34 @@ properties:
         - pattern: '^clk_(a|b|c|i)$'
 
   ports:
-    $ref: /schemas/graph.yaml#/properties/ports
+    $ref: audio-graph-port.yaml#/definitions/port-base
+    unevaluatedProperties: false
     patternProperties:
-      port(@[0-9a-f]+)?:
-        $ref: audio-graph-port.yaml#
+      "^port(@[0-9a-f]+)?$":
+        $ref: audio-graph-port.yaml#/definitions/port-base
         unevaluatedProperties: false
+        patternProperties:
+          "^endpoint(@[0-9a-f]+)?":
+            $ref: audio-graph-port.yaml#/definitions/endpoint-base
+            properties:
+              playback:
+                $ref: /schemas/types.yaml#/definitions/phandle-array
+              capture:
+                $ref: /schemas/types.yaml#/definitions/phandle-array
+            unevaluatedProperties: false
 
   port:
-    $ref: audio-graph-port.yaml#
+    $ref: audio-graph-port.yaml#/definitions/port-base
     unevaluatedProperties: false
+    patternProperties:
+      "^endpoint(@[0-9a-f]+)?":
+        $ref: audio-graph-port.yaml#/definitions/endpoint-base
+        properties:
+          playback:
+            $ref: /schemas/types.yaml#/definitions/phandle-array
+          capture:
+            $ref: /schemas/types.yaml#/definitions/phandle-array
+        unevaluatedProperties: false
 
   rcar_sound,dvc:
     description: DVC subnode.
-- 
2.25.1


  parent reply	other threads:[~2022-12-14  1:24 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-14  1:21 [PATCH 00/11] ASoC: dt-bindings: audio-graph-port related update Kuninori Morimoto
2022-12-14  1:22 ` [PATCH 01/11] ASoC: dt-bindings: audio-graph-port: use definitions for port/endpoint Kuninori Morimoto
2022-12-14  1:22 ` [PATCH 02/11] ASoC: dt-bindings: audio-graph-port: add definitions/ports Kuninori Morimoto
2022-12-16 16:33   ` Rob Herring
2022-12-18 23:58     ` Kuninori Morimoto
2022-12-14  1:22 ` [PATCH 03/11] ASoC: dt-bindings: audio-graph-port: add missing mclk-fs Kuninori Morimoto
2022-12-14  1:22 ` [PATCH 04/11] ASoC: dt-bindings: audio-graph-port: add clocks on endpoint Kuninori Morimoto
2022-12-16 16:20   ` Rob Herring
2022-12-18 23:47     ` Kuninori Morimoto
2022-12-14  1:23 ` [PATCH 05/11] ASoC: dt-bindings: audio-graph-port: remove prefix Kuninori Morimoto
2022-12-14  1:23 ` [PATCH 06/11] ASoC: dt-bindings: ti,pcm3168a: Convert to json-schema Kuninori Morimoto
2022-12-15 11:03   ` Geert Uytterhoeven
2022-12-15 11:47     ` Mark Brown
2022-12-15 12:10       ` Geert Uytterhoeven
2022-12-15 12:39         ` Mark Brown
2022-12-15 23:48           ` Kuninori Morimoto
2022-12-16  0:01             ` Kuninori Morimoto
2022-12-16 13:49             ` Mark Brown
2022-12-16 16:39         ` Rob Herring
2022-12-19  8:43           ` Geert Uytterhoeven
2022-12-19  8:45             ` Geert Uytterhoeven
2022-12-19 15:34               ` Péter Ujfalusi
2022-12-14  1:23 ` [PATCH 07/11] ASoC: dt-bindings: nvidia: use audio-graph-port.yaml ports Kuninori Morimoto
2022-12-14  1:23 ` [PATCH 08/11] ASoC: dt-bindings: ak4613: enable Of-graph (Audio-Graph-Card) style Kuninori Morimoto
2022-12-14  1:24 ` Kuninori Morimoto [this message]
2022-12-14  1:24 ` [PATCH 10/11] ASoC: dt-bindings: renesas,rsnd: #sound-dai-cells is not mandatory Kuninori Morimoto
2022-12-14  1:24 ` [PATCH 11/11] ASoC: dt-bindings: simple-card: add missing #address-cells/#size-cells Kuninori Morimoto

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=87iliepykb.wl-kuninori.morimoto.gx@renesas.com \
    --to=kuninori.morimoto.gx@renesas.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=geert+renesas@glider.be \
    --cc=jonathanh@nvidia.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=mkumard@nvidia.com \
    --cc=robh@kernel.org \
    --cc=spujar@nvidia.com \
    --cc=thierry.reding@gmail.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 NNTP newsgroup(s).