[coreboot] build service results for r6516

repository service svn at coreboot.org
Tue Apr 19 04:55:37 CEST 2011

Dear coreboot readers!

This is the automatic build system of coreboot.

The developer "stepan" checked in revision 6516 to
the coreboot repository. This caused the following 

Change Log:
Recently the 3 projects using the new AMD reference code have been
failing the check for globals (or statics) in romstage. This causes
ASRock E350M1, AMD Inagua, and AMD Persimmon builds to fail with the
message "Do not use global variables in romstage". The message is
working as intended. It is detecting data declared as 'static' when
'static const' was intended. The code executes correctly because it
never tries to modify the data.

To make reference code updates easy, it is probably best to avoid
modifying the AMD provided code if possible. The following change
bypasses the "Do not use global variables in romstage" check for
the AMD reference code only.

Signed-off-by: Scott Duplichan <scott at notabs.org>
Acked-by: Stefan Reinauer <stefan.reinauer at coreboot.org>

Build Log:
Compilation of amd:inagua has been fixed
Compilation of amd:persimmon has been fixed
Compilation of asrock:e350m1 has been fixed

If something broke during this checkin please be a pain 
in stepan's neck until the issue is fixed.

If this issue is not fixed within 24h the revision should 
be backed out.

   Best regards,
     coreboot automatic build system

More information about the coreboot mailing list