[coreboot] [PATCH] libpayload: fix VGA and serial IO breakage

Stefan Reinauer stepan at coresystems.de
Mon Aug 11 09:13:41 CEST 2008


Ulf Jordan wrote:
> Hello,
>
> Since r3482 I have not had any VGA or serial IO from libpayload. The
> reason seems to be that the CONFIG_ macros generated by kconfig are
> never made available to the C source files during compilation. Before
> r3482 the CONFIG_ macros were obtained by including autoconf.h in
> libpayload.h.
>
> In the attached patch, config.h from the new kconfig is included in
> libpayload.h, but I'm not quite sure if this is the appropriate place.
>
> Compile tested against libpayload+coreinfo svn HEAD. Runtime tested
> with libpayload+coreinfo+coreboot-v3 in QEMU.
Not sure it matters; But when including libpayload.h in other projects,
you will pull in these projects' config.h instead, which might or might
not be there, or have a different meaning.

Including "config.h" is principally not a bad idea, though, as we might
start to make function prototypes available on a CONFIG_XXX basis, too.
Not just the functions themselfes.

Otoh, we might also add -imacros config.h or similar to the CFLAGS/INCLUDES

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866





More information about the coreboot mailing list