public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: BOUWSMA Barry <freebeer.bouwsma@gmail.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: PATCH: Query DVB frontend capabilities
Date: Fri, 11 Nov 2011 16:37:21 -0200	[thread overview]
Message-ID: <4EBD6B61.7020605@redhat.com> (raw)
In-Reply-To: <alpine.DEB.2.01.1111111759060.6676@localhost.localdomain>

Em 11-11-2011 15:43, BOUWSMA Barry escreveu:
> 
> On Do (Donnerstag) 10.Nov (November) 2011, 22:20,  Mauro Carvalho Chehab wrote:
> 
>> We should also think on a way to enumerate the supported values for each DVB $
>> the enum fe_caps is not enough anymore to store everything. It currently has $
>> filled (of a total of 32 bits), and we currently have:
>> 	12 code rates (including AUTO/NONE);
> 
> I'm probably not looking at the correct source, but the numbers
> seem to match, so I'll just note that in what I'm looking at,
> there are missing the values  1/3  and  2/5 .

Those were not added yet, as no driver currently uses it.

> 
> But I have to apologise in that I've also not been paying
> attention to this conversation, and haven't even been trying
> to follow recent developments.
> 
> 
>> 	13 modulation types;
> 
> Here I see missing  QAM1024  and  QAM4096 .

Same here.

> 
> 
>> 	7 transmission modes;
>> 	7 bandwidths;
> 
> Apparently DVB-C2 allows us any bandwidth from 8MHz to 450MHz,
> rather than the discrete values used by the other systems.
> If this is also applicable to other countries with 6MHz rasters,
> would it be necessary in addition to specify carrier spacing,
> either 2,232kHz or 1,674kHz as opposed to getting this from the
> channel bandwidth?

There are 3 parameters for Satellite and Cable systems:
	- Roll off factor;
	- Symbol Rate;
	- Bandwidth.

Only two of the tree are independent, as the spec defines:
	Bandwidth = symbol rate * (1  + roll off).

For DVB-C Annex A and C, roll off is fixed (0.15 and 0.13, respectively).

ITU-T J 83 Annex B doesn't say anything about it, but the ANSI SCTE07 spec
says that the roll-off is approx. 0.18 for 256-QAM and approx. 0.12 for
256-QAM.

DVB-S also has a fixed roll-off of 0.35, while DVB-S2 allows configuring it.

Not 100% sure, but ISDB-S also seems to use a per-modulation roll-off factor.

Anyway, when the roll-off is known, only symbol rate is needed, in order
to know the needed bandwidth.

IMHO, we should add some function inside the DVB core to calculate the
bandwidth for DVB-C (and DVB-C2), as the tuner saw filters require it,
in order to filter spurious frequencies from adjacent channels. Some
demods may also need such info.

The DVBv5 API doesn't impose any step for the carrier value, as it is
specified in Hz. So, I don't think that any change would be needed at 
the userspace API, in order to support DVB-C2 "continuous" carrier spacing.

> 
> 
>> 	8 guard intervals (including AUTO);
> 
> Here I observe the absence of  1/64 .

Same here: currently, no driver implements it.

> 
> 
>> 	5 hierarchy names;
>> 	4 rolloff's (probably, we'll need to add 2 more, to distinguish between$
> 
> 
> Of course, I'm just pointing out what I find, as I really don't
> know anything about the transport systems, and someone who 
> actually does might be able to say more, and correct my errors.
> 
> So just ignore me -- I'd rather see these values added sooner
> than later if needed.  Apparently the broadcasts from Borups
> Allé scheduled to start sometime around now will be switching
> over to use those mentioned to test their increased robustness.

Implementing those parameters is not a matter of just adding new stuff at
the core. Developers need DVB-C2 capable hardware, and access to a broadcaster
using it (or access to some testing facility where DVB-C2 could be
simulated).

Regards,
Mauro

  reply	other threads:[~2011-11-11 19:04 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-10 14:18 PATCH: Query DVB frontend capabilities Manu Abraham
2011-11-10 14:44 ` Andreas Oberritter
2011-11-10 15:30   ` Manu Abraham
2011-11-10 21:20     ` Mauro Carvalho Chehab
2011-11-11  6:26       ` Manu Abraham
2011-11-11 10:12         ` Mauro Carvalho Chehab
2011-11-11 22:07           ` Manu Abraham
2011-11-11 22:38             ` Mauro Carvalho Chehab
2011-11-12  3:36               ` Andreas Oberritter
2011-11-13 11:39                 ` Mauro Carvalho Chehab
2011-11-11 14:30         ` Andreas Oberritter
2011-11-11 14:43           ` Mauro Carvalho Chehab
2011-11-11 15:06             ` Andreas Oberritter
2011-11-11 17:14               ` Mauro Carvalho Chehab
2011-11-11 20:09                 ` Andreas Oberritter
2011-11-11 22:02                   ` Mauro Carvalho Chehab
2011-11-12  4:02                     ` Andreas Oberritter
2011-11-11 22:12           ` Manu Abraham
2011-11-11  9:55       ` FE_CAN-bits (was: Re: PATCH: Query DVB frontend capabilities) Patrick Boettcher
2011-11-11 10:21         ` FE_CAN-bits Mauro Carvalho Chehab
2011-11-11 11:36         ` FE_CAN-bits Antti Palosaari
2011-11-11 12:44           ` FE_CAN-bits Mauro Carvalho Chehab
2011-11-11 17:43       ` PATCH: Query DVB frontend capabilities BOUWSMA Barry
2011-11-11 18:37         ` Mauro Carvalho Chehab [this message]
2011-11-11 22:34           ` Manu Abraham
2011-11-13 13:32             ` Mauro Carvalho Chehab
2011-11-13 15:27               ` Manu Abraham
2011-11-14 11:47                 ` Mauro Carvalho Chehab
2011-11-14 15:02                   ` Manu Abraham
2011-11-14 16:39                     ` Mauro Carvalho Chehab
2011-11-14 17:09                       ` Manu Abraham
2011-11-14 18:08                         ` Mauro Carvalho Chehab
2011-11-14 18:30                           ` Manu Abraham
2011-11-14 18:42                             ` Mauro Carvalho Chehab
2011-11-14 18:59                               ` Manu Abraham
2011-11-14 20:31                                 ` Mauro Carvalho Chehab

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=4EBD6B61.7020605@redhat.com \
    --to=mchehab@redhat.com \
    --cc=freebeer.bouwsma@gmail.com \
    --cc=linux-media@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox