On Sun, Jun 18, 2017 at 09:58:53AM +0300, Mika Westerberg wrote:
> On Tue, Jun 13, 2017 at 02:49:08PM +0300, Sakari Ailus wrote:
> > +#define DECLARE_ACPI_FWNODE_OPS(ops) \
> > + const struct fwnode_operations ops = { \
> > + .device_is_available = acpi_fwnode_device_is_available, \
> > + .property_present = acpi_fwnode_property_present, \
> > + .property_read_int_array = \
> > + acpi_fwnode_property_read_int_array, \
> > + .property_read_string_array = \
> > + acpi_fwnode_property_read_string_array, \
> > + .get_parent = acpi_node_get_parent, \
> > + .get_next_child_node = acpi_get_next_subnode, \
> > + .get_named_child_node = acpi_fwnode_get_named_child_node, \
> > + .graph_get_next_endpoint = \
> > + acpi_fwnode_graph_get_next_endpoint, \
> > + .graph_get_remote_endpoint = \
> > + acpi_fwnode_graph_get_remote_endpoint, \
> > + .graph_get_port_parent = acpi_node_get_parent, \
> > + .graph_parse_endpoint = acpi_fwnode_graph_parse_endpoint, \
> > + }; \
> > + EXPORT_SYMBOL_GPL(ops)
>
> Can you align those backslashes, like
>
> const struct fwnode_operations ops = { \
> .device_is_available = acpi_fwnode_device_is_available, \
> .property_present = acpi_fwnode_property_present, \
> ...
>
> other than that the patch looks good.
Thanks for the review.
Fixed in v7.2.
--
Sakari Ailus
e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
