On Thu, Feb 14, 2008 at 8:19 PM, Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</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;">
Ron: I'm stealing your ack although the patch has slightly changed<br>
function naming.<br>
Corey/Tom: Does this name match what you had in mind?<br>
<div class="Ih2E3d"><br>
Factor out Geode LX VPCI device disabling into a separate function which<br>
consumes one device at a time. This helps avoid array handling in the<br>
dts and allows us to use generic disabling infrastructure.<br>
<br>
Signed-off-by: Carl-Daniel Hailfinger <<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>><br>
</div><div class="Ih2E3d">Acked-by: Ronald G. Minnich <<a href="mailto:rminnich@gmail.com">rminnich@gmail.com</a>><br>
<br>
</div><div class="Ih2E3d">Index: LinuxBIOSv3-unwantedvpci/southbridge/amd/cs5536/cs5536.c<br>
===================================================================<br>
--- LinuxBIOSv3-unwantedvpci/southbridge/amd/cs5536/cs5536.c    (Revision 599)<br>
+++ LinuxBIOSv3-unwantedvpci/southbridge/amd/cs5536/cs5536.c    (Arbeitskopie)<br>
</div>@@ -570,6 +570,15 @@<br>
 }<br>
<br>
<br>
+static void hide_vpci(u32 vpci_devid)<br>
+{<br>
+       /* Hide unwanted virtual PCI device. */<br>
+       printk(BIOS_DEBUG, "Hiding VPCI device: 0x%08X\n",<br>
+               vpci_devid);<br>
+       outl(vpci_devid + 0x7C, 0xCF8);<br>
+       outl(0xDEADBEEF, 0xCFC);<br>
+}<br>
+<br>
 /**<br>
  * TODO.<br>
  *<br>
@@ -608,16 +617,6 @@<br>
<div><div></div><div class="Wj3C7c">        if (sb->enable_ide)<br>
                ide_init(dev);<br>
<br>
-#warning Add back in unwanted VPCI support<br>
-#if 0<br>
-       /* Disable unwanted virtual PCI devices. */<br>
-       for (i = 0; (i < MAX_UNWANTED_VPCI) && (0 != sb->unwanted_vpci[i]); i++) {<br>
-               printk(BIOS_DEBUG, "Disabling VPCI device: 0x%08X\n",<br>
-                      sb->unwanted_vpci[i]);<br>
-               outl(sb->unwanted_vpci[i] + 0x7C, 0xCF8);<br>
-               outl(0xDEADBEEF, 0xCFC);<br>
-       }<br>
-#endif<br>
        printk(BIOS_SPEW, "cs5536: %s() Exit\n", __FUNCTION__);<br>
 }<br>
<br>
<br>
<br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
<br>
</div></div></blockquote></div><br><br><br>Yeah, that works<br>Acked-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>