[coreboot] [commit] r5270 - trunk/src/southbridge/intel/i82801dx

Joseph Smith joe at settoplinux.org
Tue Mar 23 12:29:32 CET 2010

On Tue, 23 Mar 2010 12:20:14 +0100, Stefan Reinauer <stepan at coresystems.de>
> On 3/23/10 12:14 PM, Joseph Smith wrote:
>> Good question. It does two checks. First to see if HPET is enabled, and
>> second to verify the hpet base address bits match the input value
>> If either one of these are false it leaves with a err message, makes
>> HPET is disabled and continues on. Maybe printk_err() is a bit harsh
>> because it doesn't actually die() if there is an error, but I wanted to
>> make sure it is _KNOWN_ in the bootlog if there was an issue enabling
>> Does that make sense?
> I think it does, since we're enabling an undocumented feature here.
> printk_err is fine.. die() is more along the line of printk_emerg
The Linux Kernel also does this check. Of course I added the very hpet base
address bits, as an extra safety check. 

See arch/x86/kernel/quirks.c of the Linux Kernel for more info.

Joseph Smith

More information about the coreboot mailing list