[LinuxBIOS] [PATCH] v3: fix compilation due to multiple definitions of printk

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Thu Nov 22 21:44:03 CET 2007


The introduction of the _SHARED macros had one flaw: If multiple files
had _SHARED defined during compilation, each of them would contain an
assignment of stage0_printk to *printk. During linking, this caused
errors as multiple definitions of printk existed.
Make sure _SHARED alone gives you only the printk prototype, and iff
_MAINOBJECT is defined as well, include the assignment.

Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>

--- 
Index: LinuxBIOSv3/include/shared.h
===================================================================
--- LinuxBIOSv3/include/shared.h	(Revision 515)
+++ LinuxBIOSv3/include/shared.h	(Arbeitskopie)
@@ -29,9 +29,16 @@
  */
 #define FUNC(func, ret, attr, args...)   \
 	ret stage0_##func(args) attr
+
+#ifdef _MAINOBJECT
 #define EXTERN(func, ret, attr, args...) \
 	ret (*func)(args) attr= stage0_##func
 #else
+#define EXTERN(func, ret, attr, args...) \
+	ret *func(args) attr
+#endif
+
+#else
 #define FUNC(func, ret, attr, args...)   \
 	ret func(args) attr
 #define EXTERN(func, ret, attr, args...)
Index: LinuxBIOSv3/mainboard/adl/msm800sev/Makefile
===================================================================
--- LinuxBIOSv3/mainboard/adl/msm800sev/Makefile	(Revision 515)
+++ LinuxBIOSv3/mainboard/adl/msm800sev/Makefile	(Arbeitskopie)
@@ -34,7 +34,7 @@
 
 # Next Quest: Make a single rule out of those:
 $(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c
-	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
+	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -D_MAINOBJECT -fPIE -c $< -o $@
 $(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c
 	$(Q)mkdir -p $(dir $@)
 	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
Index: LinuxBIOSv3/mainboard/amd/norwich/Makefile
===================================================================
--- LinuxBIOSv3/mainboard/amd/norwich/Makefile	(Revision 515)
+++ LinuxBIOSv3/mainboard/amd/norwich/Makefile	(Arbeitskopie)
@@ -28,7 +28,7 @@
 
 # Next Quest: Make a single rule out of those:
 $(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c
-	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
+	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -D_MAINOBJECT -fPIE -c $< -o $@
 $(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c
 	$(Q)mkdir -p $(dir $@)
 	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
Index: LinuxBIOSv3/mainboard/artecgroup/dbe61/Makefile
===================================================================
--- LinuxBIOSv3/mainboard/artecgroup/dbe61/Makefile	(Revision 515)
+++ LinuxBIOSv3/mainboard/artecgroup/dbe61/Makefile	(Arbeitskopie)
@@ -32,7 +32,7 @@
 
 # Next Quest: Make a single rule out of those:
 $(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c
-	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
+	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -D_MAINOBJECT -fPIE -c $< -o $@
 $(obj)/northbridge/amd/geodelx/raminit.o: $(src)/northbridge/amd/geodelx/raminit.c
 	$(Q)mkdir -p $(dir $@)
 	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
Index: LinuxBIOSv3/mainboard/emulation/qemu-x86/Makefile
===================================================================
--- LinuxBIOSv3/mainboard/emulation/qemu-x86/Makefile	(Revision 515)
+++ LinuxBIOSv3/mainboard/emulation/qemu-x86/Makefile	(Arbeitskopie)
@@ -43,7 +43,7 @@
 INITRAM_OBJ = $(obj)/mainboard/$(MAINBOARDDIR)/initram.o
 
 $(obj)/mainboard/$(MAINBOARDDIR)/initram.o: $(src)/mainboard/$(MAINBOARDDIR)/initram.c
-	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -fPIE -c $< -o $@
+	$(Q)$(CC) $(INITCFLAGS) -D_SHARED -D_MAINOBJECT -fPIE -c $< -o $@
 
 $(obj)/linuxbios.initram $(obj)/linuxbios.initram.map: $(obj)/stage0.init $(obj)/stage0-prefixed.o $(INITRAM_OBJ)
 	$(Q)# initram links against stage0






More information about the coreboot mailing list