On 09/01/2014 12:54 PM, Akihiro TSUKADA wrote:
Hi,
Also, I would like to see all new drivers (demod and tuner) implemented
as a standard kernel I2C drivers (or any other bus). I have converted
already quite many drivers, si2168, si2157, m88ds3103, m88ts2022,
it913x, tda18212, ...
I wrote the code in the old style using dvb_attach()
because (I felt) it is simpler than using i2c_new_device() by
introducing new i2c-related data structures,
registering to both dvb and i2c, without any new practical
features that i2c client provides.
Of course it is simpler to do old style as you could copy & paste older
drivers and so. However, for a long term we must get rid of all DVB
specific hacks and use common kernel solutions. The gap between common
kernel solutions and DVB proprietary is already too big, without any
good reason - just a laziness of developers to find out proper solutions
as adding hacks is easier.
I mentioned quite many reasons earlier and If you look that driver you
will see you use dev_foo() logging, that does not even work properly
unless you convert driver to some kernel binding model (I2C on that
case) (as I explained earlier).
There is also review issues. For more people do own tricks and hacks the
harder code is review and also maintain as you don't never know what
breaks when you do small change, which due to some trick used causes
some other error.
Here is one example I fixed recently:
https://patchwork.linuxtv.org/patch/25776/
Lets mention that I am not even now fully happy to solution, even it
somehow now works. Proper solution is implement clock source and clock
client. Then register client to that source. And when client needs a
clock (or power) it makes call to enable clock.
But if the use of dvb_attach() is (almost) deprecated and
i2c client driver is the standard/prefered way,
I'll convert my code.
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
