On Wed, Oct 6, 2010 at 3:26 AM, G, Manjunath Kondaiah <manjugk@xxxxxx> wrote: > This patch fixes sparse warning for static declaration of variable "use_dma" > > drivers/mtd/nand/omap2.c:114:11: warning: symbol 'use_dma' was not declared. Should it be static? > > Signed-off-by: G, Manjunath Kondaiah <manjugk@xxxxxx> > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > Cc: linux-mtd@xxxxxxxxxxxxxxxxxxx > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > Cc: Nishanth Menon <nm@xxxxxx> > --- > Changes since v1: > - no logical changes, patch seperated from series based Tony's suggestion. > > drivers/mtd/nand/omap2.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c > index 133d515..439e80d 100644 > --- a/drivers/mtd/nand/omap2.c > +++ b/drivers/mtd/nand/omap2.c > @@ -111,11 +111,11 @@ static int use_dma = 1; > module_param(use_dma, bool, 0); > MODULE_PARM_DESC(use_dma, "enable/disable use of DMA"); > #else > -const int use_dma; > +static const int use_dma; > #endif > #else > const int use_prefetch; > -const int use_dma; > +static const int use_dma; I did not understand, why did it not pointed same error for 'const int use_prefetch'? -- Regards, Vimal Singh -- 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
