Re: [GIT PULL for v3.11] media patches for v3.11

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em Mon, 1 Jul 2013 16:37:58 +0530
Manu Abraham <abraham.manu@gmail.com> escreveu:

> Mauro,
> 
> On Mon, Jul 1, 2013 at 4:28 PM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
> > Hi Linus,
> >
> > Please pull from:
> >   git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media v4l_for_linus
> >
> > For the media patches for Kernel v3.11.
> >
> 
> >
> > Zoran Turalija (2):
> >       [media] stb0899: allow minimum symbol rate of 1000000
> >       [media] stb0899: allow minimum symbol rate of 2000000
> 
> 
> Somehow, I missed these patches; These are incorrect. Please revert
> these changes.
> Simply changing the advertized minima values don't change the search algorithm
> behaviour, it simply leads to broken behaviour.
> 
> NACK for these changes.

While this patch came from a sub-maintainer's tree, looking at its
history, the patch was proposed here:
	https://linuxtv.org/patch/18341/

>From what it is said there, with this patch, 6 additional channels
were discovered when using with Eutelsat 16A, that uses a symbol
rate between 2MS/s to 5 MS/s. Without this patch, those channels won't
be discovered, as the core won't try to use a symbol rate outside
the range.

Of course, transponders with a symbol rate equal or upper than 5MS/s
won't be affected by this patch.

Even if this is not a perfect patch and some changes would be 
needed to improve tuning for those low symbol rate transponders, 
it seems better than before, as at least now some channels are tuned.

The only reason I can see to reverse this patch is that if setting
the frontend to low bit ranges could damage the frontend or could
hit some bug on the hardware (or internal firmware).

Yet, from the datasheet pointed by the patch author, it seems that
this frontend allows such low symbol rates:
	http://comtech.sg1002.myweb.hinet.net/pdf/dvbs2-6899.pdf

Let me copy some people at ST. Maybe they could help to check internally
there if there are any risks for the hardware to accept low symbol rate 
transponders.

Regards,
Mauro