Re: [Linaro-acpi] [PATCH v3 05/13] ACPI: ARM: exclude calls on ARM platforms, not include them on x86

On 11/09/2013 05:07 PM, al.stone@linaro.org wrote:

From: Al Stone ahs3@redhat.com

Corrected #ifdef so that DMI is not used on ARM platforms.

Signed-off-by: Al Stone al.stone@linaro.org

drivers/acpi/bus.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c index 7fcbc6a..5e55b53 100644 --- a/drivers/acpi/bus.c +++ b/drivers/acpi/bus.c @@ -512,9 +512,9 @@ void __init acpi_early_init(void)

acpi_gbl_permanent_mmap = 1;

-#ifdef CONFIG_X86 +#if (CONFIG_ARM || CONFIG_ARM64) /*

• * NB: ARM does not use DMI; only older Intel products do.
  

• * NB: ARM does not use DMI at present.
  

  • If the machine falls into the DMI check table,
  • DSDT will be copied to memory

Whups. The if logic is backwards. And wrong. It should be:

#if !(defined(CONFIG_ARM) || defined(CONFIG_ARM64))