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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 1 Jul 2013 12:35:12 -0300
From:	Mauro Carvalho Chehab <mchehab@...hat.com>
To:	Manu Abraham <abraham.manu@...il.com>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Media Mailing List <linux-media@...r.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Zoran Turalija <zoran.turalija@...il.com>,
	Srinivas KANDAGATLA <srinivas.kandagatla@...com>,
	Nicolas THERY <nicolas.thery@...com>,
	Divneil Rai WADHAWAN <divneil.wadhawan@...com>,
	Vincent ABRIOU <vincent.abriou@...com>,
	Alain VOLMAT <alain.volmat@...com>
Subject: Re: [GIT PULL for v3.11] media patches for v3.11

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

> Mauro,
> 
> On Mon, Jul 1, 2013 at 4:28 PM, Mauro Carvalho Chehab
> <mchehab@...hat.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/