pinctrl: renesas: rzn1: Use for_each_child_of_node_scoped()

geertu · geertu · commit f07a32069134 · 2024-06-07T14:12:10.000+02:00
Use the scoped variant of for_each_child_of_node() to simplify the code. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Link: https://lore.kernel.org/r/c0a28f466c42d5d59c7fadfa1fd05fd512d43b6f.1717060708.git.geert+renesas@glider.be
diff --git a/drivers/pinctrl/renesas/pinctrl-rzn1.c b/drivers/pinctrl/renesas/pinctrl-rzn1.c
@@ -737,13 +737,12 @@ static int rzn1_pinctrl_parse_groups(struct device_node *np,
 
 static int rzn1_pinctrl_count_function_groups(struct device_node *np)
 {
-	struct device_node *child;
 	int count = 0;
 
 	if (of_property_count_u32_elems(np, RZN1_PINS_PROP) > 0)
 		count++;
 
-	for_each_child_of_node(np, child) {
+	for_each_child_of_node_scoped(np, child) {
 		if (of_property_count_u32_elems(child, RZN1_PINS_PROP) > 0)
 			count++;
 	}

Original file line number	Diff line number	Diff line change
`@@ -737,13 +737,12 @@ static int rzn1_pinctrl_parse_groups(struct device_node *np,`
`737`	`737`
`738`	`738`	`static int rzn1_pinctrl_count_function_groups(struct device_node *np)`
`739`	`739`	`{`
`740`		`- struct device_node *child;`
`741`	`740`	`int count = 0;`
`742`	`741`
`743`	`742`	`if (of_property_count_u32_elems(np, RZN1_PINS_PROP) > 0)`
`744`	`743`	`count++;`
`745`	`744`
`746`		`- for_each_child_of_node(np, child) {`
	`745`	`+ for_each_child_of_node_scoped(np, child) {`
`747`	`746`	`if (of_property_count_u32_elems(child, RZN1_PINS_PROP) > 0)`
`748`	`747`	`count++;`
`749`	`748`	`}`