Re: DVBSky T330 DVB-C regression Linux 4.1.12 to 4.3

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

 



Moikka!

On 11/19/2015 01:36 AM, Stephan Eisvogel wrote:
Hey Olli, Antti,

culprit is:

http://git.linuxtv.org/cgit.cgi/linux.git/commit/drivers/media/dvb-frontends/si2168.c?id=7adf99d20ce0e96a70755f452e3a63824b14060f

I removed it like this:
         /* error bit set? */
/*
         if ((cmd->args[0] >> 6) & 0x01) {
             ret = -EREMOTEIO;
             goto err;
         }
*/

With this change backed out I zapped through about a 100 channels, and
my DVB stick works
again. Of course demodulator error handling would be nice anyhow. Beyond
my time budget
for now.

Surprising finding. Init succeeded already as firmware was downloaded so that error likely happens during si2168_set_frontend(). As set frontend is called once for each tuning request one failure should not cause more harm than one tuning failure. It could be nice to see which function is failing and if it fails repeatedly.

To see that, debug messages should be enabled:
modprobe si2168 dyndbg==pmftl
or
modprobe si2168; echo -n 'module si2168 =pft' > /sys/kernel/debug/dynamic_debug/control

You could also replace all dev_dbg with dev_info if you don't care compile kernel with dynamic debugs enabled needed for normal debug logging.

Also, you used 4.0.19 firmware. Could you test that old one:
http://palosaari.fi/linux/v4l-dvb/firmware/Si2168/Si2168-B40/4.0.11/

Unfortunately I don't have that device at all, so I cannot do much myself. It is more up to Olli :]

regards
Antti

--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux