Did it the old way, so that i can understand what i'm sending so that i can check what i'm doing is correct, at least to my opinion :-)

Marc


On Mon, 2011-01-17 at 15:40 -0500, Kevin O'Connor wrote:
On Mon, Jan 17, 2011 at 09:13:47PM +0100, Marc Bertens wrote:
> Kevin,
> 
> This is the final patch for Kconfig for SeaBIOS. 
> 
> Tested-by Marc Bertens <mbertens@xs4all.nl>
> Signed-off Marc Bertens <mbertens@xs4all.nl>
> 
> Its now all in one big patch, it has now one config.h file. This should
> be straight forward.

Thanks.  Unfortunately, it's still too big for the mailing list.

Can you post this in two patches - one that adds the tools/kconfig/
files, and one that modifies the SeaBIOS files (Makefile, src/*).  I
can put the tools/kconfig/ part on the web, and we can discuss the
seabios parts on the mailing list.

Some comments on that patch:

The patch seems to be missing src/Kconfig.

[...]
> +#elif CONFIG_NAME_QEMU
> +/*	the Q-Emu name was selected */
> +#define CONFIG_APPNAME  "QEMU"
> +#define CONFIG_CPUNAME8 "QEMUCPU "
> +#define CONFIG_APPNAME6 "QEMU  "
> +#define CONFIG_APPNAME4 "QEMU"
> +
> +#elif CONFIG_NAME_COREBOOT
> +/*	the Coreboot name was selected */
> +#define CONFIG_APPNAME  "Coreboot"
> +#define CONFIG_CPUNAME8 "CRBTCPU "
> +#define CONFIG_APPNAME6 "COREBT"
> +#define CONFIG_APPNAME4 "CRBT"

This adds new names - which is a topic in itself.  Lets have the
first kconfig patch just add in the basic capabilities, and then we
can add new features on top of it.  I suggest leaving the names hard
coded in config.h for now - setting them via kconfig can be a follow
up patch.

Thanks for working on this.
-Kevin