All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: "André Weidemann" <Andre.Weidemann@web.de>
Cc: Steve Hill <steve@nexusuk.org>, linux-media@vger.kernel.org
Subject: Re: pctv452e
Date: Wed, 04 Jul 2012 22:01:47 +0300	[thread overview]
Message-ID: <4FF4931B.7000708@iki.fi> (raw)
In-Reply-To: <4FF4911B.9090600@web.de>

On 07/04/2012 09:53 PM, André Weidemann wrote:
> On 04.07.2012 18:22, Antti Palosaari wrote:
>> On 07/04/2012 07:04 PM, Steve Hill wrote:
>>>  >> Ps. Steve, could you please give me full version of kernel which
>>>  >> works with pctv452e?
>>>
>>> I think it was 2.6.37-1-kirkwood from Debian which I was using (this is
>>> an ARM system).
>>>
>>>  > As the new DVB-USB fixes many bugs I ask you to test it. I converted
>>>  > pctv452e driver for you:
>>>  >
>>>  >
>>> http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/pctv452e
>>>
>>>  >
>>>  > Only PCTV device supported currently, not TechnoTrend at that time.
>>>
>>> Can I ask why it only works on the PCTV devices?  I was under the
>>> impression that the TechnoTrend hardware was identical?
>>>
>>>
>>> If you are able to provide any pointers as to where the TechnoTrend
>>> support is broken (or what debugging I should be turning on to figure
>>> out where it is broken) then that would be helpful.
>>
>> I don't have hardware, no PCTV neither TechnoTrend. I just converted
>> PCTV as Marx seems to have such device and he was blaming. Code wasn't
>> 100% similar, for example TechnoTrend has CI PCTV doesn't.
>>
>> It should not fix problems but it could since I fixed some nasty bugs.
>> Lets wait test report first and make decision what to do after that.
>
> The pctv452e and TT-connect S2-3600 are identical in hardware. Only USB
> IDs and remote control codes differ between the two. The Pinnacle box
> has its own remote. The TT-connect uses the same RC as the TT-budget
> series.
> The TT-connect S2-3650-CI has an additional CI slot.

OK. In addition to CI support and remote controller there was also 
.read_mac_address() callback implemented for TechnoTrend. But as 
.read_mac_address() is heavily optional and useless it could be possible 
driver author just left it unimplemented for PCTV - or PCTV does not has 
MAC address.

You seems to be marked as a one module author, I can guess you know reason?

Also if you has hardware, could you make some tests in order to get that 
driver fixed :]

regards
Antti

-- 
http://palosaari.fi/



  reply	other threads:[~2012-07-04 19:01 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-04 16:04 pctv452e Steve Hill
2012-07-04 16:22 ` pctv452e Antti Palosaari
2012-07-04 18:46   ` pctv452e Marx
2012-07-04 18:53   ` pctv452e André Weidemann
2012-07-04 19:01     ` Antti Palosaari [this message]
2012-07-05 13:14       ` pctv452e Marx
2012-07-05 14:23         ` pctv452e Antti Palosaari
2012-07-06  6:13           ` pctv452e Marx
2012-07-06  9:34             ` pctv452e Antti Palosaari
2012-07-06 11:04               ` pctv452e Marx
2012-07-06 18:24                 ` pctv452e Antti Palosaari
2012-07-06 22:23                 ` pctv452e Marx
2012-07-07  0:00                   ` pctv452e Antti Palosaari
2012-07-07 23:07                     ` pctv452e Marx
2012-07-08 12:32                       ` pctv452e Antti Palosaari
2012-07-09  6:24                         ` pctv452e Marx
2012-07-09  8:42                           ` pctv452e Antti Palosaari
2012-07-09 16:44                             ` pctv452e Marx
2012-07-09 17:38                               ` pctv452e Antti Palosaari
2012-07-09 17:46                                 ` pctv452e Antti Palosaari
2012-07-10  6:39                                   ` pctv452e Marx
2012-07-10  9:33                                     ` pctv452e Antti Palosaari
2012-07-10 21:08                                       ` pctv452e Marx
2012-07-10 23:13                                         ` pctv452e Antti Palosaari
2012-07-11  6:25                                           ` pctv452e Marx
2012-07-11 10:00                                             ` pctv452e Antti Palosaari
2012-07-09 18:21                               ` pctv452e poma
2012-07-10  6:43                                 ` pctv452e Marx
2012-07-10 15:51                                   ` pctv452e poma
2012-07-10 15:58                                     ` pctv452e Devin Heitmueller
2012-07-10 22:54                                       ` pctv452e poma
2012-08-03 16:56   ` pctv452e Steve Hill
  -- strict thread matches above, loose matches on Subject: below --
2012-07-02 16:33 pctv452e Steve Hill
2012-07-03  7:46 ` pctv452e Marx
2012-07-03 23:04   ` pctv452e Antti Palosaari
2012-07-04  7:51     ` pctv452e Marx
2013-07-06 13:29 ` pctv452e 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=4FF4931B.7000708@iki.fi \
    --to=crope@iki.fi \
    --cc=Andre.Weidemann@web.de \
    --cc=linux-media@vger.kernel.org \
    --cc=steve@nexusuk.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.