[linux-dvb] clean RC support for rev2.1

[Ralph Metzler <rjkm@convergence.de>]

Matthias Hilbig writes:
 > the rc support at the moment is in my opinion a little bit unclean:
 > dvb.c includes header files from lirc
 > and you have to insmod lirc_dev.o before you load the dvb-driver
 > 
 > my idea was to put all lirc code in dvb.c into a seperate lirc module and
 > insert a procedure in dvb.c that registers callback functions that get 
 > called if
 > ir-events arrive:
 > 
 > f.e:
 > register_irc_handler(pointer to function)
 > unregister_irc_handler(pointer to function)
 > 
 > the function would receive ircommand
 > the IRQ handle function would start a tasklet that will call all registered 
 > functions (or just one...) with the received ircommand
 > 
 > just wanted to ask if it will be accepted in the driver, before i start the 
 > actual work :-)


OK, but I don't understand the difference to the current approach:
dvb.c includes lirc headers and registers with lirc, callback 
functions are called ...

You add another module which also has to be loaded before dvb.c, dvb.c
also will have to include the headers of this module, this module will
have to include the lirc header and have lirc_dev loaded, etc. ...

I thought that's what lirc_dev is for. Or am I missing something?


Ralph


 





-- 
Info:
To unsubscribe send a mail to listar@linuxtv.org with "unsubscribe linux-dvb" as subject.