[coreboot-gerrit] Change in coreboot[master]: soc/intel/broadwell: Use Haswell memmap.h in iomap.h

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Arthur Heymans, Patrick Rudolph.
Hello Nico Huber, Arthur Heymans, Patrick Rudolph,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/55483

to review the following change.

Change subject: soc/intel/broadwell: Use Haswell memmap.h in iomap.h
......................................................................

soc/intel/broadwell: Use Haswell memmap.h in iomap.h

Include Haswell memmap.h from Broadwell iomap.h to deduplicate identical
definitions. This also prevents the definitions from falling out of sync
while the unification process is ongoing.

Tested with BUILD_TIMELESS=1, Purism Librem 13 v1 remains identical.

Change-Id: I850e5521effba3818f4e2a13b94281bf07857d50
Signed-off-by: Angel Pons 
---
M src/soc/intel/broadwell/include/soc/iomap.h
1 file changed, 1 insertion(+), 17 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/83/55483/1

diff --git a/src/soc/intel/broadwell/include/soc/iomap.h b/src/soc/intel/broadwell/include/soc/iomap.h
index 4b4e3d9..3a2cafe 100644
--- a/src/soc/intel/broadwell/include/soc/iomap.h
+++ b/src/soc/intel/broadwell/include/soc/iomap.h
@@ -3,26 +3,10 @@
 #ifndef _BROADWELL_IOMAP_H_
 #define _BROADWELL_IOMAP_H_

-#define MCH_BASE_SIZE		0x8000
-
-#define DMI_BASE_SIZE		0x1000
-
-#define EP_BASE_SIZE		0x1000
-
-#define EDRAM_BASE_ADDRESS	0xfed80000
-#define EDRAM_BASE_SIZE		0x4000
-
-#define GDXC_BASE_ADDRESS	0xfed84000
-#define GDXC_BASE_SIZE		0x1000
+#include <northbridge/intel/haswell/memmap.h>

#define HPET_BASE_ADDRESS	0xfed00000

-#define GFXVT_BASE_ADDRESS	0xfed90000ULL
-#define GFXVT_BASE_SIZE		0x1000
-
-#define VTVC0_BASE_ADDRESS	0xfed91000ULL
-#define VTVC0_BASE_SIZE		0x1000
-
 #define ACPI_BASE_ADDRESS	0x1000
 #define ACPI_BASE_SIZE		0x100