Re: [PATCH -next] pinctrl: aspeed: Make aspeed_pinmux_ips static

From: Andrew Jeffery
Date: Thu Jul 11 2019 - 20:56:27 EST




On Thu, 11 Jul 2019, at 23:55, YueHaibing wrote:
> Fix sparse warning:
>
> drivers/pinctrl/aspeed/pinmux-aspeed.c:8:12: warning:
> symbol 'aspeed_pinmux_ips' was not declared. Should it be static?
>
> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>

Reviewed-by: Andrew Jeffery <andrew@xxxxxxxx>

> ---
> drivers/pinctrl/aspeed/pinmux-aspeed.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/aspeed/pinmux-aspeed.c
> b/drivers/pinctrl/aspeed/pinmux-aspeed.c
> index 5b0fe17..839c01b 100644
> --- a/drivers/pinctrl/aspeed/pinmux-aspeed.c
> +++ b/drivers/pinctrl/aspeed/pinmux-aspeed.c
> @@ -5,7 +5,7 @@
>
> #include "pinmux-aspeed.h"
>
> -const char *const aspeed_pinmux_ips[] = {
> +static const char *const aspeed_pinmux_ips[] = {
> [ASPEED_IP_SCU] = "SCU",
> [ASPEED_IP_GFX] = "GFX",
> [ASPEED_IP_LPC] = "LPC",
> --
> 2.7.4
>
>
>