On Fri, 19 Nov 2010 17:02:00 +0100 Jean Delvare wrote: > Building the kernel (2.6.37-rc2-git5) with CONFIG_ACPI_VIDEO=n fails > for me because of these errors: > > include/acpi/video.h:22:11: warning: "struct acpi_device" declared inside parameter list > include/acpi/video.h:22:11: warning: its scope is only this definition or declaration, which is probably not what you want > include/acpi/video.h: In function "acpi_video_get_edid": > include/acpi/video.h:24:10: error: "ENODEV" undeclared (first use in this function) > include/acpi/video.h:24:10: note: each undeclared identifier is reported only once for each function it appears in > > Adding the missing include and structure declaration fixes it. > > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx> Acked-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx> This is the third patch like this. Please apply one of them. > --- > include/acpi/video.h | 4 ++++ > 1 file changed, 4 insertions(+) > > --- linux-2.6.37-rc2.orig/include/acpi/video.h 2010-11-19 14:56:45.000000000 +0100 > +++ linux-2.6.37-rc2/include/acpi/video.h 2010-11-19 15:00:23.000000000 +0100 > @@ -1,6 +1,8 @@ > #ifndef __ACPI_VIDEO_H > #define __ACPI_VIDEO_H > > +#include <linux/errno.h> > + > #define ACPI_VIDEO_DISPLAY_CRT 1 > #define ACPI_VIDEO_DISPLAY_TV 2 > #define ACPI_VIDEO_DISPLAY_DVI 3 > @@ -10,6 +12,8 @@ > #define ACPI_VIDEO_DISPLAY_LEGACY_PANEL 0x0110 > #define ACPI_VIDEO_DISPLAY_LEGACY_TV 0x0200 > > +struct acpi_device; > + > #if (defined CONFIG_ACPI_VIDEO || defined CONFIG_ACPI_VIDEO_MODULE) > extern int acpi_video_register(void); > extern void acpi_video_unregister(void); > > > -- --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
