> -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx > [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of G, > Manjunath Kondaiah > Sent: Monday, September 27, 2010 6:40 PM > To: Samuel Ortiz > Cc: linux-omap@xxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Tony Lindgren; Menon, Nishanth > Subject: RE: [PATCH v2 08/10] TWL IRQ: Fix fucntion > declaration warnings > > > Hi Samuel, > > > -----Original Message----- > > From: Samuel Ortiz [mailto:sameo@xxxxxxxxxxxxxxx] > > Sent: Monday, September 27, 2010 4:46 PM > > To: G, Manjunath Kondaiah > > Cc: linux-omap@xxxxxxxxxxxxxxx; > > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Tony Lindgren; Menon, Nishanth > > Subject: Re: [PATCH v2 08/10] TWL IRQ: Fix fucntion declaration > > warnings > > > > 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 <manjugk@xxxxxx> > > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > > Cc: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> > > > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > > > Cc: Nishanth Menon <nm@xxxxxx> > > > --- > > > 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. > > Having extern in .c file will generate checkpatch warning as: > WARNING: externs should be avoided in .c files As an alternate, how about having twl-core.h in drivers/mfd and defining them as extern so that these API's will be available only to files under drivers/mfd? -Manjunath-- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html