public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Compilation trouble in drivers/video
@ 2002-05-17 22:50 Michal Jaegermann
  0 siblings, 0 replies; only message in thread
From: Michal Jaegermann @ 2002-05-17 22:50 UTC (permalink / raw)
  To: linux-kernel, Marcelo Tossati; +Cc: Alan Cox


There is a compilation issue in 'drivers/video'.  Namely 'fbgen.o' is
used by some other objects ('pm2fb.o', 'pm3fb.o', 'clgenfb.o',
'tridentfb.o', 'tgafb.o', 'stifb.o').  Now if at least one of these
"users" is configured as built-in, implying that fbgen.o will be compiled
as an integral part of a kernel, and some other(s) as modules then you
will see from 'depmod' complaints about unresolved symbols for functions
provided by fbgen.

A fix is obvious.  Add missing EXPORT_SYMBOL(...) to 'fbgen.c' and
'fbgen.o' to a list of export-objs in drivers/video/Makefile.
Here is a patch (for 2.4.19-pre8 but this did not really change
for quite a while).


--- linux-2.4.19-pre/drivers/video/fbgen.c~	Wed Mar 27 17:03:46 2002
+++ linux-2.4.19-pre/drivers/video/fbgen.c	Fri May 17 16:08:17 2002
@@ -448,3 +448,21 @@
 	fbgen_install_cmap(currcon, info2);
 }
 MODULE_LICENSE("GPL");
+
+
+    /*
+     *  Visible symbols for modules
+     */
+
+EXPORT_SYMBOL(fbgen_get_var);
+EXPORT_SYMBOL(fbgen_get_cmap);
+EXPORT_SYMBOL(fbgen_get_fix);
+EXPORT_SYMBOL(fbgen_set_var);
+EXPORT_SYMBOL(fbgen_set_cmap);
+EXPORT_SYMBOL(fbgen_set_disp);
+EXPORT_SYMBOL(fbgen_install_cmap);
+EXPORT_SYMBOL(fbgen_pan_display);
+EXPORT_SYMBOL(fbgen_update_var);
+EXPORT_SYMBOL(fbgen_do_set_var);
+EXPORT_SYMBOL(fbgen_switch);
+EXPORT_SYMBOL(fbgen_blank);
--- linux-2.4.19-pre/drivers/video/Makefile~	Sat Mar 30 11:07:18 2002
+++ linux-2.4.19-pre/drivers/video/Makefile	Fri May 17 16:08:50 2002
@@ -15,7 +15,7 @@
 		  fbcon-iplan2p8.o fbcon-vga-planes.o fbcon-cfb16.o \
 		  fbcon-cfb2.o fbcon-cfb24.o fbcon-cfb32.o fbcon-cfb4.o \
 		  fbcon-cfb8.o fbcon-mac.o fbcon-mfb.o \
-		  cyber2000fb.o sa1100fb.o fbcon-hga.o
+		  cyber2000fb.o sa1100fb.o fbcon-hga.o fbgen.o
 
 # Each configuration option enables a list of files.

   Michal

 

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2002-05-17 22:51 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-05-17 22:50 [PATCH] Compilation trouble in drivers/video Michal Jaegermann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox