[coreboot] [PATCH] libpayload: Fix build when both USB and PS/2 keyboard support is disabled
Ulf Jordan
jordan at chalmers.se
Sun Feb 22 10:07:41 CET 2009
Hello Mart!
On Sun, 22 Feb 2009, Mart Raudsepp wrote:
> libpayload: Fix build when both USB and PS/2 keyboard support is disabled
>
> libpayload uses -Werror for some reason right now, and the
> variable 'c' in curses_getchar is only used if CONFIG_USB_HID
> or CONFIG_PC_KEYBOARD is defined, giving an unused variable
> warning that gets promoted to an error.
> So wrap the variable declaration around appropriate #ifdef's
The variable c is also used in the serial console input code just below
the PS/2 one, so to trigger the error all of CONFIG_USB_HID,
CONFIG_PC_KEYBOARD, and CONFIG_SERIAL_CONSOLE needs to be undefined. I
guess we won't get much input to libpayload then. The patch in current
form will instead throw an error if CONFIG_SERIAL_CONSOLE is defined while
CONFIG_USB_HID and CONFIG_PC_KEYBOARD are not, i.e. for those who build
only for serial console.
I propose to include "|| defined(CONFIG_SERIAL_CONSOLE)" in the #if.
With that change included the patch is
Acked-by: Ulf Jordan <jordan at chalmers.se>
/ulf
More information about the coreboot
mailing list