On Thu, Dec 21, 2023 at 04:46:11PM -0700, Mark Hasemeyer wrote: > > > -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? Yes, the point is to leave index on the previous line and add a new one with the r. -- With Best Regards, Andy Shevchenko
