diff options
| author | David Hendricks <dhendrix@chromium.org> | 2013-08-28 12:39:11 -0700 |
|---|---|---|
| committer | Isaac Christensen <isaac.christensen@se-eng.com> | 2014-08-07 23:55:56 +0200 |
| commit | ad4afe9b043bd2ab9482c677f29c6d74e250d5e2 (patch) | |
| tree | 24dd9ab076a6970035df5771de70887dc1dc895b /src | |
| parent | 71e1c8303b1c5e805f32eda5a8b5a18f9f354399 (diff) | |
| download | coreboot-ad4afe9b043bd2ab9482c677f29c6d74e250d5e2.tar.xz | |
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>
Diffstat (limited to 'src')
| -rw-r--r-- | src/arch/armv7/mmu.c | 2 |
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: |