diff options
author | Rudolf Marek <r.marek@assembler.cz> | 2011-06-29 23:47:20 +0200 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2011-06-30 19:16:37 +0200 |
commit | 23b215272d4d136efcac5f3b5712d2a1d76a91d9 (patch) | |
tree | 369c83ea3e4fc6fddae4591caa3cee50e21c8e46 /src/northbridge | |
parent | 0d21cd36b781076409d370935faf6081d970f644 (diff) | |
download | coreboot-23b215272d4d136efcac5f3b5712d2a1d76a91d9.tar.xz |
Improve VIA K8M890 HT settings. Use recommended settings for ROMSIP and
for the transmit clock driving control. Unfortunately this is not enough
to make the HT1000 work reliably, therefore blacklist this for now in CPU
HT code. If ever anyone figure out what is wrong, it could be removed. The
downgrading now makes the board work on HT800, which is certainly better than
not at all with a HT1000 CPU.
Change-Id: I949bfd9b0b48ee12bd0234c2fb1deaaa773bd235
Signed-off-by: Rudolf Marek <r.marek@assembler.cz>
Reviewed-on: http://review.coreboot.org/68
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/northbridge')
-rw-r--r-- | src/northbridge/amd/amdk8/incoherent_ht.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/northbridge/amd/amdk8/incoherent_ht.c b/src/northbridge/amd/amdk8/incoherent_ht.c index 4b5209267d..4d0d0259f6 100644 --- a/src/northbridge/amd/amdk8/incoherent_ht.c +++ b/src/northbridge/amd/amdk8/incoherent_ht.c @@ -148,8 +148,11 @@ static uint16_t ht_read_freq_cap(device_t dev, uint8_t pos) } printk(BIOS_SPEW, "pos=0x%x, filtered freq_cap=0x%x\n", pos, freq_cap); - //printk(BIOS_SPEW, "capping to 800/600/400/200 MHz\n"); - //freq_cap &= 0x3f; + + #if CONFIG_SOUTHBRIDGE_VIA_K8M890 == 1 + freq_cap &= 0x3f; + printk(BIOS_INFO, "Limiting HT to 800/600/400/200 MHz until K8M890 HT1000 is fixed.\n"); + #endif return freq_cap; } |