public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Steven Toth <stoth@linuxtv.org>
To: Christophe Thommeret <hftom@free.fr>
Cc: Steven Toth <stoth@hauppauge.com>, linux-dvb@linuxtv.org
Subject: Re: [linux-dvb] S2API - Status  - Thu Sep 11th
Date: Sun, 14 Sep 2008 23:17:02 -0400	[thread overview]
Message-ID: <48CDD3AE.80506@linuxtv.org> (raw)
In-Reply-To: <200809150414.41360.hftom@free.fr>

Christophe Thommeret wrote:
> Le Sunday 14 September 2008 20:04:06 Christophe Thommeret, vous avez écrit :
>> Le Sunday 14 September 2008 18:47:00 Steven Toth, vous avez écrit :
>>> Steven Toth wrote:
>>>> Johannes Stezenbach wrote:
>>>>> On Fri, Sep 12, 2008, Steven Toth wrote:
>>>>>> Christophe Thommeret wrote:
>>>>>>> As far  as i understand, the cinergyT2 driver is a bit unusual, e.g.
>>>>>>> dvb_register_frontend is never called (hence no dtv_* log messages).
>>>>>>> I don't know if there is others drivers like this, but this has to
>>>>>>> be investigated cause rewritting all drivers for S2API could be a
>>>>>>> bit of work :)
>>>>>>>
>>>>>>> P.S.
>>>>>>> I think there is an alternate driver for cinergyT2 actually in
>>>>>>> developement but idon't remember where it's located neither its
>>>>>>> state.
>>>>>> Good to know. (I also saw your followup email). I have zero
>>>>>> experience with the cinergyT2 but the old api should still be working
>>>>>> reliably. I plan to investigate this, sounds like a bug! :)
>>>>> Holger was of the opinion that having the demux in dvb-core
>>>>> was stupid for devices which have no hw demux, so he
>>>>> programmed around dvb-core. His plan was to add a
>>>>> mmap-dma-buffers kind of API to the frontend device,
>>>>> but it never got implemented.
>>>>>
>>>>> Anyway, it's bad if one driver is different than all the others.
>>>> Hmm, I didn't realize this, good to know.
>>>>
>>>> Now it's peaked my interest, I'll have to look at the code.
>>>>
>>>> The existing API should still work at a bare minimum, if it's not - it
>>>> needs to.
>>> So I looked the the cinergyT2 code, that's a complete eye-opener. It has
>>> it's own ioctl handler, outside of dvb-core.
>>>
>>> It's a good news / bad news thing.
>>>
>>> The good news is that this driver will not be effected by the S2API
>>> changes, so nothing can break.
>>>
>>> The bad news is that this driver will not be effected by the S2API
>>> changes, so it doesn't get the benefit.
>>>
>>> Regardless of S2API or multiproto, I see no reason why we shouldn't
>>> bring this driver back into dvb-core.
>>>
>>> I don't have a device to test, but here's a patch (0% tested, with bugs
>>> probably) that converts the module back to a regular dvb-core compatible
>>> device, so the S2API would work with this. If anyone wants to test this,
>>> and finds bugs - I won't get back to this driver for a couple of weeks -
>>> so your patches would be welcome. :)
>>>
>>> Frankly, is S2API is selected for merge and we have enough users of the
>>> current non-dvb-core driver, I'll probably re-write it from the spec.
>>>
>>> So much to do, so little time.
>>>
>>> - Steve
>> Steve,
>>
>> as you expected, it's abit buggy and crashes at load ;)
>> i've searched for the alternative cinergyT2 driver.
>> it's available at: http://linuxtv.org/hg/~tmerle/cinergyT2/
>>
>> haven't tried it yet but will do (at least several people seems to have it
>> working).
> 
> Have it working, not as good as Holger' one (often fails to lock) but it 
> works, with both old and new api.
> So, one problem less :)
> 

Good to know Christophe, thanks.

