public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Steven Toth <stoth@kernellabs.com>,
	Linux-Media <linux-media@vger.kernel.org>
Cc: Olli Salonen <olli.salonen@iki.fi>,
	Peter Faulkner-Ball <faulkner-ball@xtra.co.nz>
Subject: Re: [PATCH][media] SI2168: Resolve unknown chip version errors with different HVR22x5 models
Date: Thu, 04 Jun 2015 15:47:43 +0300	[thread overview]
Message-ID: <557048EF.3040703@iki.fi> (raw)
In-Reply-To: <CALzAhNW=Oei7_Nziozh3Mm+X_NNHvM5EdmPVPh9ajn5Aen9O2g@mail.gmail.com>

On 06/04/2015 03:38 PM, Steven Toth wrote:
> We're seeing a mix of SI2168 demodulators appearing on HVR2205 and
> HVR2215 cards, the chips are stamped with different build dates,
> verified on my cards.
>
> The si2168 driver detects some cards fine, others not at all. I can
> reproduce the working and non-working case. The fix, if we detect a
> newer card (D40) load the B firmware.
>
> This fix works well for me and properly enables DVB-T tuning behavior
> using tzap.
>
> Thanks to Peter Faulkner-Ball for describing his workaround.

hymm, I am not sure that patch at all. It is Olli who has been 
responsible adding support for multiple chip revisions, so I will leave 
that for Olli. I have only 2 Si2168 devices and both are B40 version.

Anyhow, for me it looks like firmware major version is always increased 
when new major revision of chip is made. Due to that I expected 5.0 
after B version 5.0.
A 1.0
A 2.0
A 3.0
B 4.0
C 5.0 ?
D 6.0 ?


And how we could explain situation Olli has device that had been working 
earlier, but now it does not? Could you Olli look back you old git tree 
and test if it still works? One possible reason could be also PCIe 
interface I2C adapter bug. Or timing issue.


regards
Antti


-- 
http://palosaari.fi/

  reply	other threads:[~2015-06-04 12:47 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-04 12:38 [PATCH][media] SI2168: Resolve unknown chip version errors with different HVR22x5 models Steven Toth
2015-06-04 12:47 ` Antti Palosaari [this message]
2015-06-04 12:57   ` Steven Toth
2015-06-04 12:57   ` Olli Salonen
2015-06-04 13:22     ` Olli Salonen
2015-06-04 13:49       ` Antti Palosaari
2015-06-04 14:03       ` Steven Toth
2015-06-05 13:40         ` Olli Salonen
2015-06-05 14:18           ` Antti Palosaari
2015-06-06  7:41             ` Olli Salonen
2015-06-11 18:21           ` Steven Toth

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=557048EF.3040703@iki.fi \
    --to=crope@iki.fi \
    --cc=faulkner-ball@xtra.co.nz \
    --cc=linux-media@vger.kernel.org \
    --cc=olli.salonen@iki.fi \
    --cc=stoth@kernellabs.com \
    /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