>> > > -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? > > > clang-format should do the right thing. It formats the line as-is in the patch: with 'unsigned int index' on the next line.