| Message ID | 878r7rbg0v.wl-kuninori.morimoto.gx@renesas.com |
|---|---|
| State | New |
| Headers | show |
| Series | ASoC: makes CPU/Codec channel connection map more generic | expand |
On Wed, Oct 25, 2023 at 02:19:13AM +0000, Kuninori Morimoto wrote: > This patch adds channel-map-index property to enable handling Don't write commit messages with 'This patch' or 'This commit'. See the documentation on writing patches. > CPU:Codec = N:M connection. This is not answering Why? > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> > --- > .../devicetree/bindings/sound/audio-graph-port.yaml | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > index 60b5e3fd1115..2a7e0784d591 100644 > --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > @@ -93,6 +93,19 @@ definitions: > minimum: 1 > maximum: 64 > > + channel-map-index: > + description: It indicates CPU/Codec DAIs channel mapping index if number of > + CPU(N) / Codec(M) were not same in one dai-link. channel-map-index is not > + needed if the numbers were 1:M or N:1 or N==M. Same indexed CPU <-> Codec > + will be paired. This is CPUx2 <-> Codecx3 sample. > + CPUA { ... .channel-map-index = <0>; } > + [0] CPUA <---> CodecA CPUB { ... .channel-map-index = <1 2>; } > + [1] CPUB <-+-> CodecB > + [2] \-> CodecC CodecA { ... .channel-map-index = <0>; } > + CodecB { ... .channel-map-index = <1>; } > + CodecC { ... .channel-map-index = <2>; } We have 2 different meanings for channel-map-index here. We have the codecs defining "I am index N" and then the CPUs defining "I'm connected to codec N". That's confusing to start with. Made-up indices are something we try to avoid in DT. Are the numbers here (0, 1, 2) significant? The normal way we link from one node to another is phandles. Why not use phandles here: CPUA { ... .channel-map-index = <&CodecA>; } CPUB { ... .channel-map-index = <&CodecB &CodecC>; } However, we also have OF graph to define complex topologies/connections AND we're already using it for this binding. So why not here? You can always have more than 1 port and/or endpoint. Generally, multiple ports are independent/simultaneous data connections and multiple endpoints are either 1:N fanout or N:1 muxed connections. Rob
Hi Rob > We have 2 different meanings for channel-map-index here. We have the > codecs defining "I am index N" and then the CPUs defining "I'm connected > to codec N". That's confusing to start with. Made-up indices are > something we try to avoid in DT. Are the numbers here (0, 1, 2) > significant? The normal way we link from one node to another is > phandles. Why not use phandles here: > > CPUA { ... .channel-map-index = <&CodecA>; } > CPUB { ... .channel-map-index = <&CodecB &CodecC>; } > > > However, we also have OF graph to define complex topologies/connections > AND we're already using it for this binding. So why not here? You can > always have more than 1 port and/or endpoint. Generally, multiple ports > are independent/simultaneous data connections and multiple endpoints are > either 1:N fanout or N:1 muxed connections. Hmm... I will re-consider about it. Thank you for your help !! Best regards --- Kuninori Morimoto
diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml index 60b5e3fd1115..2a7e0784d591 100644 --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml @@ -93,6 +93,19 @@ definitions: minimum: 1 maximum: 64 + channel-map-index: + description: It indicates CPU/Codec DAIs channel mapping index if number of + CPU(N) / Codec(M) were not same in one dai-link. channel-map-index is not + needed if the numbers were 1:M or N:1 or N==M. Same indexed CPU <-> Codec + will be paired. This is CPUx2 <-> Codecx3 sample. + CPUA { ... .channel-map-index = <0>; } + [0] CPUA <---> CodecA CPUB { ... .channel-map-index = <1 2>; } + [1] CPUB <-+-> CodecB + [2] \-> CodecC CodecA { ... .channel-map-index = <0>; } + CodecB { ... .channel-map-index = <1>; } + CodecC { ... .channel-map-index = <2>; } + $ref: /schemas/types.yaml#/definitions/uint32-array + ports: $ref: "#/definitions/port-base" unevaluatedProperties: false
This patch adds channel-map-index property to enable handling CPU:Codec = N:M connection. Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> --- .../devicetree/bindings/sound/audio-graph-port.yaml | 13 +++++++++++++ 1 file changed, 13 insertions(+)