From ad4afe9b043bd2ab9482c677f29c6d74e250d5e2 Mon Sep 17 00:00:00 2001 From: David Hendricks Date: Wed, 28 Aug 2013 12:39:11 -0700 Subject: [PATCH] armv7: Fix dcache writethrough policy handling The "bufferable" bit was erroneously set for the writethrough policy making it the same as writeback. (credit to jwerner for pointing this out) Signed-off-by: David Hendricks Change-Id: I567d57f0e522cb4b82988894ba9b4638642bf8db Reviewed-on: https://chromium-review.googlesource.com/167323 Reviewed-by: Julius Werner Tested-by: David Hendricks Tested-by: ron minnich Commit-Queue: David Hendricks (cherry picked from commit 36cf13839604c349692865475f3011afd08965b4) Signed-off-by: Isaac Christensen Reviewed-on: http://review.coreboot.org/6515 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan --- src/arch/armv7/mmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/arch/armv7/mmu.c b/src/arch/armv7/mmu.c index 17ad8b73ed..cc915a6a33 100644 --- a/src/arch/armv7/mmu.c +++ b/src/arch/armv7/mmu.c @@ -91,7 +91,7 @@ void mmu_config_range(unsigned long start_mb, unsigned long size_mb, str = "writeback"; break; case DCACHE_WRITETHROUGH: - attr = (0x3 << 10) | (1 << 3) | (1 << 2) | 0x2; + attr = (0x3 << 10) | (1 << 3) | 0x2; str = "writethrough"; break; default: