From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: [PATCH] ACPI: Fix when CONFIG_ACPI_VIDEO=n Date: Fri, 19 Nov 2010 17:02:00 +0100 Message-ID: <20101119170200.2db4ab81@endymion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from zone0.gcu-squad.org ([212.85.147.21]:5698 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755232Ab0KSQCP (ORCPT ); Fri, 19 Nov 2010 11:02:15 -0500 Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Len Brown Cc: linux-acpi@vger.kernel.org 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 --- 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 + #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); -- Jean Delvare