linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Marx <acc.for.news@gmail.com>
Cc: linux-media@vger.kernel.org
Subject: Re: pctv452e
Date: Fri, 06 Jul 2012 12:34:25 +0300	[thread overview]
Message-ID: <4FF6B121.6010105@iki.fi> (raw)
In-Reply-To: <r8cic9-ht4.ln1@wuwek.kopernik.gliwice.pl>

On 07/06/2012 09:13 AM, Marx wrote:
> On 05.07.2012 16:23, Antti Palosaari wrote:
>> Check if those modules are enabled, in file .config
>> CONFIG_DVB_USB_V2=m
>> CONFIG_DVB_USB_PCTV452E=m
>>
>> use make menuconfig to enable if disabled. Then make && make
>> install_modules && make install as usually.
>
> You were right, I didn't have this options anabled. I've enabled both
> via 'make menuconfig' and recompiled kernel. Driver loaded succesfully,
> I haven time to test only a few SD channels, and they were working, but
> some encrypted and HD didn't. I have to check yet why.

Did I missed something? PCTV device does not support CI/CAM and thus no 
support for encrypted channels. Is there still CI slot?

> Anyway when using card logs are full of i2c errors

Argh! But this must be issue of earlier driver too.

I debug it and it seems to be totally clueless implementation of 
stb6100_read_reg() as it sets device address like "device address + 
register address". This makes stb6100 I2C address of tuner set for that 
request 0x66 whilst it should be 0x60. Is that code never tested...

pctv452e DVB USB driver behaves just correctly as it says this is not 
valid and returns error.

Also pctv452e I2C adapter supports only I2C operations that are done 
with repeated STOP condition - but I cannot see there is logic to sent 
STOP after last message. I suspect it is not correct as logically but 
will work - very common mistake with many I2C adapters we have.

> Jul  6 07:04:19 wuwek kernel: [    6.087199] input: HDA ATI SB Line Out
> Front as /devices/pci0000:00/0000:00:14.2/sound/card
> 1/input8
> Jul  6 07:04:19 wuwek kernel: [    6.108046] stb6100_attach: Attaching
> STB6100
> Jul  6 07:04:19 wuwek kernel: [    6.108054] pctv452e_power_ctrl: 0
> Jul  6 07:04:19 wuwek kernel: [    6.108063] usb 1-4: dvb_usbv2: 'PCTV
> HDTV USB' successfully initialized and connected
> Jul  6 07:04:19 wuwek kernel: [    7.659462] Adding 2097148k swap on
> /dev/sda2.  Priority:-1 extents:1 across:2097148k
> Jul  6 07:04:19 wuwek kernel: [    7.707592] EXT4-fs (sda1): re-mounted.
> Opts: (null)
>
> (...)
>
> Jul  6 07:04:21 wuwek kernel: [   45.112483] Bluetooth: BNEP (Ethernet
> Emulation) ver 1.3
> Jul  6 07:04:21 wuwek kernel: [   45.112496] Bluetooth: BNEP filters:
> protocol multicast
> Jul  6 07:04:40 wuwek kernel: [   64.367411] pctv452e_power_ctrl: 1
> Jul  6 07:04:48 wuwek kernel: [   72.693972] I2C error -121; AA 0B  CC
> 00 01 -> 55 0B  CC 00 00.
> Jul  6 07:04:59 wuwek kernel: [   83.605643] I2C error -121; AA 49  CC
> 00 01 -> 55 49  CC 00 00.
> Jul  6 07:05:10 wuwek kernel: [   94.565805] I2C error -121; AA EE  CC
> 00 01 -> 55 EE  CC 00 00.
> Jul  6 07:05:10 wuwek kernel: [   94.578295] I2C error -121; AA 05  CC
> 00 01 -> 55 05  CC 00 00.
> Jul  6 07:05:10 wuwek kernel: [   94.637539] I2C error -121; AA 20  CC
> 00 01 -> 55 20  CC 00 00.
> Jul  6 07:05:18 wuwek kernel: [  102.525868] I2C error -121; AA 08  CC
> 00 01 -> 55 08  CC 00 00.
> Jul  6 07:05:18 wuwek kernel: [  102.538359] I2C error -121; AA 1F  CC
> 00 01 -> 55 1F  CC 00 00.
> Jul  6 07:05:18 wuwek kernel: [  102.597603] I2C error -121; AA 3A  CC
> 00 01 -> 55 3A  CC 00 00.
> Jul  6 07:05:29 wuwek kernel: [  113.765372] I2C error -121; AA F5  CC
> 00 01 -> 55 F5  CC 00 00.
> Jul  6 07:05:29 wuwek kernel: [  113.777986] I2C error -121; AA 0C  CC
> 00 01 -> 55 0C  CC 00 00.
> Jul  6 07:05:29 wuwek kernel: [  113.837480] I2C error -121; AA 27  CC
> 00 01 -> 55 27  CC 00 00.
> Jul  6 07:05:35 wuwek kernel: [  120.069153] I2C error -121; AA CF  CC
> 00 01 -> 55 CF  CC 00 00.
> Jul  6 07:05:37 wuwek kernel: [  121.325610] I2C error -121; AA A7  CC
> 00 01 -> 55 A7  CC 00 00.
> Jul  6 07:05:38 wuwek kernel: [  122.581565] I2C error -121; AA 7F  CC
> 00 01 -> 55 7F  CC 00 00.
> Jul  6 07:05:39 wuwek kernel: [  123.841526] I2C error -121; AA 57  CC
> 00 01 -> 55 57  CC 00 00.
> Jul  6 07:05:40 wuwek kernel: [  125.096979] I2C error -121; AA 2F  CC
> 00 01 -> 55 2F  CC 00 00.
> Jul  6 07:05:42 wuwek kernel: [  126.353689] I2C error -121; AA 07  CC
> 00 01 -> 55 07  CC 00 00.

