public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Christophe Thommeret <hftom@free.fr>
To: linux-dvb@linuxtv.org
Cc: Steven Toth <stoth@hauppauge.com>
Subject: Re: [linux-dvb] S2API - Status  - Thu Sep 11th
Date: Mon, 15 Sep 2008 04:14:41 +0200	[thread overview]
Message-ID: <200809150414.41360.hftom@free.fr> (raw)
In-Reply-To: <200809142004.06876.hftom@free.fr>

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 :)

-- 
Christophe Thommeret


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

  reply	other threads:[~2008-09-15  2:14 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 [this message]
2008-09-15  3:17                   ` Steven Toth
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=200809150414.41360.hftom@free.fr \
    --to=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