[coreboot-gerrit] New patch to review for coreboot: c400497 vendorcode/intel: remove DebugDeadLoop() from fsptypes.h

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Sun Jan 11 23:57:39 CET 2015


Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8193

-gerrit

commit c400497eac5dfff5bafe2183953ec3b8aa099c80
Author: Martin Roth <gaumless at gmail.com>
Date:   Sun Jan 11 11:34:36 2015 -0700

    vendorcode/intel: remove DebugDeadLoop() from fsptypes.h
    
    When included for the CAR transition, this was causing the error:
    error: invalid storage class for function 'DebugDeadLoop'
    
    Change-Id: Idf37a8104b4468b40c29c8cbe9a40f7a357a4f17
    Signed-off-by: Martin Roth <gaumless at gmail.com>
---
 src/vendorcode/intel/fsp/baytrail/include/fsptypes.h          | 9 ++-------
 src/vendorcode/intel/fsp/ivybridge_bd82x6x/include/fsptypes.h | 9 ++-------
 src/vendorcode/intel/fsp/ivybridge_i89xx/include/fsptypes.h   | 9 ++-------
 src/vendorcode/intel/fsp/rangeley/include/fsptypes.h          | 9 ++-------
 4 files changed, 8 insertions(+), 28 deletions(-)

diff --git a/src/vendorcode/intel/fsp/baytrail/include/fsptypes.h b/src/vendorcode/intel/fsp/baytrail/include/fsptypes.h
index 54f75a2..4a313ad 100644
--- a/src/vendorcode/intel/fsp/baytrail/include/fsptypes.h
+++ b/src/vendorcode/intel/fsp/baytrail/include/fsptypes.h
@@ -100,22 +100,17 @@ typedef struct {
 #define TRUE  ((BOOLEAN)(1==1))
 #define FALSE ((BOOLEAN)(0==1))
 
-static inline void DebugDeadLoop(void) {
-  for (;;);
-}
-
 #define FSPAPI __attribute__((cdecl))
 #define EFIAPI __attribute__((cdecl))
 
-#define _ASSERT(Expression)  DebugDeadLoop()
 #define ASSERT(Expression)      \
   do {                          \
     if (!(Expression)) {        \
-      _ASSERT (Expression);     \
+      for (;;);                 \
     }                           \
   } while (FALSE)
 
 typedef UINT32 FSP_STATUS;
 typedef UINT32 EFI_STATUS;
 
-#endif
\ No newline at end of file
+#endif
diff --git a/src/vendorcode/intel/fsp/ivybridge_bd82x6x/include/fsptypes.h b/src/vendorcode/intel/fsp/ivybridge_bd82x6x/include/fsptypes.h
index 8950367..a3a54c0 100644
--- a/src/vendorcode/intel/fsp/ivybridge_bd82x6x/include/fsptypes.h
+++ b/src/vendorcode/intel/fsp/ivybridge_bd82x6x/include/fsptypes.h
@@ -100,22 +100,17 @@ typedef struct {
 #define TRUE  ((BOOLEAN)(1==1))
 #define FALSE ((BOOLEAN)(0==1))
 
-static inline void DebugDeadLoop(void) {
-  for (;;);
-}
-
 #define FSPAPI __attribute__((cdecl))
 #define EFIAPI __attribute__((cdecl))
 
-#define _ASSERT(Expression)  DebugDeadLoop()
 #define ASSERT(Expression)      \
   do {                          \
     if (!(Expression)) {        \
-      _ASSERT (Expression);     \
+      for (;;);                 \
     }                           \
   } while (FALSE)
 
 typedef UINT32 FSP_STATUS;
 typedef UINT32 EFI_STATUS;
 
-#endif
\ No newline at end of file
+#endif
diff --git a/src/vendorcode/intel/fsp/ivybridge_i89xx/include/fsptypes.h b/src/vendorcode/intel/fsp/ivybridge_i89xx/include/fsptypes.h
index 8950367..a3a54c0 100644
--- a/src/vendorcode/intel/fsp/ivybridge_i89xx/include/fsptypes.h
+++ b/src/vendorcode/intel/fsp/ivybridge_i89xx/include/fsptypes.h
@@ -100,22 +100,17 @@ typedef struct {
 #define TRUE  ((BOOLEAN)(1==1))
 #define FALSE ((BOOLEAN)(0==1))
 
-static inline void DebugDeadLoop(void) {
-  for (;;);
-}
-
 #define FSPAPI __attribute__((cdecl))
 #define EFIAPI __attribute__((cdecl))
 
-#define _ASSERT(Expression)  DebugDeadLoop()
 #define ASSERT(Expression)      \
   do {                          \
     if (!(Expression)) {        \
-      _ASSERT (Expression);     \
+      for (;;);                 \
     }                           \
   } while (FALSE)
 
 typedef UINT32 FSP_STATUS;
 typedef UINT32 EFI_STATUS;
 
-#endif
\ No newline at end of file
+#endif
diff --git a/src/vendorcode/intel/fsp/rangeley/include/fsptypes.h b/src/vendorcode/intel/fsp/rangeley/include/fsptypes.h
index 3d2e663..5912e01 100644
--- a/src/vendorcode/intel/fsp/rangeley/include/fsptypes.h
+++ b/src/vendorcode/intel/fsp/rangeley/include/fsptypes.h
@@ -100,18 +100,13 @@ typedef struct {
 #define TRUE  ((BOOLEAN)(1==1))
 #define FALSE ((BOOLEAN)(0==1))
 
-static inline void DebugDeadLoop(void) {
-  for (;;);
-}
-
 #define FSPAPI __attribute__((cdecl))
 #define EFIAPI __attribute__((cdecl))
 
-#define _ASSERT(Expression)  DebugDeadLoop()
 #define ASSERT(Expression)      \
   do {                          \
     if (!(Expression)) {        \
-      _ASSERT (Expression);     \
+      for (;;);                 \
     }                           \
   } while (FALSE)
 
@@ -180,4 +175,4 @@ typedef UINT32 EFI_STATUS;
 #define SIGNATURE_64(A, B, C, D, E, F, G, H) \
     (SIGNATURE_32 (A, B, C, D) | ((UINT64) (SIGNATURE_32 (E, F, G, H)) << 32))
 
-#endif
\ No newline at end of file
+#endif



More information about the coreboot-gerrit mailing list