From mboxrd@z Thu Jan 1 00:00:00 1970 From: sameo@linux.intel.com (Samuel Ortiz) Date: Mon, 27 Sep 2010 13:16:10 +0200 Subject: [PATCH v2 08/10] TWL IRQ: Fix fucntion declaration warnings In-Reply-To: <1285063280-4057-9-git-send-email-manjugk@ti.com> References: <1285063280-4057-1-git-send-email-manjugk@ti.com> <1285063280-4057-9-git-send-email-manjugk@ti.com> Message-ID: <20100927111609.GF2560@sortiz-mobl> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Manjunath, On Tue, Sep 21, 2010 at 03:31:18PM +0530, G, Manjunath Kondaiah wrote: > Fixes following sparse warnings for twl4030 and twl6030 irq files. > > drivers/mfd/twl4030-irq.c:783:5: warning: symbol 'twl4030_init_irq' was not declared. Should it be static? > drivers/mfd/twl4030-irq.c:863:5: warning: symbol 'twl4030_exit_irq' was not declared. Should it be static? > drivers/mfd/twl4030-irq.c:873:5: warning: symbol 'twl4030_init_chip_irq' was not declared. Should it be static? > > drivers/mfd/twl6030-irq.c:226:5: warning: symbol 'twl6030_init_irq' was not declared. Should it be static? > drivers/mfd/twl6030-irq.c:290:5: warning: symbol 'twl6030_exit_irq' was not declared. Should it be static? > > Signed-off-by: G, Manjunath Kondaiah > Cc: linux-arm-kernel at lists.infradead.org > Cc: Samuel Ortiz > Cc: Tony Lindgren > Cc: Nishanth Menon > --- > include/linux/i2c/twl.h | 5 +++++ > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/include/linux/i2c/twl.h b/include/linux/i2c/twl.h > index 6de90bf..c61e90a 100644 > --- a/include/linux/i2c/twl.h > +++ b/include/linux/i2c/twl.h > @@ -172,6 +172,11 @@ int twl_i2c_read(u8 mod_no, u8 *value, u8 reg, unsigned num_bytes); > > int twl6030_interrupt_unmask(u8 bit_mask, u8 offset); > int twl6030_interrupt_mask(u8 bit_mask, u8 offset); > +int twl6030_init_irq(int irq_num, unsigned irq_base, unsigned irq_end); > +int twl6030_exit_irq(void); > +int twl4030_init_irq(int irq_num, unsigned irq_base, unsigned irq_end); > +int twl4030_exit_irq(void); > +int twl4030_init_chip_irq(const char *chip); No, we don't want to export those. Try defining them as extern from twl*-irq.c. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/