Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45922 )
Change subject: nb/intel/ironlake: Clean up cosmetics of `fill_postcar_frame` ......................................................................
nb/intel/ironlake: Clean up cosmetics of `fill_postcar_frame`
Change-Id: Ic387d2783692344b7204899b0729998852c72f33 Signed-off-by: Angel Pons th3fanbus@gmail.com --- M src/northbridge/intel/ironlake/memmap.c 1 file changed, 2 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/45922/1
diff --git a/src/northbridge/intel/ironlake/memmap.c b/src/northbridge/intel/ironlake/memmap.c index 4ec8e3a..6771a20 100644 --- a/src/northbridge/intel/ironlake/memmap.c +++ b/src/northbridge/intel/ironlake/memmap.c @@ -35,11 +35,10 @@
void fill_postcar_frame(struct postcar_frame *pcf) { - uintptr_t top_of_ram; + uintptr_t top_of_ram = ALIGN_DOWN((uintptr_t)cbmem_top(), 8 * MiB);
/* Cache 8 MiB below the top of RAM */ - top_of_ram = ALIGN_DOWN((uintptr_t)cbmem_top(), 8*MiB); - postcar_frame_add_mtrr(pcf, top_of_ram - 8*MiB, 8*MiB, MTRR_TYPE_WRBACK); + postcar_frame_add_mtrr(pcf, top_of_ram - 8 * MiB, 8 * MiB, MTRR_TYPE_WRBACK);
/* Cache TSEG region */ postcar_enable_tseg_cache(pcf);