| Message ID | 1465320302-5770-1-git-send-email-ben.dooks@codethink.co.uk |
|---|---|
| State | New |
| Headers | show |
On 07/06/16 18:25, Ben Dooks wrote: > There are five functions in the driver that are defined but > only used locally. Make these static to avoid the following > warnings: > > drivers/pinctrl/nomadik/pinctrl-nomadik.c:1036:6: warning: symbol 'nmk_gpio_clocks_enable' was not declared. Should it be static? > drivers/pinctrl/nomadik/pinctrl-nomadik.c:1050:6: warning: symbol 'nmk_gpio_clocks_disable' was not declared. Should it be static? > drivers/pinctrl/nomadik/pinctrl-nomadik.c:1073:6: warning: symbol 'nmk_gpio_wakeups_suspend' was not declared. Should it be static? > drivers/pinctrl/nomadik/pinctrl-nomadik.c:1094:6: warning: symbol 'nmk_gpio_wakeups_resume' was not declared. Should it be static? > drivers/pinctrl/nomadik/pinctrl-nomadik.c:1120:6: warning: symbol 'nmk_gpio_read_pull' was not declared. Should it be static? Apologies, wrong version of the patch without a typo fix in it.
On 07/06/16 18:30, Ben Dooks wrote: > On 07/06/16 18:25, Ben Dooks wrote: >> There are five functions in the driver that are defined but >> only used locally. Make these static to avoid the following >> warnings: >> >> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1036:6: warning: symbol 'nmk_gpio_clocks_enable' was not declared. Should it be static? >> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1050:6: warning: symbol 'nmk_gpio_clocks_disable' was not declared. Should it be static? >> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1073:6: warning: symbol 'nmk_gpio_wakeups_suspend' was not declared. Should it be static? >> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1094:6: warning: symbol 'nmk_gpio_wakeups_resume' was not declared. Should it be static? >> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1120:6: warning: symbol 'nmk_gpio_read_pull' was not declared. Should it be static? > > Apologies, wrong version of the patch without a typo fix in it. Given these functions are not being used, should I just consider a patch to remove these. If they are not being considered for future development it might be a good idea?
On Wed, Jun 8, 2016 at 11:15 AM, Ben Dooks <ben.dooks@codethink.co.uk> wrote: > On 07/06/16 18:30, Ben Dooks wrote: >> On 07/06/16 18:25, Ben Dooks wrote: >>> There are five functions in the driver that are defined but >>> only used locally. Make these static to avoid the following >>> warnings: >>> >>> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1036:6: warning: symbol 'nmk_gpio_clocks_enable' was not declared. Should it be static? >>> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1050:6: warning: symbol 'nmk_gpio_clocks_disable' was not declared. Should it be static? >>> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1073:6: warning: symbol 'nmk_gpio_wakeups_suspend' was not declared. Should it be static? >>> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1094:6: warning: symbol 'nmk_gpio_wakeups_resume' was not declared. Should it be static? >>> drivers/pinctrl/nomadik/pinctrl-nomadik.c:1120:6: warning: symbol 'nmk_gpio_read_pull' was not declared. Should it be static? >> >> Apologies, wrong version of the patch without a typo fix in it. > > Given these functions are not being used, should I just consider a > patch to remove these. If they are not being considered for future > development it might be a good idea? Yeah just kill them. If I need them back, I can always revert. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/pinctrl/nomadik/pinctrl-nomadik.c b/drivers/pinctrl/nomadik/pinctrl-nomadik.c index 38facef..a7cce0f 100644 --- a/drivers/pinctrl/nomadik/pinctrl-nomadik.c +++ b/drivers/pinctrl/nomadik/pinctrl-nomadik.c @@ -1033,7 +1033,7 @@ static inline void nmk_gpio_dbg_show_one(struct seq_file *s, #define nmk_gpio_dbg_show NULL #endif -void nmk_gpio_clocks_enable(void) +static void nmk_gpio_clocks_enable(void) { int i; @@ -1047,7 +1047,7 @@ void nmk_gpio_clocks_enable(void) } } -void nmk_gpio_clocks_disable(void) +static void nmk_gpio_clocks_disable(void) { int i; @@ -1070,7 +1070,7 @@ void nmk_gpio_clocks_disable(void) * PM ops are not used since this needs to be done at the end, after all the * other drivers are done with their suspend callbacks. */ -void nmk_gpio_wakeups_suspend(void) +static void nmk_gpio_wakeups_suspend(void) { int i; @@ -1091,7 +1091,7 @@ void nmk_gpio_wakeups_suspend(void) } } -void nmk_gpio_wakeups_resume(void) +static void nmk_gpio_wakeups_resume(void) { int i; @@ -1117,7 +1117,7 @@ void nmk_gpio_wakeups_resume(void) * Note: only pull up/down set via this driver can * be detected due to HW limitations. */ -void nmk_gpio_read_pull(int gpio_bank, u32 *pull_up) +statoc void nmk_gpio_read_pull(int gpio_bank, u32 *pull_up) { if (gpio_bank < NUM_BANKS) { struct nmk_gpio_chip *chip = nmk_gpio_chips[gpio_bank];
There are five functions in the driver that are defined but only used locally. Make these static to avoid the following warnings: drivers/pinctrl/nomadik/pinctrl-nomadik.c:1036:6: warning: symbol 'nmk_gpio_clocks_enable' was not declared. Should it be static? drivers/pinctrl/nomadik/pinctrl-nomadik.c:1050:6: warning: symbol 'nmk_gpio_clocks_disable' was not declared. Should it be static? drivers/pinctrl/nomadik/pinctrl-nomadik.c:1073:6: warning: symbol 'nmk_gpio_wakeups_suspend' was not declared. Should it be static? drivers/pinctrl/nomadik/pinctrl-nomadik.c:1094:6: warning: symbol 'nmk_gpio_wakeups_resume' was not declared. Should it be static? drivers/pinctrl/nomadik/pinctrl-nomadik.c:1120:6: warning: symbol 'nmk_gpio_read_pull' was not declared. Should it be static? Note, this now produces warnings that these functions are defined but not used. Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> --- Cc: Linus Walleij <linus.walleij@linaro.org> Cc: Alessandro Rubini <rubini@unipv.it> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-gpio@vger.kernel.org --- drivers/pinctrl/nomadik/pinctrl-nomadik.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)