Re: [PATCH][next] gnss: make struct ubx_gserial_ops static
On Sat, Jun 30, 2018 at 12:22:46PM +0100, Colin King wrote: > From: Colin Ian King > > The structure ubx_gserial_ops is local to the source and does not need > to be in global scope, so make it static. > > Cleans up sparse warning: > symbol 'ubx_gserial_ops' was not declared. Should it be static? > > Signed-off-by: Colin Ian King This was reported by the kbuild test robot and an automated RFC patch submitted was a while back. I prefer this man-generated one with a proper commit message, so: Acked-by: Johan Hovold > --- > drivers/gnss/ubx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gnss/ubx.c b/drivers/gnss/ubx.c > index 902b6854b7db..12568aebb7f6 100644 > --- a/drivers/gnss/ubx.c > +++ b/drivers/gnss/ubx.c > @@ -59,7 +59,7 @@ static int ubx_set_power(struct gnss_serial *gserial, > return -EINVAL; > } > > -const struct gnss_serial_ops ubx_gserial_ops = { > +static const struct gnss_serial_ops ubx_gserial_ops = { > .set_power = ubx_set_power, > }; Thanks, Johan
Re: [PATCH][next] gnss: make struct ubx_gserial_ops static
On Sat, Jun 30, 2018 at 12:22:46PM +0100, Colin King wrote: > From: Colin Ian King > > The structure ubx_gserial_ops is local to the source and does not need > to be in global scope, so make it static. > > Cleans up sparse warning: > symbol 'ubx_gserial_ops' was not declared. Should it be static? > > Signed-off-by: Colin Ian King This was reported by the kbuild test robot and an automated RFC patch submitted was a while back. I prefer this man-generated one with a proper commit message, so: Acked-by: Johan Hovold > --- > drivers/gnss/ubx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gnss/ubx.c b/drivers/gnss/ubx.c > index 902b6854b7db..12568aebb7f6 100644 > --- a/drivers/gnss/ubx.c > +++ b/drivers/gnss/ubx.c > @@ -59,7 +59,7 @@ static int ubx_set_power(struct gnss_serial *gserial, > return -EINVAL; > } > > -const struct gnss_serial_ops ubx_gserial_ops = { > +static const struct gnss_serial_ops ubx_gserial_ops = { > .set_power = ubx_set_power, > }; Thanks, Johan
[PATCH][next] gnss: make struct ubx_gserial_ops static
From: Colin Ian King The structure ubx_gserial_ops is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: symbol 'ubx_gserial_ops' was not declared. Should it be static? Signed-off-by: Colin Ian King --- drivers/gnss/ubx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gnss/ubx.c b/drivers/gnss/ubx.c index 902b6854b7db..12568aebb7f6 100644 --- a/drivers/gnss/ubx.c +++ b/drivers/gnss/ubx.c @@ -59,7 +59,7 @@ static int ubx_set_power(struct gnss_serial *gserial, return -EINVAL; } -const struct gnss_serial_ops ubx_gserial_ops = { +static const struct gnss_serial_ops ubx_gserial_ops = { .set_power = ubx_set_power, }; -- 2.17.1
[PATCH][next] gnss: make struct ubx_gserial_ops static
From: Colin Ian King The structure ubx_gserial_ops is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: symbol 'ubx_gserial_ops' was not declared. Should it be static? Signed-off-by: Colin Ian King --- drivers/gnss/ubx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gnss/ubx.c b/drivers/gnss/ubx.c index 902b6854b7db..12568aebb7f6 100644 --- a/drivers/gnss/ubx.c +++ b/drivers/gnss/ubx.c @@ -59,7 +59,7 @@ static int ubx_set_power(struct gnss_serial *gserial, return -EINVAL; } -const struct gnss_serial_ops ubx_gserial_ops = { +static const struct gnss_serial_ops ubx_gserial_ops = { .set_power = ubx_set_power, }; -- 2.17.1