[coreboot] [PATCH] Fix geode lx VSA loading

Stefan Reinauer stepan at coresystems.de
Fri Jun 4 18:40:31 CEST 2010


On 6/4/10 6:37 PM, Patrick Georgi wrote:
> Am Freitag, den 04.06.2010, 16:25 +0100 schrieb Edwin Beasant:
>   
>> This patch fixes the option rom code that was buggy when it switched
>> segment registers before restoring register values. This was breaking
>> the Geode VSA, and probably would have hurt other option roms as well.
>>
>>  
>>
>> Signed-off by: Edwin Beasant edwin_beasant at virtensys.com
>>     
> Except for the first two parts (the magic values in ecx/edx and
> whitespace), and assuming this is tested, the patch is
>
> Acked-by: Patrick Georgi <patrick.georgi at coresystems.de>
>
>
>   

I did a new version of the patch with those issues fixed (and pushing
one register instead of all), see attachment

Signed-off-by: Edwin Beasant <edwin_beasant at virtensys.com>
Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: vsa.diff
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100604/e8c75dfd/attachment.ksh>


More information about the coreboot mailing list