[linux-dvb] Re: [Patch] Adding support for the Hauppage HVR1100
js at linuxtv.org
Thu Dec 1 20:25:42 CET 2005
On Thu, Dec 01, 2005, Steve Toth wrote:
> Johannes Stezenbach wrote:
> >No. If I understand your code correctly (sorry, I have very little
> >time and it's likely I misunderstand because I just skimmed
> >over your code), you use it to open the I2C gate for every i2c
> >transaction which v4l does, no matter what part v4l is talking to.
> >The "rename" thing was just to tease you ;-), I still don't get
> >why you can't use it specifically for the VIDIOC_SET_FREQ ioctl
> >and nowhere else?
> The gate is opened by atleast three (maybe four) different ioctls and
> three non-ioctl related functions that deal with the IF demod inside the
> PLL.Anything that modified the PLL needs access and anything that
> touched the tda9887 needs access.
> Two I2C devices live behind the gate. Sorry if I did not make this clear
> It's used in atleast six places and I have not looked at any possible
> cx88-tvaudio or cx88-alsa (future) implications. I have a feeling it
> could be as high as 10-15 places.
> If it was just one place I would of done that a few days ago and this
> would all be over.
Ah, OK, maybe I should read patches completely before responding...
Anyway, enable/disable_plli2c() should then probably be named
i2c_gate_ctrl(..., int enable) or something like that if it
doesn't deal with just plls.
More information about the linux-dvb