From: Antti Palosaari <crope@iki.fi>
To: Olli Salonen <olli.salonen@iki.fi>
Cc: Stephen Allan <stephena@intellectit.com.au>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: Hauppauge WinTV-HVR2205 driver feedback
Date: Wed, 03 Jun 2015 18:34:08 +0300 [thread overview]
Message-ID: <556F1E70.7070507@iki.fi> (raw)
In-Reply-To: <CAAZRmGxby0r20HX6-MqmFBcJ1de3-Op0XHyO4QrErkZ0K3Om2Q@mail.gmail.com>
On 06/03/2015 12:29 PM, Olli Salonen wrote:
> I'm seeing the same issue as well. I thought that maybe some recent
> Si2168 changes did impact this, but it does not seem to be the case.
>
> I made a quick test myself. I reverted the latest si2168 patches one
> by one, but that did not remedy the situation. Anyway, the kernel log
> does not seem to indicate that the si2168_cmd_execute itself would
> fail (which is what happens after the I2C error handling patch in case
> the demod sets the error bit).
>
> olli@dl160:~/src/media_tree/drivers/media/dvb-frontends$ git log
> --oneline si2168.c
>
> d4b3830 Revert "[media] si2168: add support for gapped clock"
> eb62eb1 Revert "[media] si2168: add I2C error handling"
> 7adf99d [media] si2168: add I2C error handling
> 8117a31 [media] si2168: add support for gapped clock
> 17d4d6a [media] si2168: add support for 1.7MHz bandwidth
> 683e98b [media] si2168: return error if set_frontend is called with invalid para
> c32b281 [media] si2168: change firmware variable name and type
> 9b7839c [media] si2168: print chip version
>
> dmesg lines when it fails (this is with a card that has worked before):
>
> [66661.336898] saa7164[0]: registered device video0 [mpeg]
> [66661.567295] saa7164[0]: registered device video1 [mpeg]
> [66661.778660] saa7164[0]: registered device vbi0 [vbi]
> [66661.778817] saa7164[0]: registered device vbi1 [vbi]
> [66675.175508] si2168:si2168_init: si2168 2-0064:
> [66675.187299] si2168:si2168_cmd_execute: si2168 2-0064: cmd execution took 6 ms
> [66675.194105] si2168:si2168_cmd_execute: si2168 2-0064: cmd execution
> took 2 ms [OLLI: The result of this I2C cmd must be bogus]
> [66675.194110] si2168 2-0064: unknown chip version Si2168-
> [66675.200244] si2168:si2168_init: si2168 2-0064: failed=-22
> [66675.213020] si2157 0-0060: found a 'Silicon Labs Si2157-A30'
> [66675.242856] si2157 0-0060: firmware version: 3.0.5
Okei, so it has been working earlier... Could you enable I2C debugs to
see what kind of data that command returns?
What I suspect in first hand is that Windows driver has downloaded
firmware to chip and linux driver does it again, but with incompatible
firmware, which leads to situation it starts failing. But if that is
issue you likely already noted it.
regards
Antti
--
http://palosaari.fi/
next prev parent reply other threads:[~2015-06-03 15:34 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-03 3:55 Hauppauge WinTV-HVR2205 driver feedback Stephen Allan
2015-06-03 7:55 ` Antti Palosaari
2015-06-03 8:02 ` Antti Palosaari
2015-06-03 9:29 ` Olli Salonen
2015-06-03 14:30 ` Steven Toth
2015-06-04 12:46 ` Steven Toth
2015-06-03 15:34 ` Antti Palosaari [this message]
2015-06-03 19:08 ` Olli Salonen
2015-06-03 19:46 ` Antti Palosaari
2015-06-04 8:07 ` Olli Salonen
2015-06-04 0:38 ` Stephen Allan
2015-06-04 1:22 ` faulkner-ball
2015-06-03 14:31 ` Steven Toth
2015-06-03 21:50 ` Peter Faulkner-Ball
2015-06-03 22:16 ` Steven Toth
-- strict thread matches above, loose matches on Subject: below --
2015-10-04 4:55 Richard Tresidder
2015-10-04 14:03 ` Steven Toth
2015-10-05 1:59 ` Richard Tresidder
2015-10-05 14:22 ` Steven Toth
2015-10-05 14:35 ` Richard Tresidder
2015-10-05 14:39 ` Richard Tresidder
2015-10-05 14:43 ` Steven Toth
2015-10-05 15:26 ` Richard Tresidder
2015-10-05 15:32 ` Steven Toth
2015-10-05 12:45 ` Tycho Lürsen
2015-10-05 14:22 ` Richard Tresidder
2015-10-05 16:03 ` Richard Tresidder
2015-10-11 6:18 ` Richard Tresidder
2015-10-11 7:33 ` Tycho Lürsen
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=556F1E70.7070507@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=olli.salonen@iki.fi \
--cc=stephena@intellectit.com.au \
/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).