diff mbox series

[v2] ASoC: dt-bindings: renesas,rsnd: remove post-init-providers property

Message ID 877c7zno1n.wl-kuninori.morimoto.gx@renesas.com (mailing list archive)
State Superseded
Headers show
Series [v2] ASoC: dt-bindings: renesas,rsnd: remove post-init-providers property | expand

Commit Message

Kuninori Morimoto Dec. 17, 2024, 12:36 a.m. UTC
commit f8da001ae7a ("ASoC: audio-graph-card2: Purge absent supplies for
device tree nodes") marks multi/dpcm/c2c nodes as no suppliers. We no
longer need to use post-init-providers property to break fw_devlink cycle
(And no one is using it yet). Let's remove it.

Fixes: a2187d0dadf ("ASoC: dt-bindings: renesas,rsnd: add post-init-providers property")
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
v1 -> v2
	- add missing DT maintainers
	- Tidyup the git-log

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

Comments

Krzysztof Kozlowski Dec. 17, 2024, 5:53 a.m. UTC | #1
On 17/12/2024 01:36, Kuninori Morimoto wrote:
> commit f8da001ae7a ("ASoC: audio-graph-card2: Purge absent supplies for
> device tree nodes") marks multi/dpcm/c2c nodes as no suppliers. We no
> longer need to use post-init-providers property to break fw_devlink cycle
> (And no one is using it yet). Let's remove it.
> 
> Fixes: a2187d0dadf ("ASoC: dt-bindings: renesas,rsnd: add post-init-providers property")
> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> v1 -> v2
> 	- add missing DT maintainers
Only some of them? And some old addresses? You CC-ed an address, which
suggests you do not work on mainline kernel or you do not use
get_maintainers.pl/b4/patman. Please rebase and always work on mainline
or start using mentioned tools, so correct addresses will be used.


Best regards,
Krzysztof
Conor Dooley Dec. 17, 2024, 6:30 p.m. UTC | #2
On Tue, Dec 17, 2024 at 12:36:21AM +0000, Kuninori Morimoto wrote:
> commit f8da001ae7a ("ASoC: audio-graph-card2: Purge absent supplies for
> device tree nodes") marks multi/dpcm/c2c nodes as no suppliers. We no
> longer need to use post-init-providers property to break fw_devlink cycle
> (And no one is using it yet). Let's remove it.
> 
> Fixes: a2187d0dadf ("ASoC: dt-bindings: renesas,rsnd: add post-init-providers property")

I'm not sure that this fixes tag is appropriate. I don't think this is a
fix for the cited commit, it is just now not required. If this were to
get backported, wouldn't stable be broken since the supply purging
commit doesn't appear to have a Fixes: tag or a CC: stable?

Thanks,
Conor.

> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> ---
> v1 -> v2
> 	- add missing DT maintainers
> 	- Tidyup the git-log
> 
>  Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> index 6d0d1514cd421..e8a2acb926460 100644
> --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> @@ -112,12 +112,6 @@ 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
> -- 
> 2.43.0
> 
> 
>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
index 6d0d1514cd421..e8a2acb926460 100644
--- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
+++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
@@ -112,12 +112,6 @@  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