linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Hans-Frieder Vogt <hfvogt@gmx.net>
Cc: linux-media@vger.kernel.org
Subject: Re: [GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 => TerraTec Cinergy T Stick [0ccd:0093]
Date: Sun, 01 Apr 2012 15:53:45 +0300	[thread overview]
Message-ID: <4F784FD9.8080304@iki.fi> (raw)
In-Reply-To: <201204011227.18739.hfvogt@gmx.net>

Ooops forget to comment these...

On 01.04.2012 13:27, Hans-Frieder Vogt wrote:
> Your new firmware loader implies that the firmware is available as a type
> "download firmware". It seems this is indeed the case for most (all?) of the
> AF9035 devices. If the IT9135 driver should be merged, the "copy firmware"
> functionality (i.e. scatter write =>  command code 0x29) would need to be
> implemented.

Technically correct way is to split out all functionality, I mean USB 
-interface, demodulator and tuner, as own driver modules and use same 
modules for all the hardware not caring integration level. Those are 
just logical entities put to inside same silicon or sold separately.

For the existing IT9135 driver, it is there and surely will not be 
removed unless it goes unmaintained. Still it is nice to offer modular 
implementation as a choice for upcoming devices.

I haven't looked so carefully IT9135 FW download process - only thing I 
noticed it uses totally much smaller firmware than af9035, which is 
somehow weird since it contains tuner which is main difference between 
af9035 and it9135 :) Maybe there is done some bug fixing or other 
improvements in silicon level too.

It is easy to implement 2 or more firmware downloader, one for each 
firmware type and select correct by driver.

>> I extracted three firmwares from windows binaries I have. I will sent
>> those you, Michael, for testing. First, and oldest, is TUA9001, 2nd is
>> from FC0012 device and 3rd no idea.
>>
>> md5sum f71efe295151ba76cac2280680b69f3f
>> LINK=11.5.9.0 OFDM=5.17.9.1

That was from TerraTec Cinergy T -Stick, the stick which was one of the 
first af9035 and I originally made driver for.

>> md5sum 7cdc1e3aba54f3a9ad052dc6a29603fd
>> LINK=11.10.10.0 OFDM=5.33.10.0

Some DealExtreme cheap stick.

>> md5sum 862604ab3fec0c94f4bf22b4cffd0d89
>> LINK=12.13.15.0 OFDM=6.20.15.0
>>
> I have got a firmware converted from the linux driver version 1.0.28 for the
> AVermedia AVerTV A867R (07ca:1867, tuner: MaxLinear Mxl5007t)
> LINK=10.10.3.0 OFDM=4.21.6.251
> (only 2 firmwares each for link and ofdm!)
>
> For the same device the Windows driver verion 8.0.0.60 has a load of firmwares
> with
> LINK=11.15.10.0 OFDM=5.48.10.0
> (3 firmwares each for link and ofdm)
>
>  From the Terratec T6 dual tuner device (fc0012) Windows driver version
> 10.09.20.01 I have got (is probably the one that you have got as well)
> LINK=12.13.15.0 OFDM=6.20.15.0
> (3 firmwares each for link and ofdm)

True.

> I got an e-mail from Terratec support that they don't see a problem with the
> distribution of the firmware that I sniffed from the Terratec T6 running under
> Windows.
>
> The Terratec Windows driver contains 3 firmwares that get selected depending on
> the available tuner:
> firmware 1 for tuners TUA8010, TUA9001, TD1316AFIHP, TDA18271, TDA18291HN,
> XC3028L, XC4000, FC2580, FC0011, PICTOR, 0x40 (?), 0x43 (?)
> firmware 2 for tuner MT2266
> firmware 3 for tuners FC0012 and 0x37
>  From this I conclude that there is not a single generic firmware that we could
> use for the af9035 devices, but maybe 3 are sufficient.

We should gather enough information to decide used firmware naming. 
Mainly for that reason I didn't release get_dvb_firmware changes I used 
to extract those.
So do you have the idea about firmware naming?
Reading tuner ID from eeprom before firmware download is needed as you 
point out. And then select firmware based of that. Or is it possible to 
split firmwares even smaller blocks like LINK and OFDM and download 
those separately. I think it is OFDM version which says if tuner is 
supported or not.


regards
Antti
-- 
http://palosaari.fi/

  parent reply	other threads:[~2012-04-01 12:53 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-30 12:33 [GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 => TerraTec Cinergy T Stick [0ccd:0093] Antti Palosaari
2012-03-30 21:45 ` Michael Büsch
2012-03-30 22:00   ` Antti Palosaari
2012-03-30 22:14     ` Michael Büsch
2012-03-31 14:04       ` Michael Büsch
2012-03-31 14:28         ` Antti Palosaari
2012-03-31 16:29           ` Michael Büsch
2012-03-31 16:34             ` Antti Palosaari
2012-03-31 16:48               ` Antti Palosaari
2012-03-31 16:52                 ` Michael Büsch
2012-04-01  4:51                   ` Antti Palosaari
2012-04-01  8:33                     ` Michael Büsch
2012-04-01 12:19                       ` Michael Büsch
2012-04-01 12:29                         ` Antti Palosaari
2012-04-01 13:11                           ` Michael Büsch
2012-04-01 13:19                             ` Antti Palosaari
2012-04-01 14:42                             ` Hans-Frieder Vogt
2012-04-01 14:56                               ` Michael Büsch
2012-04-01 16:24                                 ` Hans-Frieder Vogt
2012-04-01 16:36                                   ` Michael Büsch
2012-04-01 16:15                             ` Michael Büsch
2012-04-01 16:20                               ` Antti Palosaari
2012-04-01 16:32                                 ` Michael Büsch
2012-04-01 16:39                                   ` Antti Palosaari
2012-04-01 16:44                                     ` Michael Büsch
2012-04-01  9:56                     ` Gianluca Gennari
2012-04-01 10:27                     ` Hans-Frieder Vogt
2012-04-01 12:31                       ` Antti Palosaari
2012-04-01 14:31                         ` Hans-Frieder Vogt
2012-04-01 15:00                           ` Antti Palosaari
2012-04-01 15:04                             ` Hans-Frieder Vogt
2012-04-01 12:53                       ` Antti Palosaari [this message]
2012-04-01 17:15                     ` [PATCH][GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 support for AverTV A867R (mxl5007t tuner) Hans-Frieder Vogt
2012-04-01 17:24                       ` Antti Palosaari
2012-04-01 18:11                         ` [PATCH][GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 i2c read fix Hans-Frieder Vogt
2012-04-01 18:16                           ` Antti Palosaari
2012-04-01 21:07                           ` [PATCH][GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 support for AverTV A867R (mxl5007t), version 2 Hans-Frieder Vogt
2012-04-01 21:19                             ` [PATCH] AF9033 read_ber and read_ucblocks implementation Hans-Frieder Vogt
2012-04-01 21:56                               ` Antti Palosaari
2012-04-01 22:11                                 ` Antti Palosaari
2012-04-01 21:32                             ` [PATCH][GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 support for AverTV A867R (mxl5007t), version 2 Antti Palosaari
2012-04-02 17:02                               ` Hans-Frieder Vogt
2012-03-31 16:49               ` [GIT PULL FOR 3.5] AF9035/AF9033/TUA9001 => TerraTec Cinergy T Stick [0ccd:0093] Michael Büsch
2012-04-06 11:11 ` Antti Palosaari
2012-05-07 18:44 ` [GIT PULL FOR 3.5] AF9035/AF9033 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=4F784FD9.8080304@iki.fi \
    --to=crope@iki.fi \
    --cc=hfvogt@gmx.net \
    --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).