Re: idea on how to break the static dependencies on demodulator modules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Hey, this is a great idea! And surprisingly simple.
> No more loading of frontend drivers which are not required!
>
> Does request_module() log a message if the module cannot be loaded?
> If yes, fine. If no, a message should be logged by the DVB_ATTACH macro:
>
> #define DVB_ATTACH(RESULT, MODULE, FUNCTION,ARGS...) \
> do { \
> 	typeof(&FUNCTION) __a;				\
> 	request_module(MODULE);				\
> 	__a = symbol_get(FUNCTION);			\
> 	if (__a)					\
> 		RESULT = __a(ARGS);			\
> 	else {						\
> 		printk("unable to load %s\n", MODULE);	\
> 		RESULT = NULL;				\
> 	}						\
> } while(0);
>
> Looks perfect otherise. Great idea!

Heh, glad you agree :)

Good idea about the logging... will add it.

_______________________________________________

linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux