Hi Dan > Someone should create for_each_endpoint_of_node_scoped(). > > #define for_each_endpoint_of_node_scoped(parent, child) \ > for (struct device_node *child __free(device_node) = \ > of_graph_get_next_endpoint(parent, NULL); child != NULL; \ > child = of_graph_get_next_endpoint(parent, child)) Thank you for pointing it. I have noticed that _scoped() loop exist, but this patch-set want to focus to use existing for_each_xxx() loop first. Replacing to _scoped() macro is next step. Thank you for your help !! Best regards --- Kuninori Morimoto