Re: [PATCH 3/9] ASoC: test-component: use new of_graph functions
From: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Date: 2024-08-08 06:39:03
Also in:
dri-devel, linux-devicetree, linux-fbdev, linux-media, linux-omap, linux-sound
On 06/08/2024 07:58, Kuninori Morimoto wrote:
Current test-component.c is using for_each_endpoint_of_node() for parsing, but it should use "port" base loop instead of "endpoint", because properties are "port" base, instead of "endpoint".
The subject of the patch makes this sound like it's just using new helper functions, but the description makes me think this is a bug fix. I think it would be good to mention what are the symptoms of the bug.
quoted hunk ↗ jump to hunk
Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> --- sound/soc/generic/test-component.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/sound/soc/generic/test-component.c b/sound/soc/generic/test-component.c index e9e5e235a8a6..6f75da9f86e6 100644 --- a/sound/soc/generic/test-component.c +++ b/sound/soc/generic/test-component.c@@ -600,7 +600,7 @@ static int test_driver_probe(struct platform_device *pdev) } i = 0; - for_each_endpoint_of_node(node, ep) { + for_each_of_graph_port(node, ep) {
"ep" is endpoint, so the variable name should also be changed (even if it's not used).
snprintf(dname[i].name, TEST_NAME_LEN, "%s.%d", node->name, i); ddriv[i].name = dname[i].name;
Tomi