Sven Schnelle (sv...@stackframe.org) just uploaded a new patch set to gerrit, 
which you can find at http://review.coreboot.org/75

-gerrit

commit 55aacb4b76b37451df7215ddbe3d41f4ddd82a4f
Author: Cristian Măgherușan-Stanciu <cristi.magheru...@gmail.com>
Date:   Sat Jul 2 01:03:50 2011 +0300

    added support for multiple vendors in the HPET code
    
    Change-Id: Id13709329c8cc2b3acb55b7ec78fc53c805543eb
    Signed-off-by: Cristian Măgherușan-Stanciu <cristi.magheru...@gmail.com>
---
 src/arch/x86/boot/acpi.c |   15 +++++++++++++--
 1 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/arch/x86/boot/acpi.c b/src/arch/x86/boot/acpi.c
index caf860e..8be7437 100644
--- a/src/arch/x86/boot/acpi.c
+++ b/src/arch/x86/boot/acpi.c
@@ -376,9 +376,20 @@ void acpi_create_hpet(acpi_hpet_t *hpet)
        addr->addrl = HPET_ADDR & 0xffffffff;
        addr->addrh = HPET_ADDR >> 32;
 
-       hpet->id = 0x102282a0; /* AMD! FIXME */
+       /* XXX: Add other vendors */
+#if CONFIG_VENDOR_INTEL
+       hpet->id = 0x8086a201;
+       hpet->min_tick = 0x80;
+#endif
+#if CONFIG_VENDOR_AMD
+       hpet->id = 0x102282a0;
+       hpet->min_tick = 0x1000;
+#endif
+#if CONFIG_VENDOR_VIA
+       hpet->id = 0x11068201;
+       hpet->min_tick = 0x90;
+#endif
        hpet->number = 0;
-       hpet->min_tick = 4096;
 
        header->checksum = acpi_checksum((void *)hpet, sizeof(acpi_hpet_t));
 }

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to