arm-cci: Split the code for PMU vs driver support
authorSuzuki K. Poulose <suzuki.poulose@arm.com>
Wed, 18 Mar 2015 12:24:41 +0000 (12:24 +0000)
committerWill Deacon <will.deacon@arm.com>
Fri, 27 Mar 2015 13:44:43 +0000 (13:44 +0000)
This patch separates the PMU driver code from the low level
CCI driver code and enables the PMU driver for ARM64.

Introduces config options for both.

 ARM_CCI400_PORT_CTRL - controls the low level driver code for
  CCI400 ports.
 ARM_CCI400_PMU - controls the PMU driver code
 ARM_CCI400_COMMON - Common defintions for CCI400

This patch also changes:
 ARM_CCI - common code for probing the CCI devices. This can be
   used for adding support for newer CCI versions(e.g, CCI-500).

Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Kukjin Kim <kgene@kernel.org>
Cc: Abhilash Kesavan <a.kesavan@samsung.com>
Cc: Liviu Dudau <liviu.dudau@arm.com>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: Sudeep Holla <sudeep.holla@arm.com>
Cc: Nicolas Pitre <nicolas.pitre@linaro.org>
Cc: Punit Agrawal <punit.agrawal@arm.com>
Acked-by: Sudeep Holla <sudeep.holla@arm.com>
Acked-by: Nicolas Pitre <nicolas.pitre@linaro.org>
Acked-by: Punit Agrawal <punit.agrawal@arm.com>
Signed-off-by: Suzuki K. Poulose <suzuki.poulose@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
arch/arm/mach-exynos/Kconfig
arch/arm/mach-vexpress/Kconfig
drivers/bus/Kconfig
drivers/bus/arm-cci.c
include/linux/arm-cci.h

index 603820e5aba79bc17bab7acd56f87a69af087c45..81064cd61a0a9e72536d6dd6df56555bae352d61 100644 (file)
@@ -123,7 +123,7 @@ config SOC_EXYNOS5800
 config EXYNOS5420_MCPM
        bool "Exynos5420 Multi-Cluster PM support"
        depends on MCPM && SOC_EXYNOS5420
-       select ARM_CCI
+       select ARM_CCI400_PORT_CTRL
        select ARM_CPU_SUSPEND
        help
          This is needed to provide CPU and cluster power management
index 3c2509b4b6946bfcfd9b4e7a325b3945ba6c244c..daa7ab6cb909618064ddc948560e1b4b200740d4 100644 (file)
@@ -53,7 +53,7 @@ config ARCH_VEXPRESS_CORTEX_A5_A9_ERRATA
 config ARCH_VEXPRESS_DCSCB
        bool "Dual Cluster System Control Block (DCSCB) support"
        depends on MCPM
-       select ARM_CCI
+       select ARM_CCI400_PORT_CTRL
        help
          Support for the Dual Cluster System Configuration Block (DCSCB).
          This is needed to provide CPU and cluster power management
@@ -71,7 +71,7 @@ config ARCH_VEXPRESS_SPC
 config ARCH_VEXPRESS_TC2_PM
        bool "Versatile Express TC2 power management"
        depends on MCPM
-       select ARM_CCI
+       select ARM_CCI400_PORT_CTRL
        select ARCH_VEXPRESS_SPC
        select ARM_CPU_SUSPEND
        help
index b99729e368608bda800d03f2e2f339d86f2e4d6c..79e297b1f221c1dd1b2464f53bffb54d00a8b4cd 100644 (file)
@@ -43,12 +43,32 @@ config OMAP_INTERCONNECT
        help
          Driver to enable OMAP interconnect error handling driver.
 
-config ARM_CCI
-       bool "ARM CCI driver support"
+config ARM_CCI400_PORT_CTRL
+       bool
        depends on ARM && OF && CPU_V7
+       select ARM_CCI400_COMMON
+       help
+         Low level power management driver for CCI400 cache coherent
+         interconnect for ARM platforms.
+
+config ARM_CCI400_PMU
+       bool "ARM CCI400 PMU support"
+       default y
+       depends on ARM || ARM64
+       depends on HW_PERF_EVENTS
+       select ARM_CCI400_COMMON
        help
