Denis Carikli (gnu...@no-log.org) just uploaded a new patch set to gerrit, 
which you can find at http://review.coreboot.org/1533

-gerrit

commit dcec334895aa187e256543bc17382d567b3d1d94
Author: Denis 'GNUtoo' Carikli <gnu...@no-log.org>
Date:   Sat Sep 22 14:52:24 2012 +0200

    Fix compilation without CONFIG_WRITE_HIGH_TABLES.
    
    Without that fix we have:
          CC         boot/hardwaremain.ramstage.o
      src/boot/hardwaremain.c: In function 'hardwaremain':
      src/boot/hardwaremain.c:136:6: error: 'cbmem_post_handling' undeclared 
(first use in this function)
      src/boot/hardwaremain.c:136:6: note: each undeclared identifier is 
reported only once for each function it appears in
      src/boot/hardwaremain.c:137:3: error: implicit declaration of function 
'cbmem_post_handling' [-Werror=implicit-function-declaration]
      cc1: all warnings being treated as errors
      make: *** [build/boot/hardwaremain.ramstage.o] Error 1
    When compiling without CONFIG_WRITE_HIGH_TABLES
    
    Change-Id: Ie45f684a6db0ab55ef469bfcef57e539ae7e994c
    Signed-off-by: Denis 'GNUtoo' Carikli <gnu...@no-log.org>
---
 src/boot/hardwaremain.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/boot/hardwaremain.c b/src/boot/hardwaremain.c
index bb7f264..099ab3a 100644
--- a/src/boot/hardwaremain.c
+++ b/src/boot/hardwaremain.c
@@ -133,8 +133,10 @@ void hardwaremain(int boot_complete)
        timestamp_add(TS_DEVICE_DONE, timestamps[5]);
        timestamp_add_now(TS_CBMEM_POST);
 
+#if CONFIG_WRITE_HIGH_TABLES
        if (cbmem_post_handling)
                cbmem_post_handling();
+#endif
 
        timestamp_add_now(TS_WRITE_TABLES);
 

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to