[coreboot-gerrit] Change in coreboot[master]: nb/intel/pineview/raminit: Remove very long delays

Arthur Heymans (Code Review) gerrit at coreboot.org
Sun Apr 30 12:49:48 CEST 2017


Arthur Heymans has uploaded a new change for review. ( https://review.coreboot.org/19505 )

Change subject: nb/intel/pineview/raminit: Remove very long delays
......................................................................

nb/intel/pineview/raminit: Remove very long delays

TESTED on d510mo, boots fine without.

Change-Id: If089d6677fe95b086eeb00540acfbb66fa2e1c47
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/intel/pineview/raminit.c
1 file changed, 0 insertions(+), 5 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/05/19505/1

diff --git a/src/northbridge/intel/pineview/raminit.c b/src/northbridge/intel/pineview/raminit.c
index cf6cbb9..9d80a7e 100644
--- a/src/northbridge/intel/pineview/raminit.c
+++ b/src/northbridge/intel/pineview/raminit.c
@@ -2565,11 +2565,8 @@
 
 	/* Enable HPET */
 	enable_hpet();
-	hpet_udelay(300000);
 
 	MCHBAR16(0xc1c) = MCHBAR16(0xc1c) | (1 << 15);
-
-	hpet_udelay(100000);
 
 	sdram_clk_crossing(&si);
 
@@ -2586,8 +2583,6 @@
 		sdram_dlltiming(&si);
 		PRINTK_DEBUG("Done dlltiming\n");
 	}
-
-	hpet_udelay(200000);
 
 	if (si.boot_path != BOOT_PATH_RESET) {
 		sdram_rcomp(&si);

-- 
To view, visit https://review.coreboot.org/19505
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If089d6677fe95b086eeb00540acfbb66fa2e1c47
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>



More information about the coreboot-gerrit mailing list