On Fri, Feb 15, 2008 at 1:23 PM, ron minnich <<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
This turned into a pretty big cleanup. However, it reduced (by a small<br>
amount) the amount of code, and complexity, and I like it.<br>
<br>
Comments welcome. This one is signed off, though there is a bit of<br>
todo left as noted in the patch.<br>
<br>
boots on qemu and alix1c.<br>
<br>
thanks<br>
<font color="#888888"><br>
ron</font></blockquote><div><br>Well, since everyone else seems to be asleep, I'll have a peak.<br><br><blockquote style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;" class="gmail_quote">
--- southbridge/intel/i82371eb/i82371eb.c    (revision 600)<br>+++ southbridge/intel/i82371eb/i82371eb.c    (working copy)<br>@@ -83,7 +83,9 @@<br> }<br> <br> /* You can override or extend each operation as needed for the device. */<br>
-static struct device_operations i82371eb_isa_ops_dev = {<br>+struct device_operations i82371eb_isa = {<br>+    .id = {.type = DEVICE_ID_PCI,<br>+        .u = {.pci = {.vendor = 0x8086,.device = 0x7000}}},</blockquote><div>
<br>sweet, but why no longer static?<br><div><br>Other than that, this looks good, but I'd hold off on committing a little bit, to give someone a chance to nack.<br><br>Acked-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>
</div></div></div></div>