summaryrefslogtreecommitdiff
path: root/src/northbridge/intel/pineview
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2019-10-02 14:57:50 +0200
committerArthur Heymans <arthur@aheymans.xyz>2019-10-06 22:11:32 +0000
commitc73c92368f0c35a522ce935b9635a1ad19ad3eb7 (patch)
tree543b3df574d6bad7878530951a62e0418c27ac07 /src/northbridge/intel/pineview
parent1609a2038155da3ac4cdac8a6b1af337d0f05051 (diff)
downloadcoreboot-c73c92368f0c35a522ce935b9635a1ad19ad3eb7.tar.xz
sb/intel/nm10: Fix enabling HPET
RCBA_HPTC needs to be read back to consistently enable HPET. This ought to fix raminit failing sometimes and SeaBIOS endlessly waiting for user input. TESTED on Intel D510MO, Fixes SeaBIOS waiting for input, without a timeout. Change-Id: I20a25fd97cd09fedb70469262c64d8d3828bb684 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/35758 Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/northbridge/intel/pineview')
-rw-r--r--src/northbridge/intel/pineview/raminit.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/src/northbridge/intel/pineview/raminit.c b/src/northbridge/intel/pineview/raminit.c
index 5cece4147f..7f59d9197b 100644
--- a/src/northbridge/intel/pineview/raminit.c
+++ b/src/northbridge/intel/pineview/raminit.c
@@ -552,10 +552,12 @@ static void sdram_detect_ram_speed(struct sysinfo *s)
static void enable_hpet(void)
{
u32 reg32;
- reg32 = RCBA32(0x3404);
+ reg32 = RCBA32(HPTC);
reg32 &= ~0x3;
reg32 |= (1 << 7);
- RCBA32(0x3404) = reg32;
+ RCBA32(HPTC) = reg32;
+ /* On NM10 this only works if read back */
+ RCBA32(HPTC);
HPET32(0x10) = HPET32(0x10) | 1;
}