linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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/

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