[coreboot] PATCH: Fix CMOS Tables support for all boards.

Stefan Reinauer stefan.reinauer at coresystems.de
Thu Jul 8 00:02:13 CEST 2010


 On 7/7/10 8:28 PM, Myles Watson wrote:
> On Wed, Jul 7, 2010 at 8:24 AM, Myles Watson <mylesgw at gmail.com> wrote:
>>> Rev 5653.
>>>
>>> Thanks,
>>> Myles
>>>
>>> Not sure why there were no errors, but:
>>>
>>>       9 src/include/pc80/mc146818rtc.h:89:26: warning: option_table.h: No
>>> such file or directory
>> That's odd.  I don't see that warning.  Does it have to do with
>> parallel compilation and insufficient dependency checking?  Which
>> boards have the warning?
> http://qa.coreboot.org/log_buildbrd.php?revision=5657&device=dbm690t&vendor=amd&num=2
>
> option_table.h doesn't get generated until way down the list (after
> the warnings)  It looks like the first board that needs this:
>
>  HOSTCC     coreboot-builds/sharedutils/options/build_opt_tbl
>
> And the rest of the build doesn't wait for it.

This sure looks like a race... question is, what do we do about it?
Force an early build of option_table.h outside of the normal dependency
system?

Why does gcc only print a warning instead of an error here? Are those
#includes not needed at all?

Stefan




More information about the coreboot mailing list