On Sat, Mar 21, 2020 at 1:34 PM Thomas Gleixner <[email protected]> wrote: > > From: Peter Zijlstra <[email protected]> > > In order to avoid future header hell, remove the inclusion of > proc_fs.h from acpi_bus.h. All it needs is a forward declaration of a > struct. >
Acked-by: Andy Shevchenko <[email protected]> # for PDx86 > Signed-off-by: Peter Zijlstra (Intel) <[email protected]> > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Darren Hart <[email protected]> > Cc: Andy Shevchenko <[email protected]> > Cc: [email protected] > Cc: Greg Kroah-Hartman <[email protected]> > Cc: Zhang Rui <[email protected]> > Cc: "Rafael J. Wysocki" <[email protected]> > Cc: [email protected] > Cc: Len Brown <[email protected]> > Cc: [email protected] > --- > drivers/platform/x86/dell-smo8800.c | 1 + > drivers/platform/x86/wmi.c | 1 + > drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.c | 1 + > include/acpi/acpi_bus.h | 2 +- > 4 files changed, 4 insertions(+), 1 deletion(-) > > --- a/drivers/platform/x86/dell-smo8800.c > +++ b/drivers/platform/x86/dell-smo8800.c > @@ -16,6 +16,7 @@ > #include <linux/interrupt.h> > #include <linux/miscdevice.h> > #include <linux/uaccess.h> > +#include <linux/fs.h> > > struct smo8800_device { > u32 irq; /* acpi device irq */ > --- a/drivers/platform/x86/wmi.c > +++ b/drivers/platform/x86/wmi.c > @@ -29,6 +29,7 @@ > #include <linux/uaccess.h> > #include <linux/uuid.h> > #include <linux/wmi.h> > +#include <linux/fs.h> > #include <uapi/linux/wmi.h> > > ACPI_MODULE_NAME("wmi"); > --- a/drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.c > +++ b/drivers/thermal/intel/int340x_thermal/acpi_thermal_rel.c > @@ -19,6 +19,7 @@ > #include <linux/acpi.h> > #include <linux/uaccess.h> > #include <linux/miscdevice.h> > +#include <linux/fs.h> > #include "acpi_thermal_rel.h" > > static acpi_handle acpi_thermal_rel_handle; > --- a/include/acpi/acpi_bus.h > +++ b/include/acpi/acpi_bus.h > @@ -80,7 +80,7 @@ bool acpi_dev_present(const char *hid, c > > #ifdef CONFIG_ACPI > > -#include <linux/proc_fs.h> > +struct proc_dir_entry; > > #define ACPI_BUS_FILE_ROOT "acpi" > extern struct proc_dir_entry *acpi_root_dir; > > -- With Best Regards, Andy Shevchenko
