All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <m.chehab@samsung.com>
To: Ralph Metzler <rjkm@metzlerbros.de>
Cc: Antti Palosaari <crope@iki.fi>, linux-media@vger.kernel.org
Subject: Re: DVB-C2
Date: Mon, 04 Nov 2013 10:10:41 -0200	[thread overview]
Message-ID: <20131104101041.48c5e782@samsung.com> (raw)
In-Reply-To: <21110.45135.982014.774220@morden.metzler>

Em Sun, 03 Nov 2013 21:21:35 +0100
Ralph Metzler <rjkm@metzlerbros.de> escreveu:

> Antti Palosaari writes:
>  > On 03.11.2013 13:31, Mauro Carvalho Chehab wrote:
>  > > Em Wed, 23 Oct 2013 00:57:47 +0200
>  > > Ralph Metzler <rjkm@metzlerbros.de> escreveu:
>  > >> I am wondering if anybody looked into API extensions for DVB-C2 yet?
>  > >> Obviously, we need some more modulations, guard intervals, etc.
>  > >> even if the demod I use does not actually let me set those (only auto).
>  > >>
>  > >> But I do need to set the PLP and slice ID.
>  > >> I currently set them (8 bit each) by combining them into the 32 bit
>  > >> stream_id (DTV_STREAM_ID parameter).
>  > >
>  > > I don't like the idea of combining them into a single field. One of the
>  > > reasons is that we may have endianness issues.
>  > >
>  > > So, IMHO, the better is to add a new property for slice ID.
>  > 
>  > I tried to understand what that data slice is. So what I understand, it 
>  > is layer to group PLPs, in order to get one wide OFDM channel as OFDM is 
>  > more efficient when channel bw increases.
>  > 
>  > So, in order to tune "stream" channel on DVB-C2 system, you *must* know 
>  > (in a order from radio channel to upper layers):
>  > frequency
>  > bandwidth
>  > slice ID
>  > PLP ID
>  > 
>  > Is that right?
> 
> Yes, if you do not want to parse L1 data you need the frequency of the slice,
> bandwidth, slice ID and PLP ID.
> If you parse L1 data, you do not need the slice ID because the PLP should be
> unique in one channel. 
> 
>  > I wonder if PLP IDs are defined so that there could not be overlapping 
>  > PLP IDs in a system... But if not, then defining slice ID is likely 
>  > needed. And if and when slice ID is needed to know before PLP ID, it is 
>  > even impossible to resolve slice ID from PLP ID.
> 
> See above, you can resolve it, but then you need to get the L1 data. 
> But PLPs can even be spread over several slices to get higher bandwidth 
> for one PLP. This is probably not used for broadcast TV though. You will
> also need one tuner/demod per slice then.
> 
> So, basically you only need any frequency for the "channel" (can be spread over 
> up to 450MHz, but avoid notches) and the bandwith.
> Tune until a L1 lock, get L1 data from demod (up to 4 KB), parse for the PLP
> id you want, get the corresponding slice (or slices), tune to the slice frequency
> with slice ID set and PLP id set and wait for a full lock ...

Ok, then it is really better to have slice as a separate property, and to
document the above procedure to tune into a slice at a DVB-C2 section
to be added to the DocBook.

We'll need to define a value for slice to mean "don't bind to any slice".
Maybe 2^32-1.

With regards to the slice property, it would be possible to let it
have multiple values (just like we do with ENUM_DELSYS). Not sure if 
this makes sense or not.

Regards,
Mauro

      reply	other threads:[~2013-11-04 12:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-22 17:14 [PATCH] Full DVB driver package for Earthsoft PT3 (ISDB-S/ISDB-T) cards Буди Романто
2013-10-22 22:57 ` DVB-C2 Ralph Metzler
2013-11-03 11:31   ` DVB-C2 Mauro Carvalho Chehab
2013-11-03 16:39     ` DVB-C2 Antti Palosaari
2013-11-03 20:21       ` DVB-C2 Ralph Metzler
2013-11-04 12:10         ` Mauro Carvalho Chehab [this message]

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=20131104101041.48c5e782@samsung.com \
    --to=m.chehab@samsung.com \
    --cc=crope@iki.fi \
    --cc=linux-media@vger.kernel.org \
    --cc=rjkm@metzlerbros.de \
    /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.