From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from stevekez.vm.bytemark.co.uk ([80.68.91.30]:53180 "EHLO stevekerrison.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755378Ab1EHTNO (ORCPT ); Sun, 8 May 2011 15:13:14 -0400 Subject: Re: [PATCH 6/6] Documentation: Update to include DVB-T2 additions From: Steve Kerrison To: Mauro Carvalho Chehab Cc: Antti Palosaari , linux-media@vger.kernel.org, Andreas Oberritter In-Reply-To: <4DC6C2DC.9010102@redhat.com> References: <4DC417DA.5030107@redhat.com> <1304869873-9974-7-git-send-email-steve@stevekerrison.com> <4DC6C2DC.9010102@redhat.com> Content-Type: text/plain; charset="UTF-8" Date: Sun, 08 May 2011 20:13:08 +0100 Message-ID: <1304881988.2920.18.camel@ares> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit List-ID: Sender: Mauro Carvalho Chehab Hi Mauro > + 3) DVB-T specifies 2K and 8K as valid sizes. > > + 4) DVB-T2 specifies 1K, 2K, 4K, 8K, 16K and 32K. > > It makes sense to add here that ISDB-T specifies 2K, 4K and 8K. > (yeah, sorry, it is my fault that I didn't notice it before ;) ) Actually note 1) in that list declares the sizes supported by ISDB-T; but the patch doesn't show it. So there is no blame to assign :) > -#define DTV_MAX_COMMAND DTV_ISDBS_TS_ID > > +#define DTV_DVBT2_PLP_ID 43 > > + > > Please document the PLP_ID as well. Just like ISDB-T, the best seems to > create a section with DVB-T2 specific parameters, and add this one there, > explaining its meaning. I have created a section for DVB-T2 parameters. It's within the main ISDB-T section. If that's not appropriate I guess it can be hauled out as it grows. However, much like Antti, I don't know much about PLP or the other features of the T2 specification, so cannot contribute a great deal yet. PLP_ID isn't used by the cxd2820r driver - it's simply specified in Andreas' API patch. Regards, -- Steve Kerrison MEng Hons. http://www.stevekerrison.com/ On Sun, 2011-05-08 at 13:20 -0300, Mauro Carvalho Chehab wrote: > Em 08-05-2011 12:51, Steve Kerrison escreveu: > > A few new capabilities added to frontend.h for DVB-T2. Added these > > to the documentation plus some notes explaining that they are > > used by the T2 delivery system. > > > > Signed-off-by: Steve Kerrison > > --- > > Documentation/DocBook/dvb/dvbproperty.xml | 21 ++++++++++++++++++--- > > Documentation/DocBook/dvb/frontend.h.xml | 20 ++++++++++++++++---- > > 2 files changed, 34 insertions(+), 7 deletions(-) > > > > diff --git a/Documentation/DocBook/dvb/dvbproperty.xml b/Documentation/DocBook/dvb/dvbproperty.xml > > index 05ce603..afe204c 100644 > > --- a/Documentation/DocBook/dvb/dvbproperty.xml > > +++ b/Documentation/DocBook/dvb/dvbproperty.xml > > @@ -217,9 +217,12 @@ get/set up to 64 properties. The actual meaning of each property is described on > > Bandwidth for the channel, in HZ. > > > > Possible values: > > + 1712000, > > + 5000000, > > 6000000, > > 7000000, > > - 8000000. > > + 8000000, > > + 10000000. > > > > > > Notes: > > @@ -231,6 +234,8 @@ get/set up to 64 properties. The actual meaning of each property is described on > > 4) Bandwidth in ISDB-T is fixed (6MHz) or can be easily derived from > > other parameters (DTV_ISDBT_SB_SEGMENT_IDX, > > DTV_ISDBT_SB_SEGMENT_COUNT). > > + 5) DVB-T supports 6, 7 and 8MHz. > > + 6) In addition, DVB-T2 supports 1.172, 5 and 10MHz. > > > > > >
> > @@ -257,6 +262,7 @@ typedef enum fe_delivery_system { > > SYS_DMBTH, > > SYS_CMMB, > > SYS_DAB, > > + SYS_DVBT2, > > } fe_delivery_system_t; > > > > > > @@ -273,7 +279,10 @@ typedef enum fe_transmit_mode { > > TRANSMISSION_MODE_2K, > > TRANSMISSION_MODE_8K, > > TRANSMISSION_MODE_AUTO, > > - TRANSMISSION_MODE_4K > > + TRANSMISSION_MODE_4K, > > + TRANSMISSION_MODE_1K, > > + TRANSMISSION_MODE_16K, > > + TRANSMISSION_MODE_32K, > > } fe_transmit_mode_t; > > > > > > @@ -284,6 +293,8 @@ typedef enum fe_transmit_mode { > > 2) If DTV_TRANSMISSION_MODE is set the TRANSMISSION_MODE_AUTO the > > hardware will try to find the correct FFT-size (if capable) and will > > use TMCC to fill in the missing parameters. > > + 3) DVB-T specifies 2K and 8K as valid sizes. > > + 4) DVB-T2 specifies 1K, 2K, 4K, 8K, 16K and 32K. > > It makes sense to add here that ISDB-T specifies 2K, 4K and 8K. > (yeah, sorry, it is my fault that I didn't notice it before ;) ) > >
> > > >
> > @@ -296,7 +307,10 @@ typedef enum fe_guard_interval { > > GUARD_INTERVAL_1_16, > > GUARD_INTERVAL_1_8, > > GUARD_INTERVAL_1_4, > > - GUARD_INTERVAL_AUTO > > + GUARD_INTERVAL_AUTO, > > + GUARD_INTERVAL_1_128, > > + GUARD_INTERVAL_19_128, > > + GUARD_INTERVAL_19_256, > > } fe_guard_interval_t; > > > > > > @@ -304,6 +318,7 @@ typedef enum fe_guard_interval { > > 1) If DTV_GUARD_INTERVAL is set the GUARD_INTERVAL_AUTO the hardware will > > try to find the correct guard interval (if capable) and will use TMCC to fill > > in the missing parameters. > > + 2) Intervals 1/128, 19/128 and 19/256 are used only for DVB-T2 at present > >
> > > > > > diff --git a/Documentation/DocBook/dvb/frontend.h.xml b/Documentation/DocBook/dvb/frontend.h.xml > > index d08e0d4..d792f78 100644 > > --- a/Documentation/DocBook/dvb/frontend.h.xml > > +++ b/Documentation/DocBook/dvb/frontend.h.xml > > @@ -176,14 +176,20 @@ typedef enum fe_transmit_mode { > > TRANSMISSION_MODE_2K, > > TRANSMISSION_MODE_8K, > > TRANSMISSION_MODE_AUTO, > > - TRANSMISSION_MODE_4K > > + TRANSMISSION_MODE_4K, > > + TRANSMISSION_MODE_1K, > > + TRANSMISSION_MODE_16K, > > + TRANSMISSION_MODE_32K, > > } fe_transmit_mode_t; > > > > typedef enum fe_bandwidth { > > BANDWIDTH_8_MHZ, > > BANDWIDTH_7_MHZ, > > BANDWIDTH_6_MHZ, > > - BANDWIDTH_AUTO > > + BANDWIDTH_AUTO, > > + BANDWIDTH_5_MHZ, > > + BANDWIDTH_10_MHZ, > > + BANDWIDTH_1_712_MHZ, > > } fe_bandwidth_t; > > > > > > @@ -192,7 +198,10 @@ typedef enum fe_guard_interval { > > GUARD_INTERVAL_1_16, > > GUARD_INTERVAL_1_8, > > GUARD_INTERVAL_1_4, > > - GUARD_INTERVAL_AUTO > > + GUARD_INTERVAL_AUTO, > > + GUARD_INTERVAL_1_128, > > + GUARD_INTERVAL_19_128, > > + GUARD_INTERVAL_19_256, > > } fe_guard_interval_t; > > > > > > @@ -306,7 +315,9 @@ struct dvb_frontend_event { > > > > #define DTV_ISDBS_TS_ID 42 > > > > -#define DTV_MAX_COMMAND DTV_ISDBS_TS_ID > > +#define DTV_DVBT2_PLP_ID 43 > > + > > Please document the PLP_ID as well. Just like ISDB-T, the best seems to > create a section with DVB-T2 specific parameters, and add this one there, > explaining its meaning. > > > +#define DTV_MAX_COMMAND DTV_DVBT2_PLP_ID > > > > typedef enum fe_pilot { > > PILOT_ON, > > @@ -338,6 +349,7 @@ typedef enum fe_delivery_system { > > SYS_DMBTH, > > SYS_CMMB, > > SYS_DAB, > > + SYS_DVBT2, > > } fe_delivery_system_t; > > > > struct dtv_cmds_h { > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html