arm64: Kconfig: select PM{,_GENERIC_DOMAINS} for ARCH_VEXPRESS
authorSudeep Holla <sudeep.holla@arm.com>
Mon, 20 Jun 2016 14:56:09 +0000 (15:56 +0100)
committerOlof Johansson <olof@lixom.net>
Tue, 21 Jun 2016 05:59:54 +0000 (22:59 -0700)
The Linux AMBA bus framework probes the peripheral IDs when adding the
AMBA devices very early on the boot. Generally they are on APB bus and
just require APB clocks to be on even when most of the core logic of the
IP is powered down.

However on Juno, the entire debugsys domain needs to be ON to access
even the coresight components' CID/PID registers and hence broken by
design. Accessing those while debugsys power domain is off will lead to
the bridge stalling the transactions instead of returning the slave error.

Further, the AMBA framework can't deal with !CONFIG_PM_GENERIC_DOMAINS
case: it ignores the error and proceeds to access the device region.
It was suggested to always enable CONFIG_PM{,_GENERIC_DOMAINS} in order
to handle above explained scenario.

Cc: Arnd Bergmann <arnd@arndb.de>
Suggested-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Signed-off-by: Olof Johansson <olof@lixom.net>
arch/arm64/Kconfig.platforms

index f5e7db6d6b2e18bf6355f6e81b967c05e3d1834d..89712f71ef268cf03f49ff4e8fabeeaf9bd9d610 100644 (file)
@@ -163,6 +163,8 @@ config ARCH_VEXPRESS
        bool "ARMv8 software model (Versatile Express)"
        select ARCH_REQUIRE_GPIOLIB
        select COMMON_CLK_VERSATILE
+       select PM
+       select PM_GENERIC_DOMAINS
        select POWER_RESET_VEXPRESS
        select VEXPRESS_CONFIG
        help