On Fri, Apr 07, 2017 at 01:36:34PM +0300, Sakari Ailus wrote: ... > > > + if (is_of_node(fwn)) { > > > + if (of_node_cmp(to_of_node(fwn)->name, "ports") == 0) > > > + fwn = fwnode_get_next_parent(fwn); > > > + } else { > > > + /* The "ports" node is always there in ACPI. */ This comment is actually wrong and does not reflect the current implementation anymore. I'll fix that as well. > > > + fwn = fwnode_get_next_parent(fwn); > > > + } -- Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx