On Thu, Jan 18, 2018 at 1:31 PM, Marcin Wojtas <mw@xxxxxxxxxxxx> wrote: > Implement a new helper function fwnode_get_next_available_child_node(), > which enables obtaining next enabled child fwnode, which > works on a similar basis to OF's of_get_next_available_child(). > > This commit also introduces a macro, thanks to which it is > possible to iterate over the available fwnodes, using the > new function described above. > > Signed-off-by: Marcin Wojtas <mw@xxxxxxxxxxxx> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > --- > drivers/base/property.c | 26 ++++++++++++++++++++ > include/linux/property.h | 6 +++++ > 2 files changed, 32 insertions(+) > > diff --git a/drivers/base/property.c b/drivers/base/property.c > index 1d6c9d9..613ba82 100644 > --- a/drivers/base/property.c > +++ b/drivers/base/property.c > @@ -998,6 +998,32 @@ fwnode_get_next_child_node(const struct fwnode_handle *fwnode, > EXPORT_SYMBOL_GPL(fwnode_get_next_child_node); > > /** > + * fwnode_get_next_available_child_node - Return the next > + * available child node handle for a node > + * @fwnode: Firmware node to find the next child node for. > + * @child: Handle to one of the node's child nodes or a %NULL handle. > + */ > +struct fwnode_handle * > +fwnode_get_next_available_child_node(const struct fwnode_handle *fwnode, > + struct fwnode_handle *child) > +{ > + struct fwnode_handle *next_child = child; > + > + if (!fwnode) > + return NULL; > + > + do { > + next_child = fwnode_get_next_child_node(fwnode, next_child); > + > + if (!next_child || fwnode_device_is_available(next_child)) > + break; > + } while (next_child); > + > + return next_child; > +} > +EXPORT_SYMBOL_GPL(fwnode_get_next_available_child_node); > + > +/** > * device_get_next_child_node - Return the next child node handle for a device > * @dev: Device to find the next child node for. > * @child: Handle to one of the device's child nodes or a null handle. > diff --git a/include/linux/property.h b/include/linux/property.h > index e05889f..5b0563a 100644 > --- a/include/linux/property.h > +++ b/include/linux/property.h > @@ -83,11 +83,17 @@ struct fwnode_handle *fwnode_get_next_parent( > struct fwnode_handle *fwnode); > struct fwnode_handle *fwnode_get_next_child_node( > const struct fwnode_handle *fwnode, struct fwnode_handle *child); > +struct fwnode_handle *fwnode_get_next_available_child_node( > + const struct fwnode_handle *fwnode, struct fwnode_handle *child); > > #define fwnode_for_each_child_node(fwnode, child) \ > for (child = fwnode_get_next_child_node(fwnode, NULL); child; \ > child = fwnode_get_next_child_node(fwnode, child)) > > +#define fwnode_for_each_available_child_node(fwnode, child) \ > + for (child = fwnode_get_next_available_child_node(fwnode, NULL); child;\ > + child = fwnode_get_next_available_child_node(fwnode, child)) > + > struct fwnode_handle *device_get_next_child_node( > struct device *dev, struct fwnode_handle *child); > > -- -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html