Re: Broken ttusb-dec DVB support since, well, year(s)

Previous thread: [bad page] memcg: another bad page at page migration (2.6.26-rc5-mm3 + patch collection) by Daisuke Nishimura on Monday, June 23, 2008 - 1:53 am. (11 messages)

Next thread: 2.6.26-rc6: pcmcia stopped working after resume by Pavel Machek on Sunday, June 22, 2008 - 6:40 pm. (14 messages)
To: <linux-kernel@...>
Date: Monday, June 23, 2008 - 2:55 am

Moin moin!

Sorry I'm posting this here rather than to the more specific
linux-dvb@ mailing list where I'd rather this appear, but my
attempts to surbscibe!@! to that list have been rejected, first
as an invalid e-mail, then an insecure e-mail, and I've been
unable to progress further from this throwaway account.

Feel free to redirect/repost this message there if you're a
qualified developer and feel it's more appropriate.

There is a change that was introduced to the file
drivers/media/dvb/ttusb-dec/ttusbdecfe.c
sometime in the not-too-recent past and which resulted in breaking
the support which early 2.6.1x kernels had for my Hauppauge
DEC-3000s DVB-S device.

The particular code (below) checks the value in a register to
decide the tuning status -- hitherto one had to assume the device
was properly tuned and receiving signal and everything.

60 switch(result[3]) {
61 case 1: /* not tuned yet */
62 case 2: /* no signal/no lock*/
63 break;
64 case 3: /* signal found and locked*/
65 *status = FE_HAS_SIGNAL | FE_HAS_VITERBI |
66 FE_HAS_SYNC | FE_HAS_CARRIER | FE_HAS_LOCK;
67 break;
68 case 4:
69 *status = FE_TIMEDOUT;
70 break;
71 default:
72 pr_info("%s: returned unknown value: %d\n",
73 __func__, result[3]);
74 return -EIO;
75 }

Unfortunately, while this might work for the other flavours of
card (DVB-T?) supported by this code, in the case of my particular
device for DVB-S, the unknown value returned is 0 and does not
change regardless of tuning status.

I haven't checked whether a different register on my particular
device contains a usable value for the above code....

To: <free_beer_for_all@...>
Cc: <linux-kernel@...>, Alex Woods <linux-dvb@...>, Peter Beutner <p.beutner@...>, Mauro Carvalho Chehab <mchehab@...>
Date: Monday, June 23, 2008 - 3:37 am

These closed lists are a pain.

Lots of subprojects have moved their lists to vger.kernel.org in recent

Thanks. Cc's added.

--

To: Andrew Morton <akpm@...>
Cc: <free_beer_for_all@...>, <linux-kernel@...>, Alex Woods <linux-dvb@...>, Mauro Carvalho Chehab <mchehab@...>
Date: Monday, June 23, 2008 - 10:46 am

So only dvb-s support is broken. The DEC-2000t (hopefully) still works.
If there is still somebody around using that thing ;)

Indeed I added the above code only for the dec2000-t model. Since I have no
idea how that stuff works on the dec3000, probably the best thing to do is to
revert to the old behaviour for dec3000s models, i.e. just pretend we always have

well, i remember at least one report on the linux-dvb ml a long time ago
and iirc I have posted the same patch there, but it didn't help.
We never figured out how to make that particular dec3000s work.
And without hw documentation this is more like a puzzle game.
Which is rather demotivating.

Previous thread: [bad page] memcg: another bad page at page migration (2.6.26-rc5-mm3 + patch collection) by Daisuke Nishimura on Monday, June 23, 2008 - 1:53 am. (11 messages)

Next thread: 2.6.26-rc6: pcmcia stopped working after resume by Pavel Machek on Sunday, June 22, 2008 - 6:40 pm. (14 messages)