Hi Devin, Devin Heitmueller writes: > On Sun, Jul 3, 2011 at 12:31 PM, Oliver Endriss <o.endr...@gmx.de> wrote: > > [PATCH 01/16] tda18271c2dd: Initial check-in > > [PATCH 02/16] tda18271c2dd: Lots of coding-style fixes > > Oliver, > > Why the new driver for the 18271c2? There is already such a driver, > and in the past we've rejected multiple drivers for the same chip > unless there is a *very* compelling reason to do such. > > The existing 18271 driver supports the C2 and is actively maintained. >
AFAIR, there were at least 2 reasons. One was that the existing driver does not accept 2 (or even 4) tuners with the same address (but behind different demods) on the same I2C bus which is the case on duoflex C/T addon cards. The other was that it does not give back the intermediate frequency which the demod needs. (This is currently done by misusing get_frequency() but I added a get_if() call in newer internal versions which should be added to dvb-core/dvb_frontend.h) Feel free to change ngene/ddbridge to use the existing driver but it will need some major changes in tda18271_attach() and a few other places. Regards, Ralph -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
