On Saturday, November 09, 2013 06:36:11 PM al.stone@xxxxxxxxxx wrote: > From: Al Stone <ahs3@xxxxxxxxxx> > > To enable the hardware reduced mode of ACPI on some platforms (such as > ARM), we need to modify the kernel code and set ACPI_REDUCED_HARDWARE > to TRUE in the ACPICA source. > > This can be done more resonably by introducing a kernel config item > to enable/disable ACPI_REDUCED_HARDWARE. We can then change the kernel > config instead of having to modify the kernel source directly to enable > the reduced hardware mode of ACPI. > > Lv Zheng suggested that this configuration item does not belong in ACPICA, > the upstream source for much of the ACPI internals, but rather to the > Linux kernel itself. Hence, we introduce this flag so that we can make > ACPI_REDUCED_HARDWARE configurable. For the details of the discussion, > please refer to: http://www.spinics.net/lists/linux-acpi/msg46369.html > > Signed-off-by: Hanjun Guo <hanjun.guo@xxxxxxxxxx> > Signed-off-by: Al Stone <al.stone@xxxxxxxxxx> > --- > drivers/acpi/Kconfig | 8 ++++++++ > include/acpi/platform/aclinux.h | 4 ++++ > 2 files changed, 12 insertions(+) > > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > index 589da05..7bbd3b0 100644 > --- a/drivers/acpi/Kconfig > +++ b/drivers/acpi/Kconfig > @@ -354,6 +354,14 @@ config ACPI_BGRT > data from the firmware boot splash. It will appear under > /sys/firmware/acpi/bgrt/ . > > +config ACPI_REDUCED_HARDWARE > + bool "Hardware-reduced ACPI support" > + depends on !(IA64 || X86) Why don't you use depends on (ARM || ARM64) here instead? > + help > + This config adds support for Hardware-reduced ACPI. When this option > + is selected, will generate a specialized version of ACPICA that ONLY > + supports the ACPI "reduced hardware". > + > source "drivers/acpi/apei/Kconfig" > > endif # ACPI > diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h > index 28f4f4d..ae93a91 100644 > --- a/include/acpi/platform/aclinux.h > +++ b/include/acpi/platform/aclinux.h > @@ -67,6 +67,10 @@ > > /* Host-dependent types and defines for in-kernel ACPICA */ > > +#ifdef CONFIG_ACPI_REDUCED_HARDWARE > +#define ACPI_REDUCED_HARDWARE TRUE > +#endif > + > #define ACPI_MACHINE_WIDTH BITS_PER_LONG > #define ACPI_EXPORT_SYMBOL(symbol) EXPORT_SYMBOL(symbol); > #define strtoul simple_strtoul > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- 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
