[coreboot-gerrit] Change in coreboot[master]: nb/intel/ironlake: Do not re-read ME UMA size

[Angel Pons (Code Review)]

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45363 )

Change subject: nb/intel/ironlake: Do not re-read ME UMA size
......................................................................

nb/intel/ironlake: Do not re-read ME UMA size

It has been read twice already, so don't read it a third time.

Change-Id: I56ec3a10246f6ebe8074e7b8c164bda6b90eee87
Signed-off-by: Angel Pons 
---
M src/northbridge/intel/ironlake/raminit.c
1 file changed, 1 insertion(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/63/45363/1

diff --git a/src/northbridge/intel/ironlake/raminit.c b/src/northbridge/intel/ironlake/raminit.c
index 426ef87..44b6a71 100644
--- a/src/northbridge/intel/ironlake/raminit.c
+++ b/src/northbridge/intel/ironlake/raminit.c
@@ -1805,15 +1805,10 @@

static void setup_heci_uma(struct raminfo *info)
 {
-	u32 reg44;
-
-	reg44 = pci_read_config32(HECIDEV, 0x44);	// = 0x80010020
-	info->memory_reserved_for_heci_mb = 0;
    info->heci_uma_addr = 0;
-	if (!((reg44 & 0x10000) && !(pci_read_config32(HECIDEV, 0x40) & 0x20)))
+	if (!info->memory_reserved_for_heci_mb && !(pci_read_config32(HECIDEV, 0x40) & 0x20))
    	return;

-	info->memory_reserved_for_heci_mb = reg44 & 0x3f;
    info->heci_uma_addr =
        ((u64)
         ((((u64) pci_read_config16(NORTHBRIDGE, TOM)) << 6) -