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