On Thu, Apr 15, 2021 at 08:57:04PM +0100, Matthew Wilcox wrote: > On Thu, Apr 15, 2021 at 10:49:55PM +0300, Sakari Ailus wrote: > > On Thu, Apr 15, 2021 at 06:14:09PM +0100, Matthew Wilcox wrote: > > > On Thu, Apr 15, 2021 at 02:08:19PM -0300, Aline Santana Cordeiro wrote: > > > > -const struct atomisp_format_bridge *get_atomisp_format_bridge_from_mbus( > > > > - u32 mbus_code); > > > > +const struct atomisp_format_bridge* > > > > +get_atomisp_format_bridge_from_mbus(u32 mbus_code); > > > > > > No, this does not match coding style. Probably best to break the > > > 80-column guideline in this instance. Best would be to have a function > > > > Having the return type on the previous line is perfectly fine. There should > > be a space before the asterisk though. > > No, it's not. Linus has ranted about that before. Found it. https://lore.kernel.org/lkml/1054519757.161606@xxxxxxxxxxxxxxxxxxxxxxx/
