Re: [PATCH -mm] acpi: fix memory_hotplug externs

From: Adrian Bunk
Date: Sat Apr 01 2006 - 17:13:37 EST


On Tue, Mar 28, 2006 at 11:46:55AM -0800, Randy.Dunlap wrote:
> From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
>
> Spell CONFIG option correctly so that externs work.
> Fixes these warnings:
> drivers/acpi/acpi_memhotplug.c:248: warning: implicit declaration of function 'add_memory'
> drivers/acpi/acpi_memhotplug.c:312: warning: implicit declaration of function 'remove_memory'
>
> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> ---
> linsrc/linux-2616-mm2/include/linux/memory_hotplug.h | 2 +-
> 1 files changed, 1 insertion(+), 1 deletion(-)
>
> --- rddunlap.orig/linsrc/linux-2616-mm2/include/linux/memory_hotplug.h
> +++ rddunlap/linsrc/linux-2616-mm2/include/linux/memory_hotplug.h
> @@ -105,7 +105,7 @@ static inline int __remove_pages(struct
> }
>
> #if defined(CONFIG_MEMORY_HOTPLUG) || defined(CONFIG_ACPI_HOTPLUG_MEMORY) \
> - || defined(CONFIG_ACPI_MEMORY_HOTPLUG_MODULE)
> + || defined(CONFIG_ACPI_HOTPLUG_MEMORY_MODULE)
> extern int add_memory(u64 start, u64 size);
> extern int remove_memory(u64 start, u64 size);
> #endif

What about simply offering the prototypes unconditionally?

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/