On Mon, May 22, 2017 at 11:53:19AM +0300, Sakari Ailus wrote: > +static int acpi_fwnode_property_read_int_array( Yuck! > + struct fwnode_handle *fwnode, const char *propname, > + unsigned int elem_size, void *val, size_t nval) Can you please write it like: static int acpi_fwnode_property_read_int_array(struct fwnode_handle *fwnode, const char *propname, unsigned int elem_size, void *val, size_t nval) or even static int acpi_fwnode_property_read_int_array(struct fwnode_handle *fwnode, const char *propname, unsigned int elem_size, void *val, size_t nval) Ditto to all similar constructs in this whole series. -- 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