[coreboot] Patch set updated for coreboot: cc68333 Fix gcc-4.7 building problem.

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Wed Nov 7 20:25:23 CET 2012


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1784

-gerrit

commit cc683332c8265ac36e730ceb6556ceb854cda862
Author: Han Shen <shenhan at google.com>
Date:   Tue Oct 16 10:42:25 2012 -0700

    Fix gcc-4.7 building problem.
    
    Applied function attribute to function definition to avoid 'conflicting type' warning.
    
    Function declaration is in src/include/cpu.h
      void secondary_cpu_init(unsigned int cpu_index)__attribute__((regparm(0)));
    
    But function definition in lapic_cpu_init.c is missing the "__attribute__" part.
    
    Change-Id: Idb7cd00fda5a2d486893f9866920929c685d266e
    Signed-off-by: Han Shen <shenhan at google.com>
---
 src/cpu/x86/lapic/lapic_cpu_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c
index fc5a786..b721e8f 100644
--- a/src/cpu/x86/lapic/lapic_cpu_init.c
+++ b/src/cpu/x86/lapic/lapic_cpu_init.c
@@ -373,7 +373,7 @@ static __inline__ __attribute__((always_inline)) void writecr4(unsigned long Dat
 #endif
 
 /* C entry point of secondary cpus */
-void secondary_cpu_init(void)
+void __attribute__((regparm(0))) secondary_cpu_init(void)
 {
 	atomic_inc(&active_cpus);
 #if CONFIG_SERIAL_CPU_INIT




More information about the coreboot mailing list