Re: [PATCH 1/3] staging: media: atomisp: Fix braces on incorrect lines

[Matt Wardle <matt@mattwardle.net>]

On Mon, Feb 09, 2026 at 05:42:06PM +0200, Andy Shevchenko wrote:
> On Mon, Feb 09, 2026 at 02:29:07PM +0000, Matt Wardle wrote:
> > Fix checkpatch.pl errors:
> > 
> > ERROR: open brace '{' following function definitions go on the next line
> > ERROR: that open brace { should be on the previous line
> 
> Nice! My comments below.
> 
> ...
> 
> > store_dvs_6axis_config(
> 
> >      const struct ia_css_dvs_6axis_config *dvs_6axis_config,
> >      const struct ia_css_binary *binary,
> >      const struct ia_css_frame_info *dvs_in_frame_info,
> > -    ia_css_ptr ddr_addr_y) {
> > +    ia_css_ptr ddr_addr_y)
> > +{
> 
> This makes no sense to touch before the parameters are indented correctly.
> I dunno if open parenthesis is fine at the end of the line, but I would do it
> differently, as
> 
> store_dvs_6axis_config(const struct ia_css_dvs_6axis_config *dvs_6axis_config,
> 		       const struct ia_css_binary *binary,
> 		       ...)
> {
> 
> ...
> 
> The bottom line is, you need to check manually these and make prerequisite
> patch to first fix the indentation.
> 
> ...
> 
> >  int
> >  ia_css_get_dvs2_statistics(
> >      struct ia_css_dvs2_statistics          *host_stats,
> > -    const struct ia_css_isp_dvs_statistics *isp_stats) {
> > +    const struct ia_css_isp_dvs_statistics *isp_stats)
> > +{
> 
> Ditto.
> 
> And when you go with indentation it makes sense to combine (it's fine in this
> case) those changes with the splitting { to the next line.

Thanks, I'll fix the indentation first on the functions then resubmit
without the last patch.

This is happening in many files so it might be best I tackle the other
occurences in a separate patch.