Re: [PATCH] pinctrl: vt8500: wmt: constify gpio_chip structure

2017-08-01 Thread Linus Walleij
On Tue, Jul 11, 2017 at 8:28 PM, Gustavo A. R. Silva wrote: > This structure is only used to copy into other structure, so declare > it as const. > > This issue was detected using Coccinelle and the following semantic patch: > > @r disable optional_qualifier@ >

Re: [PATCH] pinctrl: vt8500: wmt: constify gpio_chip structure

2017-08-01 Thread Linus Walleij
On Tue, Jul 11, 2017 at 8:28 PM, Gustavo A. R. Silva wrote: > This structure is only used to copy into other structure, so declare > it as const. > > This issue was detected using Coccinelle and the following semantic patch: > > @r disable optional_qualifier@ > identifier i; > position p; > @@ >

[PATCH] pinctrl: vt8500: wmt: constify gpio_chip structure

2017-07-11 Thread Gustavo A. R. Silva
This structure is only used to copy into other structure, so declare it as const. This issue was detected using Coccinelle and the following semantic patch: @r disable optional_qualifier@ identifier i; position p; @@ static struct gpio_chip i@p = { ... }; @ok@ identifier r.i; expression e;

[PATCH] pinctrl: vt8500: wmt: constify gpio_chip structure

2017-07-11 Thread Gustavo A. R. Silva
This structure is only used to copy into other structure, so declare it as const. This issue was detected using Coccinelle and the following semantic patch: @r disable optional_qualifier@ identifier i; position p; @@ static struct gpio_chip i@p = { ... }; @ok@ identifier r.i; expression e;