Hi Rob again > > + do { > > + port = of_get_next_child(parent, port); > > + if (!port) > > + return NULL; > > + } while (of_node_cmp(port->name, "port")); > > Use of_node_name_eq() I tried this, but unfortunately it seems of_node_name_eq() is not good match to this purpose. Because it try to check with "port@n". # I noticed that I can cleanup this patch to more simpler. # I want to send v2 patch if this patch is not rejected # Please let me know Best regards --- Kuninori Morimoto