-         Driver supporting the CCI cache coherent interconnect for ARM
-         platforms.
+         Support for PMU events monitoring on the ARM CCI cache coherent
+         interconnect.
+
+         If unsure, say Y
+
+config ARM_CCI400_COMMON
+       bool
+       select ARM_CCI
+
+config ARM_CCI
+       bool
 
 config ARM_CCN
        bool "ARM CCN driver support"
index a23663c7a306422b7373202be27fec17dae13b6e..054df84562c2efa9a0377cc8ad284d36e369c42b 100644 (file)
@@ -32,6 +32,7 @@
 static void __iomem *cci_ctrl_base;
 static unsigned long cci_ctrl_phys;
 
+#ifdef CONFIG_ARM_CCI400_PORT_CTRL
 struct cci_nb_ports {
        unsigned int nb_ace;
        unsigned int nb_ace_lite;
@@ -42,12 +43,19 @@ static const struct cci_nb_ports cci400_ports = {
        .nb_ace_lite = 3
 };
 
+#define CCI400_PORTS_DATA      (&cci400_ports)
+#else
+#define CCI400_PORTS_DATA      (NULL)
+#endif
+
 static const struct of_device_id arm_cci_matches[] = {
-       {.compatible = "arm,cci-400", .data = &cci400_ports },
+#ifdef CONFIG_ARM_CCI400_COMMON
+       {.compatible = "arm,cci-400", .data = CCI400_PORTS_DATA },
+#endif
        {},
 };
 
-#ifdef CONFIG_HW_PERF_EVENTS
+#ifdef CONFIG_ARM_CCI400_PMU
 
 #define DRIVER_NAME            "CCI-400"
 #define DRIVER_NAME_PMU                DRIVER_NAME " PMU"
@@ -1022,14 +1030,16 @@ static int __init cci_platform_init(void)
        return platform_driver_register(&cci_platform_driver);
 }
 
-#else /* !CONFIG_HW_PERF_EVENTS */
+#else /* !CONFIG_ARM_CCI400_PMU */
 
 static int __init cci_platform_init(void)
 {
        return 0;
 }
 
-#endif /* CONFIG_HW_PERF_EVENTS */
+#endif /* CONFIG_ARM_CCI400_PMU */
+
+#ifdef CONFIG_ARM_CCI400_PORT_CTRL
 
 #define CCI_PORT_CTRL          0x0
 #define CCI_CTRL_STATUS                0xc
@@ -1460,6 +1470,12 @@ static int cci_probe_ports(struct device_node *np)
 
        return 0;
 }
+#else /* !CONFIG_ARM_CCI400_PORT_CTRL */
+static inline int cci_probe_ports(struct device_node *np)
+{
+       return 0;
+}
+#endif /* CONFIG_ARM_CCI400_PORT_CTRL */
 
 static int cci_probe(void)
 {
index aede5c765eec2b71bcc396b0583189a7cbd6eff5..521ec1f2e6bc08441279c0ca5245d82394885ddf 100644 (file)
@@ -30,12 +30,16 @@ struct device_node;
 
 #ifdef CONFIG_ARM_CCI
 extern bool cci_probed(void);
+#else
+static inline bool cci_probed(void) { return false; }
+#endif
+
+#ifdef CONFIG_ARM_CCI400_PORT_CTRL
 extern int cci_ace_get_port(struct device_node *dn);
 extern int cci_disable_port_by_cpu(u64 mpidr);
 extern int __cci_control_port_by_device(struct device_node *dn, bool enable);
 extern int __cci_control_port_by_index(u32 port, bool enable);
 #else
-static inline bool cci_probed(void) { return false; }
 static inline int cci_ace_get_port(struct device_node *dn)
 {
        return -ENODEV;
@@ -51,6 +55,7 @@ static inline int __cci_control_port_by_index(u32 port, bool enable)
        return -ENODEV;
 }
 #endif
+
 #define cci_disable_port_by_device(dev) \
        __cci_control_port_by_device(dev, false)
 #define cci_enable_port_by_device(dev) \