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/
next prev parent 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