Regardless of those errors it still works?

> I will test tonight when I have more time
> Marx

regards
Antti

-- 
http://palosaari.fi/



  reply	other threads:[~2012-07-06  9:34 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-04 16:04 pctv452e Steve Hill
2012-07-04 16:22 ` pctv452e Antti Palosaari
2012-07-04 18:46   ` pctv452e Marx
2012-07-04 18:53   ` pctv452e André Weidemann
2012-07-04 19:01     ` pctv452e Antti Palosaari
2012-07-05 13:14       ` pctv452e Marx
2012-07-05 14:23         ` pctv452e Antti Palosaari
2012-07-06  6:13           ` pctv452e Marx
2012-07-06  9:34             ` Antti Palosaari [this message]
2012-07-06 11:04               ` pctv452e Marx
2012-07-06 18:24                 ` pctv452e Antti Palosaari
2012-07-06 22:23                 ` pctv452e Marx
2012-07-07  0:00                   ` pctv452e Antti Palosaari
2012-07-07 23:07                     ` pctv452e Marx
2012-07-08 12:32                       ` pctv452e Antti Palosaari
2012-07-09  6:24                         ` pctv452e Marx
2012-07-09  8:42                           ` pctv452e Antti Palosaari
2012-07-09 16:44                             ` pctv452e Marx
2012-07-09 17:38                               ` pctv452e Antti Palosaari
2012-07-09 17:46                                 ` pctv452e Antti Palosaari
2012-07-10  6:39                                   ` pctv452e Marx
2012-07-10  9:33                                     ` pctv452e Antti Palosaari
2012-07-10 21:08                                       ` pctv452e Marx
2012-07-10 23:13                                         ` pctv452e Antti Palosaari
2012-07-11  6:25                                           ` pctv452e Marx
2012-07-11 10:00                                             ` pctv452e Antti Palosaari
2012-07-09 18:21                               ` pctv452e poma
2012-07-10  6:43                                 ` pctv452e Marx
2012-07-10 15:51                                   ` pctv452e poma
2012-07-10 15:58                                     ` pctv452e Devin Heitmueller
2012-07-10 22:54                                       ` pctv452e poma
2012-08-03 16:56   ` pctv452e Steve Hill
  -- strict thread matches above, loose matches on Subject: below --
2012-07-02 16:33 pctv452e Steve Hill
2012-07-03  7:46 ` pctv452e Marx
2012-07-03 23:04   ` pctv452e Antti Palosaari
2012-07-04  7:51     ` pctv452e Marx
2013-07-06 13:29 ` pctv452e Antti Palosaari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FF6B121.6010105@iki.fi \
    --to=crope@iki.fi \
    --cc=acc.for.news@gmail.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).