Skip to contentSkip to conversation

ARM: nomadik: set latencies to 8 cycles

PATCH New Linus Walleij 2 participants 2 messages Jan 4, 2016 – Jan 7, 2016

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: nomadik: set latencies to 8 cycles

The Nomadik has sporadic crashes because of these latencies, setting them to max makes the platform work nicely, so use this values for now.

These latencies were set to 2 since the Nomadik platform was merged, but I suspect they never took effect until the right size and associativity for the cache was specified in the device tree and that is why the crash comes now.

Cc stable at vger.kernel.org

---
ARM SoC folks: please apply this directly for fixes at your convenience.

---
 arch/arm/boot/dts/ste-nomadik-stn8815.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
arch/arm/boot/dts/ste-nomadik-stn8815.dtsi +3 -3
index 314f59c12162..d0c743853318 100644
@@ -25,9 +25,9 @@
 		cache-sets = <512>;
 		cache-line-size = <32>;
 		/* At full speed latency must be >=2 */
-		arm,tag-latency = <2>;
-		arm,data-latency = <2 2>;
-		arm,dirty-latency = <2>;
+		arm,tag-latency = <8>;
+		arm,data-latency = <8 8>;
+		arm,dirty-latency = <8>;
 	};
 
 	mtu0: mtu at 101e2000 {
Signature · 1 line
2.4.3
#1 lore raw

PATCH 1/1 ARM: nomadik: set latencies to 8 cycles

Quoted 9 lines · Linus Walleij

The Nomadik has sporadic crashes because of these latencies, setting them to max makes the platform work nicely, so use this values for now.

These latencies were set to 2 since the Nomadik platform was merged, but I suspect they never took effect until the right size and associativity for the cache was specified in the device tree and that is why the crash comes now.

Quoted 5 lines
Cc stable at vger.kernel.org

---
ARM SoC folks: please apply this directly for fixes at your convenience.

Applied to fixes. THanks.

-Olof