On 27.04.2012 22:01, Konstantin Dimitrov wrote:
Mauro, your reasoning makes sense to me. so, let's split them and at
least settle this part of the discussion - i will do as far as my
spare time allows, as well make sure there are no some problems
introduced after the split.
also, in one email i've just sent in answer to Antti there is enough
argument why such split, i.e. tuner-pass-through-mode is subject to
discussion about CX24116 and TDA10071 drivers too. currently, majority
of DVB-S2 demodulator drivers in the kernel are married to particular
tuners and there is no split.
I read the mail and as it was long study, I comment only that
CX24116+CX24118A and TDA10071+CX24118A demod+tuner combos versus Montage
demod+tuner combos. As you may see, CX24116 and TDA10071 are so much
different than both needs own driver. But as you said those are married
always as a demod+tuner.
So if I use your logic, what happens if CX24118A tuner is not driven by
CX24116 or TDA10071 firmware? ==> it happens we have two drivers,
CX24116 and TDA10071 *both* having similar CX24118A tuner driver code
inside! Same tuner driver code inside two demods drivers. Could you now
understand why we want it split?
The reason which saves us having CX24118A tuner driver is that it is
inside both CX24116 and TDA10071 firmware.
There is mainly two different controlling situation. Most commonly
driver controls chip but in some cases it is firmware which is
controlling. And I don't see it very important trying always to by-pass
firmware control and use driver for that.
Patrick explained those few days back in the mailing list:
http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg44814.html
You said also we cannot know if Montage demod does some tweaking for the
tuner too. Yes true, at that point we don't know. But I think it is
rather small probability whilst driver clearly controls it.
regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
