summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Hendricks <dhendrix@chromium.org>2013-08-28 12:39:11 -0700
committerIsaac Christensen <isaac.christensen@se-eng.com>2014-08-07 23:55:56 +0200
commitad4afe9b043bd2ab9482c677f29c6d74e250d5e2 (patch)
tree24dd9ab076a6970035df5771de70887dc1dc895b
parent71e1c8303b1c5e805f32eda5a8b5a18f9f354399 (diff)
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 <dhendrix@chromium.org> Change-Id: I567d57f0e522cb4b82988894ba9b4638642bf8db Reviewed-on: https://chromium-review.googlesource.com/167323 Reviewed-by: Julius Werner <jwerner@chromium.org> Tested-by: David Hendricks <dhendrix@chromium.org> Tested-by: ron minnich <rminnich@chromium.org> Commit-Queue: David Hendricks <dhendrix@chromium.org> (cherry picked from commit 36cf13839604c349692865475f3011afd08965b4) Signed-off-by: Isaac Christensen <isaac.christensen@se-eng.com> Reviewed-on: http://review.coreboot.org/6515 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
-rw-r--r--src/arch/armv7/mmu.c2
1 files changed, 1 insertions, 1 deletions
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: