<br><br><div class="gmail_quote">On Sun, Oct 18, 2009 at 7:47 AM,  <span dir="ltr"><<a href="mailto:svn@coreboot.org">svn@coreboot.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Author: uwe<br>
Date: 2009-10-18 15:47:30 +0200 (Sun, 18 Oct 2009)<br>
New Revision: 4806<br>
<br>
Modified:<br>
   trunk/coreboot-v2/src/cpu/amd/model_10xxx/Kconfig<br>
   trunk/coreboot-v2/src/cpu/amd/model_fxx/Kconfig<br>
   trunk/coreboot-v2/src/cpu/amd/socket_F_1207/Kconfig<br>
   trunk/coreboot-v2/src/mainboard/amd/serengeti_cheetah_fam10/Kconfig<br>
   trunk/coreboot-v2/src/mainboard/kontron/kt690/Kconfig<br>
   trunk/coreboot-v2/src/mainboard/supermicro/x6dhe_g/Kconfig<br>
   trunk/coreboot-v2/src/mainboard/supermicro/x6dhe_g2/Kconfig<br>
   trunk/coreboot-v2/src/mainboard/tyan/s2912_fam10/Kconfig<br>
   trunk/coreboot-v2/src/northbridge/amd/amdfam10/Kconfig<br>
   trunk/coreboot-v2/src/northbridge/amd/amdk8/Kconfig<br>
   trunk/coreboot-v2/src/northbridge/amd/lx/Kconfig<br>
   trunk/coreboot-v2/src/southbridge/nvidia/ck804/Kconfig<br>
Log:<br>
Simplify Kconfig files by using "select" where possible (trivial).<br></blockquote><div>Care needs to be taken that select doesn't break things.  It has no effect and fails silently when the CONFIG_* isn't defined. <br>
</div><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Signed-off-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>><br>
Acked-by: Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>><br>
<br>
Modified: trunk/coreboot-v2/src/northbridge/amd/amdfam10/Kconfig<br>
===================================================================<br>
--- trunk/coreboot-v2/src/northbridge/amd/amdfam10/Kconfig      2009-10-18 13:42:50 UTC (rev 4805)<br>
+++ trunk/coreboot-v2/src/northbridge/amd/amdfam10/Kconfig      2009-10-18 13:47:30 UTC (rev 4806)<br>
@@ -20,20 +20,12 @@<br>
 config NORTHBRIDGE_AMD_AMDFAM10<br>
        bool<br>
        select HAVE_HIGH_TABLES<br>
+       select HYPERTRANSPORT_PLUGIN_SUPPORT<br>
+       select HT3_SUPPORT<br>
<br></blockquote><div>HT3_SUPPORT is no longer defined.  I think this is the right place to put it since it is the only northbridge that supports HT3.  I guess it could go in src/devices/Kconfig by HYPERTRANSPORT_PLUGIN_SUPPORT, but the code there doesn't use it.  Either way it needs to be defined somewhere.<br>
<br>Thanks,<br>Myles<br></div></div><br>