2013-03-11 15:10:56

by Liu, Jinsong

[permalink] [raw]
Subject: [PATCH] Xen/acpi: remove redundant acpi/acpi_drivers.h include

>From 3e039eaf865a98971d7316e28c4ee5f3cba1d634 Mon Sep 17 00:00:00 2001
From: Liu Jinsong <[email protected]>
Date: Tue, 12 Mar 2013 06:42:16 +0800
Subject: [PATCH] Xen/acpi: remove redundant acpi/acpi_drivers.h include

It's redundant since linux/acpi.h has include it when CONFIG_ACPI enabled,
and when CONFIG_ACPI disabled it will trigger compiling warning

In file included from drivers/xen/xen-stub.c:28:0:
include/acpi/acpi_drivers.h:103:31:
warning: 'struct acpi_device' declared inside parameter list [enabled by default]
include/acpi/acpi_drivers.h:103:31:
warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]
include/acpi/acpi_drivers.h:107:43:
warning: 'struct acpi_pci_root' declared inside parameter list [enabled by default]

Reported-by: Wu Fengguang <[email protected]>
Signed-off-by: Liu Jinsong <[email protected]>
---
drivers/xen/xen-stub.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/xen/xen-stub.c b/drivers/xen/xen-stub.c
index d85e411..bbef194 100644
--- a/drivers/xen/xen-stub.c
+++ b/drivers/xen/xen-stub.c
@@ -25,7 +25,6 @@
#include <linux/export.h>
#include <linux/types.h>
#include <linux/acpi.h>
-#include <acpi/acpi_drivers.h>
#include <xen/acpi.h>

#ifdef CONFIG_ACPI
--
1.7.1


Attachments:
0001-Xen-acpi-remove-redundant-acpi-acpi_drivers.h-includ.patch (1.30 kB)
0001-Xen-acpi-remove-redundant-acpi-acpi_drivers.h-includ.patch