Re: [Outreachy kernel] [PATCH] staging: i4l: move open brace to previous line

[Elizabeth Ferdman <gnudevliz@gmail.com>]

On Fri, Oct 07, 2016 at 07:23:12AM +0200, Julia Lawall wrote:
> 
> 
> On Thu, 6 Oct 2016, Elizabeth Ferdman wrote:
> 
> > Fix checkpatch error "open brace should be on previous line" since only
> > functions should have their opening brace on a new line.
> >
> > Signed-off-by: Elizabeth Ferdman <gnudevliz@gmail.com>
> > ---
> >  drivers/staging/i4l/act2000/module.c | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/drivers/staging/i4l/act2000/module.c b/drivers/staging/i4l/act2000/module.c
> > index 0573f51..4746a62 100644
> > --- a/drivers/staging/i4l/act2000/module.c
> > +++ b/drivers/staging/i4l/act2000/module.c
> > @@ -19,8 +19,7 @@
> >  #include <linux/slab.h>
> >  #include <linux/init.h>
> >
> > -static unsigned short act2000_isa_ports[] =
> > -{
> > +static unsigned short act2000_isa_ports[] = {
> >  	0x0200, 0x0240, 0x0280, 0x02c0, 0x0300, 0x0340, 0x0380,
> >  	0xcfe0, 0xcfa0, 0xcf60, 0xcf20, 0xcee0, 0xcea0, 0xce60,
> >  };
> 
> Not related to this patch, but could this array be const?
> 
hey julia,
i'm looking into that but I don't know how to definitively answer that
question... what's a good way to go about that?

So far I've looked at the file, searched for occurrences of that array
in the file, and I don't see the array being altered. However it is
passed to some functions as in if(somefunction(act2000_isa_ports))...
And then those functions pass it to other functions. The functions
aren't saying const to show that they won't be altering the array. 

So it's kind of hard to know if it ever gets changed. Do you think that
just adding the word "const" after static, and then successfully 
compiling would mean that it does work? 

thanks,
liz

> julia
> 
> > --
> > 2.1.4
> >
> > --
> > You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> > To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> > To post to this group, send email to outreachy-kernel@googlegroups.com.
> > To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20161006214116.GA10879%40localhost.
> > For more options, visit https://groups.google.com/d/optout.
> >