* [PATCH 0/2] ARM: mvebu: clear shared L2 bit on Armada XP
@ 2015-07-08 14:09 Thomas Petazzoni
2015-07-08 14:09 ` [PATCH 1/2] ARM: mvebu: add support to " Thomas Petazzoni
2015-07-08 14:09 ` [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree Thomas Petazzoni
0 siblings, 2 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2015-07-08 14:09 UTC (permalink / raw
To: linux-arm-kernel
Hello,
This set of two patches add support for clearing the shared L2 bit on
Marvell Armada XP. This is part of an effort to progressively close
the gap between Marvell's vendor BSP and the mainline kernel.
Best regards,
Thomas
Thomas Petazzoni (2):
ARM: mvebu: add support to clear shared L2 bit on Armada XP
ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree
.../devicetree/bindings/arm/mvebu-cpu-config.txt | 20 +++++++
arch/arm/boot/dts/armada-370.dtsi | 5 ++
arch/arm/boot/dts/armada-xp.dtsi | 5 ++
arch/arm/mach-mvebu/coherency.c | 61 ++++++++++++++++++++++
4 files changed, 91 insertions(+)
create mode 100644 Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
--
2.4.5
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] ARM: mvebu: add support to clear shared L2 bit on Armada XP
2015-07-08 14:09 [PATCH 0/2] ARM: mvebu: clear shared L2 bit on Armada XP Thomas Petazzoni
@ 2015-07-08 14:09 ` Thomas Petazzoni
2015-09-29 13:42 ` Gregory CLEMENT
2015-07-08 14:09 ` [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree Thomas Petazzoni
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2015-07-08 14:09 UTC (permalink / raw
To: linux-arm-kernel
For optimal performance, in a HW I/O coherency context such as the one
used on Armada XP, the shared L2 bit of the CPU configuration register
should be cleared.
This commit adjusts the coherency fabric code used by Marvell EBU
processors to clear this bit on Armada XP. Since it's a per-CPU
register, it's cleared in set_cpu_coherent() for the boot CPU, and
through a CPU notifier for the non-boot CPUs.
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
.../devicetree/bindings/arm/mvebu-cpu-config.txt | 20 +++++++
arch/arm/mach-mvebu/coherency.c | 61 ++++++++++++++++++++++
2 files changed, 81 insertions(+)
create mode 100644 Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
diff --git a/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt b/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
new file mode 100644
index 0000000..2cdcd71
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
@@ -0,0 +1,20 @@
+MVEBU CPU Config registers
+--------------------------
+
+MVEBU (Marvell SOCs: Armada 370/XP)
+
+Required properties:
+
+- compatible: one of:
+ - "marvell,armada-370-cpu-config"
+ - "marvell,armada-xp-cpu-config"
+
+- reg: Should contain CPU config registers location and length, in
+ their per-CPU variant
+
+Example:
+
+ cpu-config at 21000 {
+ compatible = "marvell,armada-xp-cpu-config";
+ reg = <0x21000 0x8>;
+ };
diff --git a/arch/arm/mach-mvebu/coherency.c b/arch/arm/mach-mvebu/coherency.c
index e46e9ea..48655f8 100644
--- a/arch/arm/mach-mvebu/coherency.c
+++ b/arch/arm/mach-mvebu/coherency.c
@@ -40,6 +40,7 @@
unsigned long coherency_phys_base;
void __iomem *coherency_base;
static void __iomem *coherency_cpu_base;
+static void __iomem *cpu_config_base;
/* Coherency fabric registers */
#define IO_SYNC_BARRIER_CTL_OFFSET 0x0
@@ -65,6 +66,31 @@ static const struct of_device_id of_coherency_table[] = {
int ll_enable_coherency(void);
void ll_add_cpu_to_smp_group(void);
+#define CPU_CONFIG_SHARED_L2 BIT(16)
+
+/*
+ * Disable the "Shared L2 Present" bit in CPU Configuration register
+ * on Armada XP.
+ *
+ * The "Shared L2 Present" bit affects the "level of coherence" value
+ * in the clidr CP15 register. Cache operation functions such as
+ * "flush all" and "invalidate all" operate on all the cache levels
+ * that included in the defined level of coherence. When HW I/O
+ * coherency is used, this bit causes unnecessary flushes of the L2
+ * cache.
+ */
+static void armada_xp_clear_shared_l2(void)
+{
+ u32 reg;
+
+ if (!cpu_config_base)
+ return;
+
+ reg = readl(cpu_config_base);
+ reg &= ~CPU_CONFIG_SHARED_L2;
+ writel(reg, cpu_config_base);
+}
+
int set_cpu_coherent(void)
{
if (!coherency_base) {
@@ -73,10 +99,13 @@ int set_cpu_coherent(void)
return 1;
}
+ armada_xp_clear_shared_l2();
ll_add_cpu_to_smp_group();
return ll_enable_coherency();
}
+
+
static int mvebu_hwcc_notifier(struct notifier_block *nb,
unsigned long event, void *__dev)
{
@@ -97,9 +126,24 @@ static struct notifier_block mvebu_hwcc_pci_nb = {
.notifier_call = mvebu_hwcc_notifier,
};
+static int armada_xp_clear_shared_l2_notifier_func(struct notifier_block *nfb,
+ unsigned long action, void *hcpu)
+{
+ if (action == CPU_STARTING || action == CPU_STARTING_FROZEN)
+ armada_xp_clear_shared_l2();
+
+ return NOTIFY_OK;
+}
+
+static struct notifier_block armada_xp_clear_shared_l2_notifier = {
+ .notifier_call = armada_xp_clear_shared_l2_notifier_func,
+ .priority = 100,
+};
+
static void __init armada_370_coherency_init(struct device_node *np)
{
struct resource res;
+ struct device_node *cpu_config_np;
of_address_to_resource(np, 0, &res);
coherency_phys_base = res.start;
@@ -112,6 +156,23 @@ static void __init armada_370_coherency_init(struct device_node *np)
sync_cache_w(&coherency_phys_base);
coherency_base = of_iomap(np, 0);
coherency_cpu_base = of_iomap(np, 1);
+
+ cpu_config_np = of_find_compatible_node(NULL, NULL,
+ "marvell,armada-xp-cpu-config");
+ if (!cpu_config_np)
+ goto exit;
+
+ cpu_config_base = of_iomap(cpu_config_np, 0);
+ if (!cpu_config_base) {
+ of_node_put(cpu_config_np);
+ goto exit;
+ }
+
+ of_node_put(cpu_config_np);
+
+ register_cpu_notifier(&armada_xp_clear_shared_l2_notifier);
+
+exit:
set_cpu_coherent();
}
--
2.4.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree
2015-07-08 14:09 [PATCH 0/2] ARM: mvebu: clear shared L2 bit on Armada XP Thomas Petazzoni
2015-07-08 14:09 ` [PATCH 1/2] ARM: mvebu: add support to " Thomas Petazzoni
@ 2015-07-08 14:09 ` Thomas Petazzoni
2015-09-29 13:43 ` Gregory CLEMENT
1 sibling, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2015-07-08 14:09 UTC (permalink / raw
To: linux-arm-kernel
This commit adds the description of the CPU config registers in the
Armada 370 and Armada XP Device Tree. Since the registers are in fact
different between the two SoCs, a different compatible string is used.
Note that the Armada 370 node is currently unused, but it is
nonetheless added for consistency with the addition on the Armada XP
side.
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
arch/arm/boot/dts/armada-370.dtsi | 5 +++++
arch/arm/boot/dts/armada-xp.dtsi | 5 +++++
2 files changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/armada-370.dtsi b/arch/arm/boot/dts/armada-370.dtsi
index 53a1a5a..45eff24 100644
--- a/arch/arm/boot/dts/armada-370.dtsi
+++ b/arch/arm/boot/dts/armada-370.dtsi
@@ -256,6 +256,11 @@
reg = <0x20800 0x8>;
};
+ cpu-config at 21000 {
+ compatible = "marvell,armada-370-cpu-config";
+ reg = <0x21000 0x8>;
+ };
+
audio_controller: audio-controller at 30000 {
#sound-dai-cells = <1>;
compatible = "marvell,armada370-audio";
diff --git a/arch/arm/boot/dts/armada-xp.dtsi b/arch/arm/boot/dts/armada-xp.dtsi
index 3de9b76..1c64ac3 100644
--- a/arch/arm/boot/dts/armada-xp.dtsi
+++ b/arch/arm/boot/dts/armada-xp.dtsi
@@ -184,6 +184,11 @@
reg = <0x20800 0x20>;
};
+ cpu-config at 21000 {
+ compatible = "marvell,armada-xp-cpu-config";
+ reg = <0x21000 0x8>;
+ };
+
eth2: ethernet at 30000 {
compatible = "marvell,armada-xp-neta";
reg = <0x30000 0x4000>;
--
2.4.5
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 1/2] ARM: mvebu: add support to clear shared L2 bit on Armada XP
2015-07-08 14:09 ` [PATCH 1/2] ARM: mvebu: add support to " Thomas Petazzoni
@ 2015-09-29 13:42 ` Gregory CLEMENT
0 siblings, 0 replies; 5+ messages in thread
From: Gregory CLEMENT @ 2015-09-29 13:42 UTC (permalink / raw
To: linux-arm-kernel
Hi Thomas,
On mer., juil. 08 2015, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
> For optimal performance, in a HW I/O coherency context such as the one
> used on Armada XP, the shared L2 bit of the CPU configuration register
> should be cleared.
>
> This commit adjusts the coherency fabric code used by Marvell EBU
> processors to clear this bit on Armada XP. Since it's a per-CPU
> register, it's cleared in set_cpu_coherent() for the boot CPU, and
> through a CPU notifier for the non-boot CPUs.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Applied on mvebu/soc with a conflict resolution due to the rebase on 4.3-rc1
Thanks,
Gregory
> ---
> .../devicetree/bindings/arm/mvebu-cpu-config.txt | 20 +++++++
> arch/arm/mach-mvebu/coherency.c | 61 ++++++++++++++++++++++
> 2 files changed, 81 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
>
> diff --git a/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt b/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
> new file mode 100644
> index 0000000..2cdcd71
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/mvebu-cpu-config.txt
> @@ -0,0 +1,20 @@
> +MVEBU CPU Config registers
> +--------------------------
> +
> +MVEBU (Marvell SOCs: Armada 370/XP)
> +
> +Required properties:
> +
> +- compatible: one of:
> + - "marvell,armada-370-cpu-config"
> + - "marvell,armada-xp-cpu-config"
> +
> +- reg: Should contain CPU config registers location and length, in
> + their per-CPU variant
> +
> +Example:
> +
> + cpu-config at 21000 {
> + compatible = "marvell,armada-xp-cpu-config";
> + reg = <0x21000 0x8>;
> + };
> diff --git a/arch/arm/mach-mvebu/coherency.c b/arch/arm/mach-mvebu/coherency.c
> index e46e9ea..48655f8 100644
> --- a/arch/arm/mach-mvebu/coherency.c
> +++ b/arch/arm/mach-mvebu/coherency.c
> @@ -40,6 +40,7 @@
> unsigned long coherency_phys_base;
> void __iomem *coherency_base;
> static void __iomem *coherency_cpu_base;
> +static void __iomem *cpu_config_base;
>
> /* Coherency fabric registers */
> #define IO_SYNC_BARRIER_CTL_OFFSET 0x0
> @@ -65,6 +66,31 @@ static const struct of_device_id of_coherency_table[] = {
> int ll_enable_coherency(void);
> void ll_add_cpu_to_smp_group(void);
>
> +#define CPU_CONFIG_SHARED_L2 BIT(16)
> +
> +/*
> + * Disable the "Shared L2 Present" bit in CPU Configuration register
> + * on Armada XP.
> + *
> + * The "Shared L2 Present" bit affects the "level of coherence" value
> + * in the clidr CP15 register. Cache operation functions such as
> + * "flush all" and "invalidate all" operate on all the cache levels
> + * that included in the defined level of coherence. When HW I/O
> + * coherency is used, this bit causes unnecessary flushes of the L2
> + * cache.
> + */
> +static void armada_xp_clear_shared_l2(void)
> +{
> + u32 reg;
> +
> + if (!cpu_config_base)
> + return;
> +
> + reg = readl(cpu_config_base);
> + reg &= ~CPU_CONFIG_SHARED_L2;
> + writel(reg, cpu_config_base);
> +}
> +
> int set_cpu_coherent(void)
> {
> if (!coherency_base) {
> @@ -73,10 +99,13 @@ int set_cpu_coherent(void)
> return 1;
> }
>
> + armada_xp_clear_shared_l2();
> ll_add_cpu_to_smp_group();
> return ll_enable_coherency();
> }
>
> +
> +
> static int mvebu_hwcc_notifier(struct notifier_block *nb,
> unsigned long event, void *__dev)
> {
> @@ -97,9 +126,24 @@ static struct notifier_block mvebu_hwcc_pci_nb = {
> .notifier_call = mvebu_hwcc_notifier,
> };
>
> +static int armada_xp_clear_shared_l2_notifier_func(struct notifier_block *nfb,
> + unsigned long action, void *hcpu)
> +{
> + if (action == CPU_STARTING || action == CPU_STARTING_FROZEN)
> + armada_xp_clear_shared_l2();
> +
> + return NOTIFY_OK;
> +}
> +
> +static struct notifier_block armada_xp_clear_shared_l2_notifier = {
> + .notifier_call = armada_xp_clear_shared_l2_notifier_func,
> + .priority = 100,
> +};
> +
> static void __init armada_370_coherency_init(struct device_node *np)
> {
> struct resource res;
> + struct device_node *cpu_config_np;
>
> of_address_to_resource(np, 0, &res);
> coherency_phys_base = res.start;
> @@ -112,6 +156,23 @@ static void __init armada_370_coherency_init(struct device_node *np)
> sync_cache_w(&coherency_phys_base);
> coherency_base = of_iomap(np, 0);
> coherency_cpu_base = of_iomap(np, 1);
> +
> + cpu_config_np = of_find_compatible_node(NULL, NULL,
> + "marvell,armada-xp-cpu-config");
> + if (!cpu_config_np)
> + goto exit;
> +
> + cpu_config_base = of_iomap(cpu_config_np, 0);
> + if (!cpu_config_base) {
> + of_node_put(cpu_config_np);
> + goto exit;
> + }
> +
> + of_node_put(cpu_config_np);
> +
> + register_cpu_notifier(&armada_xp_clear_shared_l2_notifier);
> +
> +exit:
> set_cpu_coherent();
> }
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree
2015-07-08 14:09 ` [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree Thomas Petazzoni
@ 2015-09-29 13:43 ` Gregory CLEMENT
0 siblings, 0 replies; 5+ messages in thread
From: Gregory CLEMENT @ 2015-09-29 13:43 UTC (permalink / raw
To: linux-arm-kernel
Hi Thomas,
On mer., juil. 08 2015, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
> This commit adds the description of the CPU config registers in the
> Armada 370 and Armada XP Device Tree. Since the registers are in fact
> different between the two SoCs, a different compatible string is used.
>
> Note that the Armada 370 node is currently unused, but it is
> nonetheless added for consistency with the addition on the Armada XP
> side.
Applied on mvebu/dt
Thanks,
Gregory
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
> arch/arm/boot/dts/armada-370.dtsi | 5 +++++
> arch/arm/boot/dts/armada-xp.dtsi | 5 +++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/armada-370.dtsi b/arch/arm/boot/dts/armada-370.dtsi
> index 53a1a5a..45eff24 100644
> --- a/arch/arm/boot/dts/armada-370.dtsi
> +++ b/arch/arm/boot/dts/armada-370.dtsi
> @@ -256,6 +256,11 @@
> reg = <0x20800 0x8>;
> };
>
> + cpu-config at 21000 {
> + compatible = "marvell,armada-370-cpu-config";
> + reg = <0x21000 0x8>;
> + };
> +
> audio_controller: audio-controller at 30000 {
> #sound-dai-cells = <1>;
> compatible = "marvell,armada370-audio";
> diff --git a/arch/arm/boot/dts/armada-xp.dtsi b/arch/arm/boot/dts/armada-xp.dtsi
> index 3de9b76..1c64ac3 100644
> --- a/arch/arm/boot/dts/armada-xp.dtsi
> +++ b/arch/arm/boot/dts/armada-xp.dtsi
> @@ -184,6 +184,11 @@
> reg = <0x20800 0x20>;
> };
>
> + cpu-config at 21000 {
> + compatible = "marvell,armada-xp-cpu-config";
> + reg = <0x21000 0x8>;
> + };
> +
> eth2: ethernet at 30000 {
> compatible = "marvell,armada-xp-neta";
> reg = <0x30000 0x4000>;
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-09-29 13:43 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-08 14:09 [PATCH 0/2] ARM: mvebu: clear shared L2 bit on Armada XP Thomas Petazzoni
2015-07-08 14:09 ` [PATCH 1/2] ARM: mvebu: add support to " Thomas Petazzoni
2015-09-29 13:42 ` Gregory CLEMENT
2015-07-08 14:09 ` [PATCH 2/2] ARM: mvebu: add CPU config registers in the Armada 370/XP Device Tree Thomas Petazzoni
2015-09-29 13:43 ` Gregory CLEMENT
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).