Dear Mauro,

first of all, thank you for providing a proper fix that quickly.

On Freitag, 5. April 2013 13:18:54 Mauro Carvalho Chehab wrote:
> Em Fri, 05 Apr 2013 13:25:01 +0100
> 
> Adam Sampson <ats@offog.org> escreveu:
> > Hans-Peter Jansen <hpj@urpla.net> writes:
> > > In one of my systems, I've used a
> > > Hauppauge Nova-S-Plus DVB-S card successfully, but after a system
> > > upgrade to openSUSE 12.2, it cannot tune in all but one channel.
> > 
> > [...]
> > 
> > > initial transponder 12551500 V 22000000 5
> > > 
> > >>>> tune to: 12551:v:0:22000
> > > 
> > > DVB-S IF freq is 1951500
> > > WARNING: >>> tuning failed!!!
> > 
> > I suspect you might be running into this problem:
> >   https://bugzilla.kernel.org/show_bug.cgi?id=9476
> > 
> > The bug title is misleading -- the problem is actually that the card
> > doesn't get configured properly to send the 22kHz tone for high-band
> > transponders, like the one in your error above.
> > 
> > Applying this patch makes my Nova-S-Plus work with recent kernels:
> >   https://bugzilla.kernel.org/attachment.cgi?id=21905&action=edit
> 
> Applying that patch would break support for all other devices with
> isl6421.
> 
> Could you please test the enclosed patch? It allows the bridge
> driver to tell if the set_tone should be overrided by isl6421 or
> not. The code only changes it for Hauppauge model 92001.

Unfortunately, it appears to be more problematic. While the fix allows to scan 
the channel list, it is not complete (in another setup at the same dish (via 
multiswitch), vdrs channel list has about 1600 channels, while scan does 
collect 1138 only.

More importantly, a single channel (arte) is received with 0 BER and a S/N 
ratio of 99%, while all other channels produce more BER, eg. "Das Erste" with 
about 320 BER (SNR 99%, a few artifacts/distortions occasionally), 
"ZDF" about 6400 BER, (SNR drops down to 75%, constant distortions, and many 
channels doesn't produce anything beyond distortions with a video stream  
below 0.3 MBit/s and about 160000 BER. (measured using vdr femon plugin v. 
1.6.7)

So, still no cigar, sorry.

I've tested both patches, just to be sure, with the same result. I had to 
relocate and refresh yours in order to apply it to 3.4, since the paths 
changed, result attached.

> If it works, please answer this email with a:
> 	Tested-by: your name <your@email>
> 
> For me to add it when merging the patch upstream.
> 
> Regards,
> Mauro.

It looks like the idea is sound, but the logic is still missing something that 
prevents it from tuning most channels properly.

Cheers,
Pete