<br><br><div class="gmail_quote">On Sat, Oct 17, 2009 at 7:47 AM, Uwe Hermann <span dir="ltr"><<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</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;">
On Sat, Oct 17, 2009 at 03:25:07PM +0200, <a href="mailto:svn@coreboot.org">svn@coreboot.org</a> wrote:<br>
> Remove CONFIG_ from #defines that aren't config variables.  Trivial.<br>
<br>
Why this? Shouldn't they have been added to src/Kconfig (for example)<br>
instead?<br></blockquote><div>No.  Many of them are only mentioned in one place.  They probably should be removed completely.  I think it will improve the quality of Coreboot to limit configuration options to ones that actually do something.<br>
<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;">
<br>
> Modified: trunk/coreboot-v2/src/arch/i386/init/car.S<br>
> ===================================================================<br>
> --- trunk/coreboot-v2/src/arch/i386/init/car.S        2009-10-17 03:00:04 UTC (rev 4801)<br>
> +++ trunk/coreboot-v2/src/arch/i386/init/car.S        2009-10-17 13:25:07 UTC (rev 4802)<br>
> @@ -269,7 +269,7 @@<br>
><br>
><br>
>       /* TODO: make this a config variable */<br>
> -#if CONFIG_CARTEST<br>
> +#if CARTEST<br>
<br>
This one, for example, should likely be an option in the "Debugging"<br>
menu, maybe only visible if EXPERT is enabled (will post EXPERT patch<br>
soon).<br></blockquote><div>I think there's too little gain for something this trivial. <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;">

I personally think that _all_ options should be kconfig options (i.e.<br>
listed in a Kconfig file with some default). Whether or not they should<br>
also be visible in menuconfig is another issue and depends on the<br>
option, though.<br></blockquote><div>These aren't really options.  They really are just #ifdefs.  They were never in Options.lb.<br><br>Thanks,<br>Myles <br></div></div><br>