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. -- 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