2023-08-01 09:28:28

by Daniel Baluta (OSS)

[permalink] [raw]
Subject: [PATCH 2/2] ASoC: dt-bindings: simple-card: Document new DAI flags playback-only/capture-only

From: Daniel Baluta <[email protected]>

Document new playback-only and capture-only flags which can be used when
dai link can only support just one direction: playback or capture but
not both.

Signed-off-by: Daniel Baluta <[email protected]>
---
Documentation/devicetree/bindings/sound/simple-card.yaml | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/simple-card.yaml b/Documentation/devicetree/bindings/sound/simple-card.yaml
index 59ac2d1d1ccf..1bf331f095a4 100644
--- a/Documentation/devicetree/bindings/sound/simple-card.yaml
+++ b/Documentation/devicetree/bindings/sound/simple-card.yaml
@@ -27,6 +27,14 @@ definitions:
description: dai-link uses bit clock inversion
$ref: /schemas/types.yaml#/definitions/flag

+ playback-only:
+ description: dai-link is used only for playback
+ $ref: /schemas/types.yaml#/definitions/flag
+
+ capture-only:
+ description: dai-link is used only for capture
+ $ref: /schemas/types.yaml#/definitions/flag
+
dai-tdm-slot-num:
description: see tdm-slot.txt.
$ref: /schemas/types.yaml#/definitions/uint32
--
2.25.1



2023-08-01 19:50:27

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 2/2] ASoC: dt-bindings: simple-card: Document new DAI flags playback-only/capture-only

On Tue, Aug 01, 2023 at 11:24:33AM +0300, Daniel Baluta wrote:
> From: Daniel Baluta <[email protected]>
>
> Document new playback-only and capture-only flags which can be used when
> dai link can only support just one direction: playback or capture but
> not both.
>
> Signed-off-by: Daniel Baluta <[email protected]>
> ---
> Documentation/devicetree/bindings/sound/simple-card.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)

Please add new features to audio-graph-card2 - we're trying to deprecate
simple-card, audio-graph-card2 is a superset with more flexibility.
It's not the end of the world to also support things in simple-card but
it definitely shouldn't have any capabilities that the newer card lacks.


Attachments:
(No filename) (746.00 B)
signature.asc (499.00 B)
Download all attachments

2023-08-11 20:55:43

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH 2/2] ASoC: dt-bindings: simple-card: Document new DAI flags playback-only/capture-only

On Tue, Aug 01, 2023 at 11:24:33AM +0300, Daniel Baluta wrote:
> From: Daniel Baluta <[email protected]>
>
> Document new playback-only and capture-only flags which can be used when
> dai link can only support just one direction: playback or capture but
> not both.
>
> Signed-off-by: Daniel Baluta <[email protected]>
> ---
> Documentation/devicetree/bindings/sound/simple-card.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/sound/simple-card.yaml b/Documentation/devicetree/bindings/sound/simple-card.yaml
> index 59ac2d1d1ccf..1bf331f095a4 100644
> --- a/Documentation/devicetree/bindings/sound/simple-card.yaml
> +++ b/Documentation/devicetree/bindings/sound/simple-card.yaml
> @@ -27,6 +27,14 @@ definitions:
> description: dai-link uses bit clock inversion
> $ref: /schemas/types.yaml#/definitions/flag
>
> + playback-only:
> + description: dai-link is used only for playback
> + $ref: /schemas/types.yaml#/definitions/flag
> +
> + capture-only:
> + description: dai-link is used only for capture
> + $ref: /schemas/types.yaml#/definitions/flag

Wouldn't this be implicit based on limitations in the either the cpu or
codec DAI?

Rob

2023-08-13 18:54:48

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 2/2] ASoC: dt-bindings: simple-card: Document new DAI flags playback-only/capture-only

On Fri, Aug 11, 2023 at 01:12:36PM -0600, Rob Herring wrote:
> On Tue, Aug 01, 2023 at 11:24:33AM +0300, Daniel Baluta wrote:

> > + playback-only:
> > + description: dai-link is used only for playback
> > + $ref: /schemas/types.yaml#/definitions/flag

> > + capture-only:
> > + description: dai-link is used only for capture
> > + $ref: /schemas/types.yaml#/definitions/flag

> Wouldn't this be implicit based on limitations in the either the cpu or
> codec DAI?

You can see cases where people just don't connect some of the signals
for whatever reason so even if the two devices could do bidrectional
audio the board can't, and there are also cases like the at91sam9g20ek
where the DAI is connected for bidrectional audio but there's not
actually any audio inputs you can connect (even loopbacks) for one of
the directions so it's best to just mask things out from the user.


Attachments:
(No filename) (913.00 B)
signature.asc (499.00 B)
Download all attachments