commit 9b04f56d4a40295110c4c964b47df0722836e08f
author Angel Pons <th3fanbus@gmail.com> Wed Jan 20 12:33:18 2021 +0100
committer Patrick Georgi <pgeorgi@google.com> Mon Feb 01 08:56:08 2021 +0000
tree c09614f73dbb2568670d449057f814ba07b11214
parent cffc938934a9ee3b344ef594874e55f131bea77b

nb/intel/i945: Drop casts from DEFAULT_{MCH,DMI}BAR

They aren't necessary. Removing them changes the binary because the
corresponding access macros no longer perform pointer arithmetics.

Change-Id: I9723a00b58ee35befdce6a3a51aa2b1fce8efa80
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49745
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/northbridge/intel/i945/memmap.h

diff --git a/src/northbridge/intel/i945/memmap.h b/src/northbridge/intel/i945/memmap.h
index a890445..fcda447 100644
--- a/src/northbridge/intel/i945/memmap.h
+++ b/src/northbridge/intel/i945/memmap.h
@@ -5,13 +5,8 @@
 
 /* Northbridge BARs */
 #define DEFAULT_X60BAR		0xfed13000
-#ifndef __ACPI__
-#define DEFAULT_MCHBAR		((u8 *)0xfed14000)	/* 16 KB */
-#define DEFAULT_DMIBAR		((u8 *)0xfed18000)	/* 4 KB */
-#else
 #define DEFAULT_MCHBAR		0xfed14000	/* 16 KB */
 #define DEFAULT_DMIBAR		0xfed18000	/* 4 KB */
-#endif
 #define DEFAULT_EPBAR		0xfed19000	/* 4 KB */
 
 #endif