Skip to contentSkip to conversation

ARM: mvebu: Fix the Aurora L2 cache node with the required cache-unified property

PATCH New Gregory CLEMENT 3 participants 3 messages Oct 6, 2014 – Nov 1, 2014 Device tree bindings

lore
Share & tools
lore

Gregory CLEMENT

author
#0 lore raw

PATCH 1/1 ARM: mvebu: Fix the Aurora L2 cache node with the required cache-unified property

The L2 cache controller on the Armada 370 and Armada XP SoCs is a unified cache. Moreover, the Aurora cache controller is compatible with the L2x0 cache controller: the "cache-unified" property is required by its binding.

This patch fixes the Aurora L2 cache node for the Armada 370 and Armada XP SoCs by adding this property.

Reported-by Sebastian Hesselbarth <[email protected]>
Signed-off-by Gregory CLEMENT <[email protected]>
---
 arch/arm/boot/dts/armada-370.dtsi | 1 +
 arch/arm/boot/dts/armada-xp.dtsi  | 1 +
 2 files changed, 2 insertions(+)
arch/arm/boot/dts/armada-370.dtsi +1 -0
index 21b588b6f6bd..095ae7e1d8c7 100644
@@ -95,6 +95,7 @@
 				compatible = "marvell,aurora-outer-cache";
 				reg = <0x08000 0x1000>;
 				cache-id-part = <0x100>;
+				cache-unified;
 				wt-override;
 			};
arch/arm/boot/dts/armada-xp.dtsi +1 -0
index bff9f6c18db1..e1707133ee18 100644
@@ -39,6 +39,7 @@
 				compatible = "marvell,aurora-system-cache";
 				reg = <0x08000 0x1000>;
 				cache-id-part = <0x100>;
+				cache-unified;
 				wt-override;
 			};
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 ARM: mvebu: Fix the Aurora L2 cache node with the required cache-unified property

Quoted 10 lines · Gregory CLEMENT

The L2 cache controller on the Armada 370 and Armada XP SoCs is a unified cache. Moreover, the Aurora cache controller is compatible with the L2x0 cache controller: the "cache-unified" property is required by its binding.

This patch fixes the Aurora L2 cache node for the Armada 370 and Armada XP SoCs by adding this property.

Reported-by Sebastian Hesselbarth <[email protected]>
Signed-off-by Gregory CLEMENT <[email protected]>
Acked-by Sebastian Hesselbarth <[email protected]>

Thanks for the quick patch!

Sebastian

Quoted 30 lines
---
  arch/arm/boot/dts/armada-370.dtsi | 1 +
  arch/arm/boot/dts/armada-xp.dtsi  | 1 +
  2 files changed, 2 insertions(+)
arch/arm/boot/dts/armada-370.dtsi +1 -0
index 21b588b6f6bd..095ae7e1d8c7 100644
@@ -95,6 +95,7 @@
  				compatible = "marvell,aurora-outer-cache";
  				reg = <0x08000 0x1000>;
  				cache-id-part = <0x100>;
+				cache-unified;
  				wt-override;
  			};
arch/arm/boot/dts/armada-xp.dtsi +1 -0
index bff9f6c18db1..e1707133ee18 100644
@@ -39,6 +39,7 @@
  				compatible = "marvell,aurora-system-cache";
  				reg = <0x08000 0x1000>;
  				cache-id-part = <0x100>;
+				cache-unified;
  				wt-override;
  			};
Applied
#2 lore raw

PATCH 1/1 ARM: mvebu: Fix the Aurora L2 cache node with the required cache-unified property

In reply to Gregory CLEMENT
Quoted 8 lines · Gregory CLEMENT

The L2 cache controller on the Armada 370 and Armada XP SoCs is a unified cache. Moreover, the Aurora cache controller is compatible with the L2x0 cache controller: the "cache-unified" property is required by its binding.

This patch fixes the Aurora L2 cache node for the Armada 370 and Armada XP SoCs by adding this property.

Quoted 6 lines
Reported-by Sebastian Hesselbarth <[email protected]>
Signed-off-by Gregory CLEMENT <[email protected]>
---
 arch/arm/boot/dts/armada-370.dtsi | 1 +
 arch/arm/boot/dts/armada-xp.dtsi  | 1 +
 2 files changed, 2 insertions(+)

Applied to mvebu/dt with Sebastian's Ack.

thx,

Jason.