| Message ID | 20220927185359.294322-1-marex@denx.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2] ASoC: dt-bindings: Document audio OF graph dai-tdm-slot-num dai-tdm-slot-width props | expand |
On Tue, 27 Sep 2022 20:53:59 +0200, Marek Vasut wrote: > Document dai-tdm-slot-num and dai-tdm-slot-width props as those are > parsed by simple graph card and may therefore appear in audio OF graph > node. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: devicetree@vger.kernel.org > Cc: alsa-devel@alsa-project.org > To: linux-arm-kernel@lists.infradead.org > --- > V2: Drop the definition refs and just fill in type and description > --- > .../devicetree/bindings/sound/audio-graph-port.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
On Tue, 27 Sep 2022 20:53:59 +0200, Marek Vasut wrote: > Document dai-tdm-slot-num and dai-tdm-slot-width props as those are > parsed by simple graph card and may therefore appear in audio OF graph > node. > > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/1] ASoC: dt-bindings: Document audio OF graph dai-tdm-slot-num dai-tdm-slot-width props commit: 853110992cfefec433ca58cf7d69df4f639abe18 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
On 27/09/2022 14:53, Marek Vasut wrote: > Document dai-tdm-slot-num and dai-tdm-slot-width props as those are > parsed by simple graph card and may therefore appear in audio OF graph > node. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: devicetree@vger.kernel.org > Cc: alsa-devel@alsa-project.org > To: linux-arm-kernel@lists.infradead.org > --- > V2: Drop the definition refs and just fill in type and description > --- > .../devicetree/bindings/sound/audio-graph-port.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > index bc46a95ed8400..64654ceef2089 100644 > --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml > @@ -74,6 +74,12 @@ patternProperties: > convert-sample-format: > $ref: "/schemas/sound/dai-params.yaml#/$defs/dai-sample-format" > > + dai-tdm-slot-num: > + description: Number of slots in use. > + $ref: /schemas/types.yaml#/definitions/uint32 > + dai-tdm-slot-width: > + description: Width in bits for each slot. > + $ref: /schemas/types.yaml#/definitions/uint32 Isn't the slot width already part of dai-tdm-slot-width-map? Number of slots maybe as well can be deducted from number of tuples in dai-tdm-slot-width-map? > dai-tdm-slot-width-map: > description: Mapping of sample widths to slot widths. For hardware > that cannot support a fixed slot width or a slot width always Best regards, Krzysztof
On 10/14/22 01:34, Krzysztof Kozlowski wrote: > On 27/09/2022 14:53, Marek Vasut wrote: >> Document dai-tdm-slot-num and dai-tdm-slot-width props as those are >> parsed by simple graph card and may therefore appear in audio OF graph >> node. >> >> Signed-off-by: Marek Vasut <marex@denx.de> >> --- >> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> Cc: Liam Girdwood <lgirdwood@gmail.com> >> Cc: Mark Brown <broonie@kernel.org> >> Cc: Rob Herring <robh+dt@kernel.org> >> Cc: devicetree@vger.kernel.org >> Cc: alsa-devel@alsa-project.org >> To: linux-arm-kernel@lists.infradead.org >> --- >> V2: Drop the definition refs and just fill in type and description >> --- >> .../devicetree/bindings/sound/audio-graph-port.yaml | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml >> index bc46a95ed8400..64654ceef2089 100644 >> --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml >> +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml >> @@ -74,6 +74,12 @@ patternProperties: >> convert-sample-format: >> $ref: "/schemas/sound/dai-params.yaml#/$defs/dai-sample-format" >> >> + dai-tdm-slot-num: >> + description: Number of slots in use. >> + $ref: /schemas/types.yaml#/definitions/uint32 >> + dai-tdm-slot-width: >> + description: Width in bits for each slot. >> + $ref: /schemas/types.yaml#/definitions/uint32 > > Isn't the slot width already part of dai-tdm-slot-width-map? Number of > slots maybe as well can be deducted from number of tuples in > dai-tdm-slot-width-map? It seems to me per commit: 26e5366dd3056 ("ASoC: dt-bindings: audio-graph-port: Add dai-tdm-slot-width-map") that "dai-tdm-slot-width-map" was meant to address some sort of hardware special case, or rather limitation, where the hardware was unable to be configured in arbitrary manner, hence the list of available usable configurations. However, there seem to be no users of such a list in tree: next$ git grep -l dai-tdm-slot-width-map Documentation/devicetree/bindings/sound/audio-graph-port.yaml sound/soc/generic/simple-card-utils.c There are users of the dai-tdm-slot-width however: next$ git grep -l dai-tdm-slot-width arch arch/arm/boot/dts/stm32mp15xx-dkx.dtsi arch/arm64/boot/dts/allwinner/sun50i-a64-pinephone.dtsi arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi arch/arm64/boot/dts/freescale/imx8mn-evk.dtsi arch/arm64/boot/dts/freescale/imx8mq-evk.dts As far as I can tell, the dai-tdm-slot-width describes generic hardware without limitations, which can be configured in an arbitrary manner, so this is the preferred DT property (over the map one). Also, sorry for the late reply. >> dai-tdm-slot-width-map: >> description: Mapping of sample widths to slot widths. For hardware >> that cannot support a fixed slot width or a slot width always [...]
diff --git a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml index bc46a95ed8400..64654ceef2089 100644 --- a/Documentation/devicetree/bindings/sound/audio-graph-port.yaml +++ b/Documentation/devicetree/bindings/sound/audio-graph-port.yaml @@ -74,6 +74,12 @@ patternProperties: convert-sample-format: $ref: "/schemas/sound/dai-params.yaml#/$defs/dai-sample-format" + dai-tdm-slot-num: + description: Number of slots in use. + $ref: /schemas/types.yaml#/definitions/uint32 + dai-tdm-slot-width: + description: Width in bits for each slot. + $ref: /schemas/types.yaml#/definitions/uint32 dai-tdm-slot-width-map: description: Mapping of sample widths to slot widths. For hardware that cannot support a fixed slot width or a slot width always
Document dai-tdm-slot-num and dai-tdm-slot-width props as those are parsed by simple graph card and may therefore appear in audio OF graph node. Signed-off-by: Marek Vasut <marex@denx.de> --- Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Cc: Liam Girdwood <lgirdwood@gmail.com> Cc: Mark Brown <broonie@kernel.org> Cc: Rob Herring <robh+dt@kernel.org> Cc: devicetree@vger.kernel.org Cc: alsa-devel@alsa-project.org To: linux-arm-kernel@lists.infradead.org --- V2: Drop the definition refs and just fill in type and description --- .../devicetree/bindings/sound/audio-graph-port.yaml | 6 ++++++ 1 file changed, 6 insertions(+)