Re: [PATCH] fc0011: Reduce number of retries

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03.04.2012 18:33, Michael Büsch wrote:
On Tue, 03 Apr 2012 18:24:20 +0300
Antti Palosaari<crope@xxxxxx>  wrote:

On 03.04.2012 12:05, Michael Büsch wrote:
Now that i2c transfers are fixed, 3 retries are enough.

Signed-off-by: Michael Buesch<m@xxxxxxx>

Applied, thanks!
http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/af9035_experimental

I think I will update original af9035 PULL request soon for the same
level as af9035_experimental is currently.

That's great. The driver really works well for me.

On another thing:
The af9035 driver doesn't look multi-device safe. There are lots of static
variables around that keep device state. So it looks like this will
blow up if multiple devices are present in the system. Unlikely, but still... .
Are there any plans to fix this up?
If not, I'll probably take a look at this. But don't hold your breath.

That's true and same applies for many other DVB USB drivers. Main reason for current hackish situation is DVB USB core limits. For example priv is not available until frontend attach etc. It "just" works even a little bit luck. Good example is that sequence counter, if you have multiple devices it runs wrongly as all increases same counter. But as a firmware does not care sequence numbers it still works. Remote controller is other big problem - coming from same limitations. And that is not first time these are spoken :)

I have thought to redesign whole DVB USB framework, but as I am too busy always I haven't done that. Feel free to start fixing.


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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux