On Mon, 04 Apr 2011 14:02:31 +0200 Lutz Sammer <johns98@xxxxxxx> wrote: > Fixes stb0899 not locking. > See http://www.spinics.net/lists/linux-media/msg30486.html ... > > When stb0899_check_data is entered, it could happen, that the data is > already locked and the data search looped. stb0899_check_data fails > to lock on a good frequency. stb0899_search_data uses an extrem big > search step and fails to lock. > > The new code checks for lock before starting a new search. > The first read ignores the loop bit, for the case that the loop bit is > set during the search setup. I also added the msleep to reduce the > traffic on the i2c bus. > Thanks Lutz for getting down to the problem :) ! Manu, Mauro, Any comments ? Let's have that finally sorted. I think its proven now that its a bug. We have a fix. --- A few test result on 2.6.39-rc3 from vdr-portal(thx to jrie, hope its ok for him). This is tuning a pre defined channel list until we have a lock and then tune the next. Astra_only.txt + Original TOT: lok_errs =172, runs=1136 of sequ=1135, multi=56032, multi_max=931 real 101m40.777s user 0m0.083s sys 0m19.039s Astra_only.txt + stb0899_not_locking_fix.diff TOT: lok_errs =0, runs=1136 of sequ=1135, multi=289, multi_max=99 real 17m15.636s user 0m0.007s sys 0m9.445s -- 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
