| Message ID | 87jzk3akoo.wl-kuninori.morimoto.gx@renesas.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | c6f597bc598a8c3456fa017eb0c55e7a41bfce2c |
| Headers | show |
| Series | ASoC: audio-graph-card2: remove unneeded of_node_get() | expand |
On Thu, 09 May 2024 00:33:11 +0000, Kuninori Morimoto wrote: > "lnk" is used as "port0", and it will be used to get "port1" by > of_get_next_child(ports, lnk). It will call of_node_put() inside. This > function is calling of_node_get() to make up for it, but it doesn't call > paired of_node_put(port0) when it quit function. > This of_node_get() itself is not needed, let's remove it. > > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/1] ASoC: audio-graph-card2: remove unneeded of_node_get() commit: c6f597bc598a8c3456fa017eb0c55e7a41bfce2c 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 --git a/sound/soc/generic/audio-graph-card2.c b/sound/soc/generic/audio-graph-card2.c index b1c675c6b6db..825125d8985b 100644 --- a/sound/soc/generic/audio-graph-card2.c +++ b/sound/soc/generic/audio-graph-card2.c @@ -1021,8 +1021,6 @@ static int graph_count_c2c(struct asoc_simple_priv *priv, struct device_node *codec0 = of_graph_get_remote_port(ep0); struct device_node *codec1 = of_graph_get_remote_port(ep1); - of_node_get(lnk); - /* * codec2codec { * ports {
"lnk" is used as "port0", and it will be used to get "port1" by of_get_next_child(ports, lnk). It will call of_node_put() inside. This function is calling of_node_get() to make up for it, but it doesn't call paired of_node_put(port0) when it quit function. This of_node_get() itself is not needed, let's remove it. Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> --- sound/soc/generic/audio-graph-card2.c | 2 -- 1 file changed, 2 deletions(-)