[PATCH v3] OMAP: NAND: Fix static declaration warning
Vimal Singh
vimal.newwork at gmail.com
Thu Oct 7 13:35:10 EDT 2010
On Wed, Oct 6, 2010 at 3:26 AM, G, Manjunath Kondaiah <manjugk at ti.com> 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 at ti.com>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-mtd at lists.infradead.org
> Cc: Tony Lindgren <tony at atomide.com>
> Cc: Nishanth Menon <nm at ti.com>
> ---
> 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
More information about the linux-arm-kernel
mailing list