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

[Rolf Evers-Fischer]

> Antti Palosaari  hat am 21. Mai 2016 um 02:16 geschrieben:
> 
> Reason is that silicon vendor has changed firmware API somewhere between 
> 4.0.11 and 4.0.19 so that newer firmwares will put device to full sleep 
> which causes even device firmware lost.
> 
> Fix is here, and I hope I can push it to 4.8 - it will took about half 
> year from this day until it is on mainline (it is not regression so I 
> cannot send it to older kernels and for 4.7 it is too late). Before that 
> just use 4.0.11 firmware.
> 
> https://git.linuxtv.org/anttip/media_tree.git/commit/?h=mygica=cfd6ab8e840815eb54eb777c9f64807022ba922c
> 

Thank you for the fix. I can confirm that it works with 4.0.11 and 4.0.19
firmware. Let's hope that you can push it at least to 4.8.

Kind regards,
 Rolf
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Antti Palosaari]

On 05/21/2016 01:47 AM, Rolf Evers-Fischer wrote:

Dear Antti,
I apologize for tackling this old problem, but I just ran into the same
situation with my "DVBSky T330 DVB-C" and found that I'm not the only one.

.

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/



I've just tried the old 4.0.11 firmware - and the error is gone. Now the tuning
works perfectly!


Reason is that silicon vendor has changed firmware API somewhere between 
4.0.11 and 4.0.19 so that newer firmwares will put device to full sleep 
which causes even device firmware lost.


Fix is here, and I hope I can push it to 4.8 - it will took about half 
year from this day until it is on mainline (it is not regression so I 
cannot send it to older kernels and for 4.7 it is too late). Before that 
just use 4.0.11 firmware.


https://git.linuxtv.org/anttip/media_tree.git/commit/?h=mygica=cfd6ab8e840815eb54eb777c9f64807022ba922c

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

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

[Rolf Evers-Fischer]

Dear Antti,
I apologize for tackling this old problem, but I just ran into the same
situation with my "DVBSky T330 DVB-C" and found that I'm not the only one.

Antti Palosaari  writes:

> 
> 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

Reverting this commit helps, but is not very convenient.

> 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.
> 

Dynamic debug didn't work properly on my system. I'll replace all dev_dbg with
dev_info and provide you the output as soon as possible, if you are still
interested.

> 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/
> 

I've just tried the old 4.0.11 firmware - and the error is gone. Now the tuning
works perfectly!

Best regards,
 Rolf
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Antti Palosaari]

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Olli Salonen]

Hi Stephan,

I had a look at recent changes to si2168 and dvb-usb-dvbsky drivers
that are the ones most relevant here. si2157 is probably not the issue
here as your demod does lock.

si2168:

2015-08-11[media] dvb-frontends: Drop owner assignment from i2c_driver
2015-06-09[media] dvb: Get rid of typedev usage for enums
2015-06-05[media] si2168: Implement own I2C adapter locking

dvb-usb-dvbsky:

2015-10-03[media] Add Terratec H7 Revision 4 to DVBSky driver
2015-06-10[media] TS2020: Calculate tuner gain correctly
2015-06-09[media] dvb: Get rid of typedev usage for enums

All these changes seem rather innocent. I've got the same tuner (sold
as a TechnoTrend device) and I could try the 4.3 kernel to confirm
it's still ok for DVB-T/T2 broadcasts that I've got available here,
but it'll be a while as I'm travelling at the moment. Hopefully
someone else can confirm before that.

Cheers,
-olli


On 8 November 2015 at 02:37, Stephan Eisvogel  wrote:
> Hi Antti, hi Olli,
>
> I'm a Raspberry Pi 2 + TVHeadend + DVBSky T330 clone owner/user and am
> observing
> a regression from Linux 4.1.12 to 4.3. TVheadend is delivering this error:
>
> Nov 04 19:32:17 RPI2 tvheadend[714]: mpegts: 450MHz in Kabel Deutschland -
> scan no data, failed
>
> Basically EPG and TV is full dead. I saw the
>
>   "9cd700e m88ds3103: use own update_bits() implementation"
>   "a8e2219 m88ds3103: use regmap for I2C register access"
>
> bits that went into 4.3 recently. Anything like that applicable to the
> Si2168/Si2157 USB varieties?
>
> Device:
>
> Bus 001 Device 005: ID 0572:0320 Conexant Systems (Rockwell), Inc. DVBSky
> T330 DVB-T2/C tuner
>
> Relevant dmesg:
>
> [2.063240] usb 1-1.2: new high-speed USB device number 5 using dwc_otg
> [2.157399] usb 1-1.2: New USB device found, idVendor=0572,
> idProduct=0320
> [2.157422] usb 1-1.2: New USB device strings: Mfr=1, Product=2,
> SerialNumber=3
> [2.157433] usb 1-1.2: Product: DVB-T2/C USB-Stick
> [2.157443] usb 1-1.2: Manufacturer: Bestunar Inc
> [2.157453] usb 1-1.2: SerialNumber: 20140126
> [6.273255] usb 1-1.2: dvb_usb_v2: found a 'DVBSky T330' in warm state
> [6.273832] usb 1-1.2: dvb_usb_v2: will pass the complete MPEG2 transport
> stream to the software demuxer
> [6.273921] DVB: registering new adapter (DVBSky T330)
> [6.275286] usb 1-1.2: dvb_usb_v2: MAC address: 00:cc:10:a5:33:0c
> [6.474110] i2c i2c-3: Added multiplexed i2c bus 4
> [6.474138] si2168 3-0064: Silicon Labs Si2168 successfully attached
> [6.680772] si2157 4-0060: Silicon Labs Si2147/2148/2157/2158
> successfully attached
> [6.680835] usb 1-1.2: DVB: registering adapter 0 frontend 0 (Silicon
> Labs Si2168)...
> [6.933236] Registered IR keymap rc-dvbsky
> [6.933705] input: DVBSky T330 as
> /devices/platform/soc/3f98.usb/usb1/1-1/1-1.2/rc/rc0/input4
> [6.933972] rc0: DVBSky T330 as
> /devices/platform/soc/3f98.usb/usb1/1-1/1-1.2/rc/rc0
> [6.933998] usb 1-1.2: dvb_usb_v2: schedule remote query interval to 300
> msecs
> [6.934013] usb 1-1.2: dvb_usb_v2: 'DVBSky T330' successfully initialized
> and connected
> [6.934146] usbcore: registered new interface driver dvb_usb_dvbsky
> [   15.066829] si2168 3-0064: found a 'Silicon Labs Si2168-B40'
> [   15.086772] si2168 3-0064: downloading firmware from file
> 'dvb-demod-si2168-b40-01.fw'
> [   16.568679] si2168 3-0064: firmware version: 4.0.19
> [   16.579069] si2157 4-0060: found a 'Silicon Labs Si2158-A20'
> [   16.599232] si2157 4-0060: downloading firmware from file
> 'dvb-tuner-si2158-a20-01.fw'
> [   17.581060] si2157 4-0060: firmware version: 2.1.6
> [   17.581145] usb 1-1.2: DVB: adapter 0 frontend 0 frequency 0 out of range
> (5500..86200)
>
>
> Thanks for any insight!
>
> Best regards from Nuremberg/Germany,
> Stephan
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html