Em Seg, 2007-08-20 às 11:22 -0400, Michael Krufky escreveu:> Manu Abraham wrote:> > You will need to handle the IOCTL calls for analog operations some> > place, whatever you remove.> >> > I don't see at any place you are handling the IOCTL's directly in the> > drivers. So you will be using callbacks from the place where you> > are applying the ioctl's> You are getting ahead of things, Manu. The tuner.ko module will > continue to remain as the v4l2 / i2c_client interface to the tuning > functionality. For now, at a first glance, the way it was mapped seems fine. The waythe struct is described, a frontend can handle digital and/or analogtuning. This seems OK. Using void* priv will just create some troubles, since this will avoidgcc to do typecast checkings for no gain. > We do not (yet) have to add these ioctl calls to the > card-level drivers, as they will continue to interact with the various > tuner modules via tuner.ko's i2c_client interface.> > I do have plans to remove this tuner.ko interface in the future, but > that is way way in the future, after most of my planned tuner > refactoring work is complete. There will be a completely separate RFC > issued at that time. When that happens, the card level drivers will > include dvb_frontend.h, just as the dvb card drivers do, and access the > tuning methods that way, through the dvb_frontend structure. Removing the common ioctl handling code done by tuner.ko doesn't seem tobe nice, since this would generate overhead on other drivers that willneed to absorb the common tuner handling and probing. However, aspointed, this should be discussed later, after finishing the tunerrefactoring. Michael, I didn't reviewed yet the tea5767 changes. I expect to do it later thisweek (maybe today night). Cheers,Mauro _______________________________________________linux-dvb mailing listlinux-dvb@xxxxxxxxxxxxxxx://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
