> > -int fwnode_irq_get(const struct fwnode_handle *fwnode, unsigned int index) > > +int fwnode_irq_get_resource(const struct fwnode_handle *fwnode, > > + unsigned int index, struct resource *r) > > It's perfectly fine to replace ) by , on the previous line, no need > to make it shorter. That puts the line at 115 chars? checkpatch.pl allows a maximum line length of 100. I can bump the 'index' argument up a line and keep it to a length of 95?