lkml.org 
[lkml]   [2005]   [Jan]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: 2.6.11-rc2-mm1: v4l-saa7134-module compile error
> > The patch below should fix this.
>
> Not completely:

> CC drivers/media/video/saa7134/saa7134-core.o
> drivers/media/video/saa7134/saa7134-core.c: In function `saa7134_initdev':
> drivers/media/video/saa7134/saa7134-core.c:997: error: `need_empress' undeclared (first use in this function)

New version, this time using a #define, which should kill the reference
to need_* as well ...

Gerd

Index: linux-2005-01-23/drivers/media/video/saa7134/saa7134-core.c
===================================================================
--- linux-2005-01-23.orig/drivers/media/video/saa7134/saa7134-core.c 2005-01-24 18:43:20.000000000 +0100
+++ linux-2005-01-23/drivers/media/video/saa7134/saa7134-core.c 2005-01-25 10:04:17.000000000 +0100
@@ -21,6 +21,7 @@
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*/

+#include <linux/config.h>
#include <linux/init.h>
#include <linux/list.h>
#include <linux/module.h>
@@ -225,6 +226,8 @@ static void dump_statusregs(struct saa71
/* ----------------------------------------------------------- */
/* delayed request_module */

+#ifdef CONFIG_MODULES
+
static int need_empress;
static int need_dvb;

@@ -265,6 +268,12 @@ static void request_module_depend(char *
}
}

+#else
+
+#define request_module_depend(name,flag)
+
+#endif /* CONFIG_MODULES */
+
/* ------------------------------------------------------------------ */

/* nr of (saa7134-)pages for the given buffer size */
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 14:09    [W:0.093 / U:1.948 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site