Hamish Moffatt wrote:

On Fri, Jan 16, 2004 at 09:55:49AM +0100, Roland Gersch wrote:


Hamish Moffatt wrote:


There's a linux/include/config/dvb/tda1004x/firmware/file.h, and that
seems to get pulled into <linux/config.h>, which tda1004x.c doesn't use.
Looks like that would fix it.



I didn't investigate further into file.h, but maybe the following patch
fixes the problem:

--- drivers/media/dvb/frontends/tda1004x.c 2004-01-10 13:57:47.000000000
+0100
+++ drivers/media/dvb/frontends/tda1004x.c 2004-01-16 07:49:39.000000000 +0100
@@ -41,11 +41,14 @@
#include <linux/unistd.h>
#include <linux/fcntl.h>
#include <linux/errno.h>
+#include <linux/config.h>
#include "dvb_frontend.h"
#include "dvb_functions.h"


-#ifndef DVB_TDA1004X_FIRMWARE_FILE
+#ifndef CONFIG_DVB_TDA1004X_FIRMWARE_FILE
#define DVB_TDA1004X_FIRMWARE_FILE "/usr/lib/hotplug/firmware/tda1004x.mc"
+#else
+#define DVB_TDA1004X_FIRMWARE_FILE CONFIG_DVB_TDA1004X_FIRMWARE_FILE
#endif

static int tda1004x_debug = 0;



That seems reasonable, though linux/config.h might be cleaner.




I don't understand. The patch makes tda1004x.c include linux/config.h.

Sincerely,
Roland




-- Info: To unsubscribe send a mail to [EMAIL PROTECTED] with "unsubscribe linux-dvb" as subject.



Reply via email to