[PATCH] [RFC] - Should s2api consider ATSC a legacy delivery system?

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

 



I tried out Christophe Thommeret's Kaffeine s2api changes, and ran
into a problem.  The u.vsb.modulation field was always zero, resulting
in lgdt330x indicating failures.

I traced through the new dvb_frontend code and it appears that this
field is set when we consider the delivery system to be "legacy".

The following patch causes the s2api to start working with ATSC
devices.  I'm not 100% sure this is the *right* fix, so I am
soliciting feedback on whether this was just an oversight.

I ran into this with both lgdt3303 and the s5h1411 based devices (so
it doesn't appear specific to any particular atsc demod), so unless
I'm crazy it seems like ATSC tuning through s2api is 100% broken.

Regards,

Devin

-- 
Devin J. Heitmueller
http://www.devinheitmueller.com
AIM: devinheitmueller
s2api - ATSC should be considered a legacy delivery system

From: Devin Heitmueller <devin.heitmueller@xxxxxxxxx>

ATSC should be considered a legacy delivery system, or else fields such as 
p->u.vsb.modulation do not get populated (resulting in set_frontend failures)

Signed-off-by: Devin Heitmueller <devin.heitmueller@xxxxxxxxx>

diff -r 46604f47fca1 linux/drivers/media/dvb/dvb-core/dvb_frontend.c
--- a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c	Fri Nov 07 15:24:18 2008 -0200
+++ b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c	Sun Nov 09 21:04:05 2008 -0500
@@ -1064,7 +1064,8 @@
 int is_legacy_delivery_system(fe_delivery_system_t s)
 {
 	if((s == SYS_UNDEFINED) || (s == SYS_DVBC_ANNEX_AC) ||
-		(s == SYS_DVBC_ANNEX_B) || (s == SYS_DVBT) || (s == SYS_DVBS))
+	   (s == SYS_DVBC_ANNEX_B) || (s == SYS_DVBT) || (s == SYS_DVBS) ||
+	   (s == SYS_ATSC))
 		return 1;
 
 	return 0;
_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux