diff mbox series

[v3] ASoC: dt-bindings: renesas,rsnd: add post-init-providers property

Message ID 87wmjkifob.wl-kuninori.morimoto.gx@renesas.com
State Accepted
Commit a2187d0dadfc308551bbb1b8d6caee69e2ad4744
Headers show
Series [v3] ASoC: dt-bindings: renesas,rsnd: add post-init-providers property | expand

Commit Message

Kuninori Morimoto Sept. 9, 2024, 11:13 p.m. UTC
At least if rsnd is using DPCM connection on Audio-Graph-Card2,
fw_devlink might doesn't have enough information to break the cycle
(Same problem might occur with Multi-CPU/Codec or Codec2Codec).
In such case, rsnd driver will not be probed.
Add post-init-providers support to break the link cycle.

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
v2 -> v3
	- remove "|" from description
	- removed common post-init-providers description
	- 100 char to 80 char for 1 line

 Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Rob Herring Sept. 11, 2024, 2:50 p.m. UTC | #1
On Mon, 09 Sep 2024 23:13:47 +0000, Kuninori Morimoto wrote:
> At least if rsnd is using DPCM connection on Audio-Graph-Card2,
> fw_devlink might doesn't have enough information to break the cycle
> (Same problem might occur with Multi-CPU/Codec or Codec2Codec).
> In such case, rsnd driver will not be probed.
> Add post-init-providers support to break the link cycle.
> 
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> v2 -> v3
> 	- remove "|" from description
> 	- removed common post-init-providers description
> 	- 100 char to 80 char for 1 line
> 
>  Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 6 ++++++
>  1 file changed, 6 insertions(+)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Mark Brown Sept. 11, 2024, 5:27 p.m. UTC | #2
On Mon, 09 Sep 2024 23:13:47 +0000, Kuninori Morimoto wrote:
> At least if rsnd is using DPCM connection on Audio-Graph-Card2,
> fw_devlink might doesn't have enough information to break the cycle
> (Same problem might occur with Multi-CPU/Codec or Codec2Codec).
> In such case, rsnd driver will not be probed.
> Add post-init-providers support to break the link cycle.
> 
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: dt-bindings: renesas,rsnd: add post-init-providers property
      commit: a2187d0dadfc308551bbb1b8d6caee69e2ad4744

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
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
index 07ec6247d9def..3bc93c59535e9 100644
--- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
+++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
@@ -112,6 +112,12 @@  properties:
     description: List of necessary clock names.
     # details are defined below
 
+  post-init-providers:
+    description: At least if rsnd is using DPCM connection on Audio-Graph-Card2,
+      fw_devlink might doesn't have enough information to break the cycle. rsnd
+      driver will not be probed in such case. Same problem might occur with
+      Multi-CPU/Codec or Codec2Codec.
+
   # ports is below
   port:
     $ref: audio-graph-port.yaml#/definitions/port-base