On Tue, Jul 24, 2018 at 12:21 PM, Will Deacon [off-list ref] wrote:

On Tue, Jul 24, 2018 at 11:54:25AM +0200, Rafael J. Wysocki wrote:

quoted

On Tue, Jul 24, 2018 at 11:48 AM, Arnd Bergmann [off-list ref] wrote:

quoted

Kconfig reports a warning on x86 builds after the ARM64 dependency
was added.

drivers/acpi/Kconfig:6:error: recursive dependency detected!
drivers/acpi/Kconfig:6:       symbol ACPI depends on EFI

This rephrases the dependency to keep the ARM64 details out of the
shared Kconfig file, so Kconfig no longer gets confused by it.

For consistency, all three architectures that support ACPI now
select ARCH_SUPPORTS_ACPI in exactly the configuration in which
they allow it. We still need the 'default x86', as each one
wants a different default: default-y on x86, default-n on arm64,
and always-y on ia64.

Fixes: 5bcd44083a08 ("drivers: acpi: add dependency of EFI for arm64")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
v2: use 'select ARCH_SUPPORTS_ACPI' for all three.

LGTM

Reviewed-by: Rafael J. Wysocki <redacted>

Same here:

Acked-by: Will Deacon <redacted>

Would it be ok if I take this via the arm64 tree, since that's where the
warning was introduced?

Yes, it would, from the ACPI side.