Re: pcdp-build-fix.patch added to -mm tree

[Alexey Dobriyan]

On Tuesday 31 May 2005 13:24, akpm@xxxxxxxx wrote:
> --- 25/drivers/firmware/pcdp.c~pcdp-build-fix
> +++ 25-akpm/drivers/firmware/pcdp.c
> @@ -11,6 +11,7 @@
>   * published by the Free Software Foundation.
>   */
>  
> +#include <linux/config.h>
>  #include <linux/acpi.h>

Does this patch make sense?
===========================
Recent restoration of quirk_via_irqpic() #ifdef'fed whole include/linux/acpi.h
with CONFIG_ACPI. So, if acpi.h is unlucky enough to be included before
config.h, bad things happen:

		[tested, patch fixes the warning]

	drivers/serial/8250_acpi.c: In function `acpi_serial_ext_irq':
	drivers/serial/8250_acpi.c:51: warning: implicit declaration of
	function `acpi_register_gsi'

or even

		[untested]

	In file included from drivers/firmware/pcdp.c:18:
	drivers/firmware/pcdp.h:48: error: field 'addr' has incomplete type

Signed-off-by: Alexey Dobriyan <adobriyan@xxxxxxxxx>

--- linux-vanilla/include/linux/acpi.h	2005-05-28 02:59:59.000000000 +0400
+++ linux-8250/include/linux/acpi.h	2005-05-28 03:39:25.000000000 +0400
@@ -25,6 +25,8 @@
 #ifndef _LINUX_ACPI_H
 #define _LINUX_ACPI_H
 
+#include <linux/config.h>
+
 #ifdef	CONFIG_ACPI
 
 #ifndef _LINUX
-
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/