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

[Matthias Hilbig <hilbig@upb.de>]

i wanted to include something like the lirc_register_plugin into the dvb 
code:

dvb.c would export two symbols:
register_irc_handler(callback) and
unregister_irc_handler(callback)

The callback will be called by dvb.c if an interrupt occures.

The lirc_dvb module will have to register itself at the dvb driver and then
gets the int variable ircommand delivered per callback function.
its something like a redirect of the IRQ function...

dvb.c won't have to include any other headers and no other module won't have
to be loaded before the dvb driver.

Another thought of mine was to write also a directvdr module, because lirc 
didn't react very fast in my experiments and i want a rc that responds very 
fast (like my tv...)

i hope the idea has become clear...

Matthias
--On Monday, October 15, 2001 17:25:04 +0200 Ralph Metzler 
<rjkm@convergence.de> wrote:

> 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.