[coreboot] [PATCH] ifdef & config problems

Myles Watson mylesgw at gmail.com
Sat Oct 17 16:23:28 CEST 2009


Try to make CONFIG_ options more standard.

config_console.diff: #ifdef->#if
fan_ctl.diff: make CONFIG_HAVE_FANCTL specific to the SuperIO that supports
it
vt8237r.diff: #ifdef->#if

Config options used with #ifdef makes it so your board can break silently
when someone makes a simple change in Kconfig.

Signed-off-by: Myles Watson <mylesgw at gmail.com>

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091017/7f6bac2d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config_console.diff
Type: text/x-patch
Size: 1265 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091017/7f6bac2d/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fan_ctl.diff
Type: text/x-patch
Size: 3282 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091017/7f6bac2d/attachment-0001.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vt8237r.diff
Type: text/x-patch
Size: 5708 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091017/7f6bac2d/attachment-0002.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ifdef.diff
Type: text/x-patch
Size: 3330 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20091017/7f6bac2d/attachment-0003.diff>


More information about the coreboot mailing list