<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
On 6/4/10 6:40 PM, Stefan Reinauer wrote:
<blockquote cite="mid:4C092C7F.5030302@coresystems.de" type="cite">
<pre wrap="">On 6/4/10 6:37 PM, Patrick Georgi wrote:
</pre>
<blockquote type="cite">
<pre wrap="">Am Freitag, den 04.06.2010, 16:25 +0100 schrieb Edwin Beasant:
</pre>
<blockquote type="cite">
<pre wrap="">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 <a class="moz-txt-link-abbreviated" href="mailto:edwin_beasant@virtensys.com">edwin_beasant@virtensys.com</a>
</pre>
</blockquote>
<pre wrap="">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 <a class="moz-txt-link-rfc2396E" href="mailto:patrick.georgi@coresystems.de"><patrick.georgi@coresystems.de></a>
</pre>
</blockquote>
<pre wrap="">
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 <a class="moz-txt-link-rfc2396E" href="mailto:edwin_beasant@virtensys.com"><edwin_beasant@virtensys.com></a>
Signed-off-by: Stefan Reinauer <a class="moz-txt-link-rfc2396E" href="mailto:stepan@coresystems.de"><stepan@coresystems.de></a>
</pre>
</blockquote>
<font size="-1"><font face="Helvetica, Arial, sans-serif">Updated patch.</font></font><br>
</body>
</html>