On 29.05.19 16:38, YueHaibing wrote: > Fix sparse warnings: > > drivers/platform/x86/pcengines-apuv2.c:80:27: warning: symbol 'gpios_led_table' was not declared. Should it be static? > drivers/platform/x86/pcengines-apuv2.c:113:27: warning: symbol 'gpios_key_table' was not declared. Should it be static? > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx> > --- > drivers/platform/x86/pcengines-apuv2.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c > index c1ca931e1fab..b0d3110ae378 100644 > --- a/drivers/platform/x86/pcengines-apuv2.c > +++ b/drivers/platform/x86/pcengines-apuv2.c > @@ -77,7 +77,7 @@ static const struct gpio_led_platform_data apu2_leds_pdata = { > .leds = apu2_leds, > }; > > -struct gpiod_lookup_table gpios_led_table = { > +static struct gpiod_lookup_table gpios_led_table = { > .dev_id = "leds-gpio", > .table = { > GPIO_LOOKUP_IDX(AMD_FCH_GPIO_DRIVER_NAME, APU2_GPIO_LINE_LED1, > @@ -110,7 +110,7 @@ static const struct gpio_keys_platform_data apu2_keys_pdata = { > .name = "apu2-keys", > }; > > -struct gpiod_lookup_table gpios_key_table = { > +static struct gpiod_lookup_table gpios_key_table = { > .dev_id = "gpio-keys-polled", > .table = { > GPIO_LOOKUP_IDX(AMD_FCH_GPIO_DRIVER_NAME, APU2_GPIO_LINE_MODESW, > thanks. Acked-By: Enrico Weigelt, metux IT consult <info@xxxxxxxxx> --mtx -- Enrico Weigelt, metux IT consult Free software and Linux embedded engineering info@xxxxxxxxx -- +49-151-27565287