Messages in this thread | | | Date | Mon, 2 Jul 2018 10:59:33 +0200 | From | Johan Hovold <> | Subject | 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 <colin.king@canonical.com> > > 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 <colin.king@canonical.com>
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 <johan@kernel.org>
> --- > 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
| |