- Steve


_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

  reply	other threads:[~2008-09-15  3:17 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-12  5:51 [linux-dvb] S2API - Status - Thu Sep 11th Steven Toth
2008-09-12  6:26 ` Christophe Thommeret
2008-09-12 13:18   ` Steven Toth
2008-09-12 13:29     ` Christophe Thommeret
2008-09-12 13:31       ` Christophe Thommeret
2008-09-12 14:08       ` Steven Toth
2008-09-12 15:27         ` Johannes Stezenbach
2008-09-12 16:17           ` Steven Toth
2008-09-14 16:47             ` Steven Toth
2008-09-14 18:04               ` Christophe Thommeret
2008-09-15  2:14                 ` Christophe Thommeret
2008-09-15  3:17                   ` Steven Toth [this message]
2008-09-15 17:08                     ` Igor M. Liplianin
2008-09-15 20:02                       ` Gregoire Favre
2008-09-15 20:14                         ` Igor M. Liplianin
     [not found]                       ` <48CE9E22.9060405@hauppauge.com>
2008-09-16 21:37                         ` [linux-dvb] [PATCH] S2API - Silicon Labs si2109/2110 demodulator support Igor M. Liplianin
2008-09-17 22:27                           ` [linux-dvb] [PATCH] S2API - Add support for USB card modification with si2109/2110 demodulator Igor M. Liplianin
2008-09-22  4:09                             ` Steven Toth
2008-09-22  4:07                           ` [linux-dvb] [PATCH] S2API - Silicon Labs si2109/2110 demodulator support Steven Toth
2008-09-23 19:01                           ` [linux-dvb] [PATCH] S2API - ST stv0288 " Igor M. Liplianin
2008-09-23 19:04                             ` Igor M. Liplianin
2008-09-23 19:09                               ` Steven Toth
2008-09-24  1:38                               ` Steven Toth
2008-09-25  2:30                             ` [linux-dvb] [PATCH] S2API - Kconfig dependency fixes for cards with stv0288 and si21xx Igor M. Liplianin
2008-09-15 11:43                   ` [linux-dvb] S2API - Status - Thu Sep 11th Johannes Stezenbach
2008-09-16  2:58                     ` Christophe Thommeret
2008-09-16 21:24                       ` Johannes Stezenbach
     [not found]   ` <48CC12BF.7050803@hauppauge.com>
2008-09-14 14:46     ` Christophe Thommeret
2008-09-14 14:56       ` Christophe Thommeret
2008-09-14 16:49         ` Steven Toth
2008-09-14 15:01       ` Steven Toth
2008-09-14 15:27         ` Christophe Thommeret
2008-09-12  8:00 ` [linux-dvb] Siano ISDB [was: Re: S2API - Status - Thu Sep 11th] barry bouwsma
2008-09-12  8:24   ` [linux-dvb] Siano ISDB barry bouwsma
2008-09-12 11:47     ` Uri Shkolnik
     [not found]   ` <48CA6BBB.5010802@hauppauge.com>
2008-09-12 13:33     ` [linux-dvb] Siano ISDB [was: Re: S2API - Status - Thu Sep 11th] barry bouwsma
2008-09-12 14:13       ` Steven Toth
2008-09-12 14:38         ` barry bouwsma
2008-09-12 18:01     ` Michael Krufky
2008-09-12  9:17 ` [linux-dvb] S2API - Status - Thu Sep 11th Patrick Boettcher
2008-09-12 14:04   ` Steven Toth
2008-09-13 20:02   ` Steven Toth
2008-09-14 12:36     ` Hans Werner
2008-09-14 14:40       ` Steven Toth
2008-09-12 11:09 ` Hans Werner

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=48CDD3AE.80506@linuxtv.org \
    --to=stoth@linuxtv.org \
    --cc=hftom@free.fr \
    --cc=linux-dvb@linuxtv.org \
    --cc=stoth@hauppauge.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