On 07/11/2017 05:15 PM, Gustavo A. R. Silva wrote:
This structure is only used to copy into another structure, so declare it as const.
<snip>
Signed-off-by: Gustavo A. R. Silva <garsi...@embeddedor.com> --- drivers/gpio/gpio-altera-a10sr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/gpio-altera-a10sr.c b/drivers/gpio/gpio-altera-a10sr.c index 16a8951..6b11f13 100644 --- a/drivers/gpio/gpio-altera-a10sr.c +++ b/drivers/gpio/gpio-altera-a10sr.c @@ -71,7 +71,7 @@ static int altr_a10sr_gpio_direction_output(struct gpio_chip *gc, return -EINVAL; }-static struct gpio_chip altr_a10sr_gc = {+static const struct gpio_chip altr_a10sr_gc = { .label = "altr_a10sr_gpio", .owner = THIS_MODULE, .get = altr_a10sr_gpio_get,
Reviewed-by: Thor Thayer <thor.tha...@linux.intel.com>
