Stefan Reinauer ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/1753

-gerrit

commit 49a34f4d12511a957d9a26d0807174953729d887
Author: Vadim Bendebury <[email protected]>
Date:   Thu Sep 27 18:42:57 2012 -0700

    Prevent inclusion of tsc.h when not needed
    
    src/include/timestamp.h is an interface describing timestamp storage
    in coreboot. Exporting this interface is complicated by inclusion of
    tsc.h which is needed only for the API and is not used in structure
    definitions. Including this dependency only when needed fixes the
    problem.
    
    Change-Id: Ie6b1460b1dab0f5b5781cb5a9fa89a1a52aa9f17
    Signed-off-by: Vadim Bendebury <[email protected]>
---
 src/include/timestamp.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/include/timestamp.h b/src/include/timestamp.h
index 65bdf21..8bf5287 100644
--- a/src/include/timestamp.h
+++ b/src/include/timestamp.h
@@ -20,8 +20,6 @@
 #ifndef __TIMESTAMP_H__
 #define __TIMESTAMP_H__
 
-#include <cpu/x86/tsc.h>
-
 struct timestamp_entry {
        uint32_t        entry_id;
        uint64_t        entry_stamp;
@@ -55,6 +53,7 @@ enum timestamp_id {
 };
 
 #if CONFIG_COLLECT_TIMESTAMPS
+#include <cpu/x86/tsc.h>
 void timestamp_init(tsc_t base);
 void timestamp_add(enum timestamp_id id, tsc_t ts_time);
 void timestamp_add_now(enum timestamp_id id);

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to