[coreboot] Fwd: [RFC] Here we go... the SLOF biosemu forcoreboot-v3

Pattrick Hueper phueper at hueper.net
Wed Jan 7 09:49:57 CET 2009


Hi,

On Wed, Jan 7, 2009 at 12:51 AM, Peter Stuge <peter at stuge.se> wrote:
> Hi, I found what seems to be a bug in the current yabel/biosemu.c.
> You seem to have addressed it in a patch already, though..
>
>
> Pattrick Hueper wrote:
> ..
>> +u32
>> +biosemu(u8 *biosmem, u32 biosmem_size, struct device * dev)
>> +{
> ..
>> +     // setup default Interrupt Vectors
>> +     // some expansion ROMs seem to check for these addresses..
>> +     // each handler is only an IRET (0xCF) instruction
>> +     // ROM BIOS Int 10 Handler F000:F065
>> +     my_wrl(0x10 * 4, 0xf000f065);
>> +     my_wrb(0x000ff065, 0xcf);
>
> Has this gotten some feedback? The fix is correct!
>

This code is currently in svn, isnt it? It should be correct, all it
does is let the INT10 int handler point to F000:F065 (the default
location) and the code at that location is 0xCF (IRET).

Patty




More information about the coreboot mailing list