Re: [PATCH] iio: proximity: as3935: constify attribute_group structures
On 01/04/17 09:39, simran singhal wrote: > Check for attribute_group structures that are only stored in the > attrs filed of iio_info structure. As the attrs field of iio_info > structures is constant, so these attribute_group structures can also be > declared constant. > Done using coccinelle: > > @r1 disable optional_qualifier @ > identifier i; > position p; > @@ > static struct attribute_group i@p = {...}; > > @ok1@ > identifier r1.i; > position p; > struct iio_info x; > @@ > x.attrs=@p; > > @bad@ > position p!={r1.p,ok1.p}; > identifier r1.i; > @@ > i@p > > @depends on !bad disable optional_qualifier@ > identifier r1.i; > @@ > static > +const > struct attribute_group i={...}; > > @depends on !bad disable optional_qualifier@ > identifier r1.i; > @@ > +const > struct attribute_group i; > > File size before: >text data bss dec hex filename >4037 288 0432510e5 drivers/iio/proximity/as3935.o > > File size after: >text data bss dec hex filename >4101 256 043571105 drivers/iio/proximity/as3935.o > > Signed-off-by: simran singhalapplied to the togreg branch of iio.git and pushed out as testing like normal. Thanks, Jonathan > --- > drivers/iio/proximity/as3935.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/proximity/as3935.c b/drivers/iio/proximity/as3935.c > index 5656deb..1b8b472 100644 > --- a/drivers/iio/proximity/as3935.c > +++ b/drivers/iio/proximity/as3935.c > @@ -155,7 +155,7 @@ static struct attribute *as3935_attributes[] = { > NULL, > }; > > -static struct attribute_group as3935_attribute_group = { > +static const struct attribute_group as3935_attribute_group = { > .attrs = as3935_attributes, > }; > >
Re: [PATCH] iio: proximity: as3935: constify attribute_group structures
On 01/04/17 09:39, simran singhal wrote: > Check for attribute_group structures that are only stored in the > attrs filed of iio_info structure. As the attrs field of iio_info > structures is constant, so these attribute_group structures can also be > declared constant. > Done using coccinelle: > > @r1 disable optional_qualifier @ > identifier i; > position p; > @@ > static struct attribute_group i@p = {...}; > > @ok1@ > identifier r1.i; > position p; > struct iio_info x; > @@ > x.attrs=@p; > > @bad@ > position p!={r1.p,ok1.p}; > identifier r1.i; > @@ > i@p > > @depends on !bad disable optional_qualifier@ > identifier r1.i; > @@ > static > +const > struct attribute_group i={...}; > > @depends on !bad disable optional_qualifier@ > identifier r1.i; > @@ > +const > struct attribute_group i; > > File size before: >text data bss dec hex filename >4037 288 0432510e5 drivers/iio/proximity/as3935.o > > File size after: >text data bss dec hex filename >4101 256 043571105 drivers/iio/proximity/as3935.o > > Signed-off-by: simran singhal applied to the togreg branch of iio.git and pushed out as testing like normal. Thanks, Jonathan > --- > drivers/iio/proximity/as3935.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/proximity/as3935.c b/drivers/iio/proximity/as3935.c > index 5656deb..1b8b472 100644 > --- a/drivers/iio/proximity/as3935.c > +++ b/drivers/iio/proximity/as3935.c > @@ -155,7 +155,7 @@ static struct attribute *as3935_attributes[] = { > NULL, > }; > > -static struct attribute_group as3935_attribute_group = { > +static const struct attribute_group as3935_attribute_group = { > .attrs = as3935_attributes, > }; > >
[PATCH] iio: proximity: as3935: constify attribute_group structures
Check for attribute_group structures that are only stored in the attrs filed of iio_info structure. As the attrs field of iio_info structures is constant, so these attribute_group structures can also be declared constant. Done using coccinelle: @r1 disable optional_qualifier @ identifier i; position p; @@ static struct attribute_group i@p = {...}; @ok1@ identifier r1.i; position p; struct iio_info x; @@ x.attrs=@p; @bad@ position p!={r1.p,ok1.p}; identifier r1.i; @@ i@p @depends on !bad disable optional_qualifier@ identifier r1.i; @@ static +const struct attribute_group i={...}; @depends on !bad disable optional_qualifier@ identifier r1.i; @@ +const struct attribute_group i; File size before: textdata bss dec hex filename 4037 288 0432510e5 drivers/iio/proximity/as3935.o File size after: textdata bss dec hex filename 4101 256 043571105 drivers/iio/proximity/as3935.o Signed-off-by: simran singhal--- drivers/iio/proximity/as3935.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/proximity/as3935.c b/drivers/iio/proximity/as3935.c index 5656deb..1b8b472 100644 --- a/drivers/iio/proximity/as3935.c +++ b/drivers/iio/proximity/as3935.c @@ -155,7 +155,7 @@ static struct attribute *as3935_attributes[] = { NULL, }; -static struct attribute_group as3935_attribute_group = { +static const struct attribute_group as3935_attribute_group = { .attrs = as3935_attributes, }; -- 2.7.4
[PATCH] iio: proximity: as3935: constify attribute_group structures
Check for attribute_group structures that are only stored in the attrs filed of iio_info structure. As the attrs field of iio_info structures is constant, so these attribute_group structures can also be declared constant. Done using coccinelle: @r1 disable optional_qualifier @ identifier i; position p; @@ static struct attribute_group i@p = {...}; @ok1@ identifier r1.i; position p; struct iio_info x; @@ x.attrs=@p; @bad@ position p!={r1.p,ok1.p}; identifier r1.i; @@ i@p @depends on !bad disable optional_qualifier@ identifier r1.i; @@ static +const struct attribute_group i={...}; @depends on !bad disable optional_qualifier@ identifier r1.i; @@ +const struct attribute_group i; File size before: textdata bss dec hex filename 4037 288 0432510e5 drivers/iio/proximity/as3935.o File size after: textdata bss dec hex filename 4101 256 043571105 drivers/iio/proximity/as3935.o Signed-off-by: simran singhal --- drivers/iio/proximity/as3935.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/proximity/as3935.c b/drivers/iio/proximity/as3935.c index 5656deb..1b8b472 100644 --- a/drivers/iio/proximity/as3935.c +++ b/drivers/iio/proximity/as3935.c @@ -155,7 +155,7 @@ static struct attribute *as3935_attributes[] = { NULL, }; -static struct attribute_group as3935_attribute_group = { +static const struct attribute_group as3935_attribute_group = { .attrs = as3935_attributes, }; -- 2.7.4