<br><br><div class="gmail_quote">On Mon, Oct 19, 2009 at 3:39 PM, Peter Stuge <span dir="ltr"><<a href="mailto:peter@stuge.se">peter@stuge.se</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;">
Myles Watson wrote:<br>
> +config GENERATE_ACPI_TABLES<br>
>       bool<br>
> +     default y if HAVE_ACPI_TABLES<br>
> +     default n<br>
<br>
Can it be simply:<br>
<br>
default HAVE_ACPI_TABLES<br></blockquote><div>It looks like it could.  We have a mix of the two styles right now.<br><br>src/console/Kconfig:    default MAXIMUM_CONSOLE_LOGLEVEL_8<br>src/console/Kconfig:    default DEFAULT_CONSOLE_LOGLEVEL_8<br>
src/mainboard/Kconfig:    default VENDOR_EMULATION<br>src/mainboard/Kconfig:    default COREBOOT_ROMSIZE_KB_128 if BOARD_ROMSIZE_KB_128<br>src/mainboard/Kconfig:    default COREBOOT_ROMSIZE_KB_256 if BOARD_ROMSIZE_KB_256<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
> +++ svn/src/northbridge/amd/amdfam10/Kconfig<br>
> @@ -21,11 +21,35 @@ config NORTHBRIDGE_AMD_AMDFAM10<br>
>       bool<br>
>       select HAVE_HIGH_TABLES<br>
>       select HYPERTRANSPORT_PLUGIN_SUPPORT<br>
> -     select HT3_SUPPORT<br>
><br>
>  config AGP_APERTURE_SIZE<br>
>       hex<br>
>       default 0x4000000<br>
>       depends on NORTHBRIDGE_AMD_AMDFAM10<br>
><br>
> +config HT3_SUPPORT<br>
> +     bool<br>
> +     default y<br>
> +     depends on NORTHBRIDGE_AMD_AMDFAM10<br>
<br>
Is this good?<br></blockquote><div>It has to be defined somewhere.  The alternative is to put config HT3_SUPPORT somewhere else, then use select in this file.  Since there are no other devices with HT3_SUPPORT at this time, I think this is the place to put it.<br>
<br>If you use select and HT3_SUPPORT isn't defined somewhere else, it silently fails. <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;">

I find having single line selects instead of repeating the horribly<br>
long NB option name over and over very appealing</blockquote><div>Me too.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> but I don't know<br>

enough about Kconfig to say if it makes a difference?<br></blockquote><div>In this case it does.<br><br>Thanks,<br>Myles<br></div></div><br>