[PATCH] pinctrl: nsp-gpio: fix non-static functions

Ray Jui ray.jui at broadcom.com
Tue Jun 7 10:00:57 PDT 2016


Hi Ben,

On 6/7/2016 9:58 AM, Ben Dooks wrote:
> Fixup warnings from functions that are not exported and
> therefore should be marked static. Fixes:
>
> drivers/pinctrl/bcm/pinctrl-nsp-gpio.c:461:5: warning: symbol 'nsp_pin_config_group_get' was not declared. Should it be static?
> drivers/pinctrl/bcm/pinctrl-nsp-gpio.c:467:5: warning: symbol 'nsp_pin_config_group_set' was not declared. Should it be static?
>
> Signed-off-by: Ben Dooks <ben.dooks at codethink.co.uk>
> ---
> Cc: Linus Walleij <linus.walleij at linaro.org>
> Cc: Ray Jui <rjui at broadcom.com>
> Cc: Scott Branden <sbranden at broadcom.com>
> Cc: Jon Mason <jonmason at broadcom.com>
> Cc: Yendapally Reddy Dhananjaya Reddy <yrdreddy at broadcom.com>
> Cc: linux-gpio at vger.kernel.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: bcm-kernel-feedback-list at broadcom.com
> ---
>  drivers/pinctrl/bcm/pinctrl-nsp-gpio.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/bcm/pinctrl-nsp-gpio.c b/drivers/pinctrl/bcm/pinctrl-nsp-gpio.c
> index a8b37a9..35783db 100644
> --- a/drivers/pinctrl/bcm/pinctrl-nsp-gpio.c
> +++ b/drivers/pinctrl/bcm/pinctrl-nsp-gpio.c
> @@ -458,13 +458,15 @@ static int nsp_gpio_get_strength(struct nsp_gpio *chip, unsigned gpio,
>  	return 0;
>  }
>
> -int nsp_pin_config_group_get(struct pinctrl_dev *pctldev, unsigned selector,
> +static int nsp_pin_config_group_get(struct pinctrl_dev *pctldev,
> +				    unsigned selector,
>  			     unsigned long *config)
>  {
>  	return 0;
>  }
>
> -int nsp_pin_config_group_set(struct pinctrl_dev *pctldev, unsigned selector,
> +static int nsp_pin_config_group_set(struct pinctrl_dev *pctldev,
> +				    unsigned selector,
>  			     unsigned long *configs, unsigned num_configs)
>  {
>  	return 0;
>

Looks good!

Thanks,

Acked-by: Ray Jui <ray.jui at broadcom.com>



More information about the linux-arm-kernel mailing list