* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver [not found] <E1QHwSm-0006hA-A9@www.linuxtv.org> @ 2011-05-06 10:01 ` Andreas Oberritter 2011-05-06 10:42 ` Steve Kerrison 0 siblings, 1 reply; 8+ messages in thread From: Andreas Oberritter @ 2011-05-06 10:01 UTC (permalink / raw) To: linux-media; +Cc: Antti Palosaari, Mauro Carvalho Chehab On 05/05/2011 12:53 PM, Mauro Carvalho Chehab wrote: > + switch (priv->delivery_system) { > + case SYS_UNDEFINED: > + if (c->delivery_system == SYS_DVBT) { > + /* SLEEP => DVB-T */ > + ret = cxd2820r_set_frontend_t(fe, p); > + } else { > + /* SLEEP => DVB-T2 */ > + ret = cxd2820r_set_frontend_t2(fe, p); > + } > + break; > + case SYS_DVBT: > + if (c->delivery_system == SYS_DVBT) { > + /* DVB-T => DVB-T */ > + ret = cxd2820r_set_frontend_t(fe, p); > + } else if (c->delivery_system == SYS_DVBT2) { > + /* DVB-T => DVB-T2 */ > + ret = cxd2820r_sleep_t(fe); > + ret = cxd2820r_set_frontend_t2(fe, p); > + } > + break; > + case SYS_DVBT2: > + if (c->delivery_system == SYS_DVBT2) { Is this driver compilable? I don't see the necessary changes to linux/dvb/frontend.h to add SYS_DVBT2 in your tree. See below for a patch that I used for testing DVB-T2 internally. Regards, Andreas -- commit e89f95641f29b7a4457e7a68649f4374933e36a2 Author: Andreas Oberritter <obi@linuxtv.org> Date: Mon Mar 15 14:43:52 2010 +0100 DVB: Add basic API support for DVB-T2 and bump minor version Signed-off-by: Andreas Oberritter <obi@linuxtv.org> diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c index f5016ae..6f06efe 100644 --- a/drivers/media/dvb/dvb-core/dvb_frontend.c +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c @@ -1141,10 +1141,9 @@ static void dtv_property_adv_params_sync(struct dvb_frontend *fe) break; } - if(c->delivery_system == SYS_ISDBT) { - /* Fake out a generic DVB-T request so we pass validation in the ioctl */ - p->frequency = c->frequency; - p->inversion = c->inversion; + /* Fake out a generic DVB-T request so we pass validation in the ioctl */ + if ((c->delivery_system == SYS_ISDBT) || + (c->delivery_system == SYS_DVBT2)) { p->u.ofdm.constellation = QAM_AUTO; p->u.ofdm.code_rate_HP = FEC_AUTO; p->u.ofdm.code_rate_LP = FEC_AUTO; diff --git a/include/linux/dvb/frontend.h b/include/linux/dvb/frontend.h index 493a2bf..36a3ed6 100644 --- a/include/linux/dvb/frontend.h +++ b/include/linux/dvb/frontend.h @@ -175,14 +175,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; @@ -191,7 +197,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; @@ -305,7 +314,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 + +#define DTV_MAX_COMMAND DTV_DVBT2_PLP_ID typedef enum fe_pilot { PILOT_ON, @@ -337,6 +348,7 @@ typedef enum fe_delivery_system { SYS_DMBTH, SYS_CMMB, SYS_DAB, + SYS_DVBT2, } fe_delivery_system_t; struct dtv_cmds_h { diff --git a/include/linux/dvb/version.h b/include/linux/dvb/version.h index 5a7546c..1421cc8 100644 --- a/include/linux/dvb/version.h +++ b/include/linux/dvb/version.h @@ -24,6 +24,6 @@ #define _DVBVERSION_H_ #define DVB_API_VERSION 5 -#define DVB_API_VERSION_MINOR 2 +#define DVB_API_VERSION_MINOR 3 #endif /*_DVBVERSION_H_*/ ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver 2011-05-06 10:01 ` [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver Andreas Oberritter @ 2011-05-06 10:42 ` Steve Kerrison 2011-05-06 12:23 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 8+ messages in thread From: Steve Kerrison @ 2011-05-06 10:42 UTC (permalink / raw) To: Andreas Oberritter; +Cc: linux-media, Antti Palosaari, Mauro Carvalho Chehab Hi Andreas, >From cxd2820r_priv.h: > +/* > + * FIXME: These are totally wrong and must be added properly to the API. > + * Only temporary solution in order to get driver compile. > + */ > +#define SYS_DVBT2 SYS_DAB > +#define TRANSMISSION_MODE_1K 0 > +#define TRANSMISSION_MODE_16K 0 > +#define TRANSMISSION_MODE_32K 0 > +#define GUARD_INTERVAL_1_128 0 > +#define GUARD_INTERVAL_19_128 0 > +#define GUARD_INTERVAL_19_256 0 I believe Antti didn't want to make frontent.h changes until a consensus was reached on how to develop the API for T2 support. Regards, -- Steve Kerrison MEng Hons. http://www.stevekerrison.com/ On Fri, 2011-05-06 at 12:01 +0200, Andreas Oberritter wrote: > On 05/05/2011 12:53 PM, Mauro Carvalho Chehab wrote: > > + switch (priv->delivery_system) { > > + case SYS_UNDEFINED: > > + if (c->delivery_system == SYS_DVBT) { > > + /* SLEEP => DVB-T */ > > + ret = cxd2820r_set_frontend_t(fe, p); > > + } else { > > + /* SLEEP => DVB-T2 */ > > + ret = cxd2820r_set_frontend_t2(fe, p); > > + } > > + break; > > + case SYS_DVBT: > > + if (c->delivery_system == SYS_DVBT) { > > + /* DVB-T => DVB-T */ > > + ret = cxd2820r_set_frontend_t(fe, p); > > + } else if (c->delivery_system == SYS_DVBT2) { > > + /* DVB-T => DVB-T2 */ > > + ret = cxd2820r_sleep_t(fe); > > + ret = cxd2820r_set_frontend_t2(fe, p); > > + } > > + break; > > + case SYS_DVBT2: > > + if (c->delivery_system == SYS_DVBT2) { > > Is this driver compilable? I don't see the necessary changes to > linux/dvb/frontend.h to add SYS_DVBT2 in your tree. > > See below for a patch that I used for testing DVB-T2 internally. > > Regards, > Andreas > > -- > commit e89f95641f29b7a4457e7a68649f4374933e36a2 > Author: Andreas Oberritter <obi@linuxtv.org> > Date: Mon Mar 15 14:43:52 2010 +0100 > > DVB: Add basic API support for DVB-T2 and bump minor version > > Signed-off-by: Andreas Oberritter <obi@linuxtv.org> > > diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c > index f5016ae..6f06efe 100644 > --- a/drivers/media/dvb/dvb-core/dvb_frontend.c > +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c > @@ -1141,10 +1141,9 @@ static void dtv_property_adv_params_sync(struct dvb_frontend *fe) > break; > } > > - if(c->delivery_system == SYS_ISDBT) { > - /* Fake out a generic DVB-T request so we pass validation in the ioctl */ > - p->frequency = c->frequency; > - p->inversion = c->inversion; > + /* Fake out a generic DVB-T request so we pass validation in the ioctl */ > + if ((c->delivery_system == SYS_ISDBT) || > + (c->delivery_system == SYS_DVBT2)) { > p->u.ofdm.constellation = QAM_AUTO; > p->u.ofdm.code_rate_HP = FEC_AUTO; > p->u.ofdm.code_rate_LP = FEC_AUTO; > diff --git a/include/linux/dvb/frontend.h b/include/linux/dvb/frontend.h > index 493a2bf..36a3ed6 100644 > --- a/include/linux/dvb/frontend.h > +++ b/include/linux/dvb/frontend.h > @@ -175,14 +175,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; > > > @@ -191,7 +197,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; > > > @@ -305,7 +314,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 > + > +#define DTV_MAX_COMMAND DTV_DVBT2_PLP_ID > > typedef enum fe_pilot { > PILOT_ON, > @@ -337,6 +348,7 @@ typedef enum fe_delivery_system { > SYS_DMBTH, > SYS_CMMB, > SYS_DAB, > + SYS_DVBT2, > } fe_delivery_system_t; > > struct dtv_cmds_h { > diff --git a/include/linux/dvb/version.h b/include/linux/dvb/version.h > index 5a7546c..1421cc8 100644 > --- a/include/linux/dvb/version.h > +++ b/include/linux/dvb/version.h > @@ -24,6 +24,6 @@ > #define _DVBVERSION_H_ > > #define DVB_API_VERSION 5 > -#define DVB_API_VERSION_MINOR 2 > +#define DVB_API_VERSION_MINOR 3 > > #endif /*_DVBVERSION_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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver 2011-05-06 10:42 ` Steve Kerrison @ 2011-05-06 12:23 ` Mauro Carvalho Chehab 2011-05-06 12:36 ` Andreas Oberritter 0 siblings, 1 reply; 8+ messages in thread From: Mauro Carvalho Chehab @ 2011-05-06 12:23 UTC (permalink / raw) To: Steve Kerrison; +Cc: Andreas Oberritter, linux-media, Antti Palosaari Em 06-05-2011 07:42, Steve Kerrison escreveu: > Hi Andreas, > > From cxd2820r_priv.h: > >> +/* >> + * FIXME: These are totally wrong and must be added properly to the API. >> + * Only temporary solution in order to get driver compile. >> + */ >> +#define SYS_DVBT2 SYS_DAB >> +#define TRANSMISSION_MODE_1K 0 >> +#define TRANSMISSION_MODE_16K 0 >> +#define TRANSMISSION_MODE_32K 0 >> +#define GUARD_INTERVAL_1_128 0 >> +#define GUARD_INTERVAL_19_128 0 >> +#define GUARD_INTERVAL_19_256 0 > > > I believe Antti didn't want to make frontent.h changes until a consensus > was reached on how to develop the API for T2 support. Yeah. Andreas/Antti, It seems more appropriate to remove the above hack and add Andreas patch. I've reviewed it and it seemed ok on my eyes, provided that we also update DVB specs to reflect the changes. In special, the new DVB command should be documented: +#define DTV_DVBT2_PLP_ID 43 Thanks, Mauro ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver 2011-05-06 12:23 ` Mauro Carvalho Chehab @ 2011-05-06 12:36 ` Andreas Oberritter 2011-05-06 14:35 ` Antti Palosaari 0 siblings, 1 reply; 8+ messages in thread From: Andreas Oberritter @ 2011-05-06 12:36 UTC (permalink / raw) To: Mauro Carvalho Chehab; +Cc: Steve Kerrison, linux-media, Antti Palosaari On 05/06/2011 02:23 PM, Mauro Carvalho Chehab wrote: > Em 06-05-2011 07:42, Steve Kerrison escreveu: >> Hi Andreas, >> >> From cxd2820r_priv.h: >> >>> +/* >>> + * FIXME: These are totally wrong and must be added properly to the API. >>> + * Only temporary solution in order to get driver compile. >>> + */ >>> +#define SYS_DVBT2 SYS_DAB >>> +#define TRANSMISSION_MODE_1K 0 >>> +#define TRANSMISSION_MODE_16K 0 >>> +#define TRANSMISSION_MODE_32K 0 >>> +#define GUARD_INTERVAL_1_128 0 >>> +#define GUARD_INTERVAL_19_128 0 >>> +#define GUARD_INTERVAL_19_256 0 >> >> >> I believe Antti didn't want to make frontent.h changes until a consensus >> was reached on how to develop the API for T2 support. > > Yeah. > > Andreas/Antti, > > It seems more appropriate to remove the above hack and add Andreas patch. > I've reviewed it and it seemed ok on my eyes, provided that we also update > DVB specs to reflect the changes. > > In special, the new DVB command should be documented: > +#define DTV_DVBT2_PLP_ID 43 In addition to the patch, the PLP ID needs to be stored in struct dtv_frontend_properties and used by property cache functions in dvb_frontend.c. Antti, could you please complete the patch and test it with your device? This patch was adapted from an older kernel and only compile-tested few weeks ago. Regards, Andreas ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver 2011-05-06 12:36 ` Andreas Oberritter @ 2011-05-06 14:35 ` Antti Palosaari 0 siblings, 0 replies; 8+ messages in thread From: Antti Palosaari @ 2011-05-06 14:35 UTC (permalink / raw) To: Andreas Oberritter Cc: Mauro Carvalho Chehab, Steve Kerrison, linux-media, Antti Palosaari pe 6.5.2011 15:36 Andreas Oberritter kirjoitti: > On 05/06/2011 02:23 PM, Mauro Carvalho Chehab wrote: >> Em 06-05-2011 07:42, Steve Kerrison escreveu: >>> Hi Andreas, >>> >>> From cxd2820r_priv.h: >>> >>>> +/* >>>> + * FIXME: These are totally wrong and must be added properly to the >>>> API. >>>> + * Only temporary solution in order to get driver compile. >>>> + */ >>>> +#define SYS_DVBT2 SYS_DAB >>>> +#define TRANSMISSION_MODE_1K 0 >>>> +#define TRANSMISSION_MODE_16K 0 >>>> +#define TRANSMISSION_MODE_32K 0 >>>> +#define GUARD_INTERVAL_1_128 0 >>>> +#define GUARD_INTERVAL_19_128 0 >>>> +#define GUARD_INTERVAL_19_256 0 >>> >>> >>> I believe Antti didn't want to make frontent.h changes until a >>> consensus >>> was reached on how to develop the API for T2 support. >> >> Yeah. >> >> Andreas/Antti, >> >> It seems more appropriate to remove the above hack and add Andreas >> patch. >> I've reviewed it and it seemed ok on my eyes, provided that we also >> update >> DVB specs to reflect the changes. >> >> In special, the new DVB command should be documented: >> +#define DTV_DVBT2_PLP_ID 43 > > In addition to the patch, the PLP ID needs to be stored in struct > dtv_frontend_properties and used by property cache functions in > dvb_frontend.c. > > Antti, could you please complete the patch and test it with your device? > This patch was adapted from an older kernel and only compile-tested few > weeks ago. I can test it next week. I added also those first to the frontend.h but rebased that patch out before PULL request just since I didn't have enough time to make proper API for 2.6.40 schedule. It can be already found from my old GIT tree. As I looked DVB-T2 specs it was rather complex and large to learn. For example how to handle multiple PLPs (driver does not handle it currently). Maybe PLP_ID is enough for that. Antti ^ permalink raw reply [flat|nested] 8+ messages in thread
[parent not found: <20110506125542.ADA1D162E7@stevekerrison.com>]
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver [not found] <20110506125542.ADA1D162E7@stevekerrison.com> @ 2011-05-06 13:46 ` Antti Palosaari 2011-05-06 15:30 ` Mauro Carvalho Chehab 1 sibling, 0 replies; 8+ messages in thread From: Antti Palosaari @ 2011-05-06 13:46 UTC (permalink / raw) To: Steve Kerrison, Andreas Oberritter, Mauro Carvalho Chehab; +Cc: linux-media Steve or Andreas can do it. I am now on holiday weekend, snowboarding maybe last time for this season :) I cannot do much until next week. All help is welcome! Antti -- http://palosaari.fi/ ----- Original message ----- > If antti doesn't do this before me, I will look at this over the weekend > and generate a patch against antti's current code... if that's > appropriate of course (I'm new at this ;)) > > Regards, > Steve Kerrison. > > ----- Reply message ----- > From: "Andreas Oberritter" <obi@linuxtv.org> > Date: Fri, May 6, 2011 13:36 > Subject: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C > demodulator driver To: "Mauro Carvalho Chehab" <mchehab@redhat.com> > Cc: "Steve Kerrison" <steve@stevekerrison.com>, > <linux-media@vger.kernel.org>, "Antti Palosaari" <crope@iki.fi> > > > On 05/06/2011 02:23 PM, Mauro Carvalho Chehab wrote: > > Em 06-05-2011 07:42, Steve Kerrison escreveu: > > > Hi Andreas, > > > > > > From cxd2820r_priv.h: > > > > > > > +/* > > > > + * FIXME: These are totally wrong and must be added properly to > > > > the API. + * Only temporary solution in order to get driver > > > > compile. + */ > > > > +#define SYS_DVBT2 SYS_DAB > > > > +#define TRANSMISSION_MODE_1K 0 > > > > +#define TRANSMISSION_MODE_16K 0 > > > > +#define TRANSMISSION_MODE_32K 0 > > > > +#define GUARD_INTERVAL_1_128 0 > > > > +#define GUARD_INTERVAL_19_128 0 > > > > +#define GUARD_INTERVAL_19_256 0 > > > > > > > > > I believe Antti didn't want to make frontent.h changes until a > > > consensus was reached on how to develop the API for T2 support. > > > > Yeah. > > > > Andreas/Antti, > > > > It seems more appropriate to remove the above hack and add Andreas > > patch. I've reviewed it and it seemed ok on my eyes, provided that we > > also update DVB specs to reflect the changes. > > > > In special, the new DVB command should be documented: > > +#define DTV_DVBT2_PLP_ID 43 > > In addition to the patch, the PLP ID needs to be stored in struct > dtv_frontend_properties and used by property cache functions in > dvb_frontend.c. > > Antti, could you please complete the patch and test it with your device? > This patch was adapted from an older kernel and only compile-tested few > weeks ago. > > Regards, > Andreas > -- > 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 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver [not found] <20110506125542.ADA1D162E7@stevekerrison.com> 2011-05-06 13:46 ` Antti Palosaari @ 2011-05-06 15:30 ` Mauro Carvalho Chehab 2011-05-06 15:46 ` Mauro Carvalho Chehab 1 sibling, 1 reply; 8+ messages in thread From: Mauro Carvalho Chehab @ 2011-05-06 15:30 UTC (permalink / raw) To: Steve Kerrison; +Cc: Andreas Oberritter, linux-media, Antti Palosaari Em 06-05-2011 09:55, Steve Kerrison escreveu: > If antti doesn't do this before me, I will look at this over the weekend and generate a patch against antti's current code... if that's appropriate of course (I'm new at this ;)) Feel free to do it. I suspect that Antti won't work on it during this weekend. From what I understood, he's travelling in vacations. It helps if you could also add the bits into the frontend API DocBook: Documentation/DocBook/dvb/dvbproperty.xml The chapter that describes DVBv5 extensions is at: http://linuxtv.org/downloads/v4l-dvb-apis/FE_GET_PROPERTY.html As you may realize, this chapter were originally written with a description of the ISDB-T extensions. I've imported it from a .txt file, and did just some small adjustments to glue it into the specs, but I didn't have time to make it generic enough. So, the descriptions there for FE_[GET|SET]_PROPERTY are focused on ISDB-T. Yet, it shouldn't be hard to make it generic. I'm sending right now a patch I just made that will better document those two calls. I'm not adding the other non-documented properties. So, there are still lots of other DVBv5 property types not documented there (feel free to send us patches if you have time for that), and there are other missing changes from DVB APIv3 time, but at least we reduce a little bit the differences between the code and the spec [1]. In the past, the specs were shipped on a separate tree, using LaTex format, and outside the kernel. So, they weren't changed when new code were added. Now that we have it together with the kernel, we should only extend the API together with API specs, to avoid increasing the specs gap. With the specs at the kernel, all that needs to do to re-generate is to change the xml files and run: $ make htmldocs Then look at the specs using your favourite browser with an URL like: file:///home/myhome/media-tree/Documentation/DocBook/media/FE_GET_SET_PROPERTY.html [1] From DVB API v3 to v5, the differences are at the FE_[GET/SET]_PROPERTY ioctls. Thanks! Mauro. > > Regards, > Steve Kerrison. > > ----- Reply message ----- > From: "Andreas Oberritter" <obi@linuxtv.org> > Date: Fri, May 6, 2011 13:36 > Subject: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver > To: "Mauro Carvalho Chehab" <mchehab@redhat.com> > Cc: "Steve Kerrison" <steve@stevekerrison.com>, <linux-media@vger.kernel.org>, "Antti Palosaari" <crope@iki.fi> > > > On 05/06/2011 02:23 PM, Mauro Carvalho Chehab wrote: >> Em 06-05-2011 07:42, Steve Kerrison escreveu: >>> Hi Andreas, >>> >>> From cxd2820r_priv.h: >>> >>>> +/* >>>> + * FIXME: These are totally wrong and must be added properly to the API. >>>> + * Only temporary solution in order to get driver compile. >>>> + */ >>>> +#define SYS_DVBT2 SYS_DAB >>>> +#define TRANSMISSION_MODE_1K 0 >>>> +#define TRANSMISSION_MODE_16K 0 >>>> +#define TRANSMISSION_MODE_32K 0 >>>> +#define GUARD_INTERVAL_1_128 0 >>>> +#define GUARD_INTERVAL_19_128 0 >>>> +#define GUARD_INTERVAL_19_256 0 >>> >>> >>> I believe Antti didn't want to make frontent.h changes until a consensus >>> was reached on how to develop the API for T2 support. >> >> Yeah. >> >> Andreas/Antti, >> >> It seems more appropriate to remove the above hack and add Andreas patch. >> I've reviewed it and it seemed ok on my eyes, provided that we also update >> DVB specs to reflect the changes. >> >> In special, the new DVB command should be documented: >> +#define DTV_DVBT2_PLP_ID 43 > > In addition to the patch, the PLP ID needs to be stored in struct > dtv_frontend_properties and used by property cache functions in > dvb_frontend.c. > > Antti, could you please complete the patch and test it with your device? > This patch was adapted from an older kernel and only compile-tested few > weeks ago. > > Regards, > Andreas > -- > 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 > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver 2011-05-06 15:30 ` Mauro Carvalho Chehab @ 2011-05-06 15:46 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 8+ messages in thread From: Mauro Carvalho Chehab @ 2011-05-06 15:46 UTC (permalink / raw) To: Steve Kerrison; +Cc: Andreas Oberritter, linux-media, Antti Palosaari Em 06-05-2011 12:30, Mauro Carvalho Chehab escreveu: > Em 06-05-2011 09:55, Steve Kerrison escreveu: >> If antti doesn't do this before me, I will look at this over the weekend and generate a patch against antti's current code... if that's appropriate of course (I'm new at this ;)) > > Feel free to do it. I suspect that Antti won't work on it during this weekend. From > what I understood, he's travelling in vacations. > > It helps if you could also add the bits into the frontend API DocBook: > Documentation/DocBook/dvb/dvbproperty.xml > > The chapter that describes DVBv5 extensions is at: > http://linuxtv.org/downloads/v4l-dvb-apis/FE_GET_PROPERTY.html Just updated the above URL to reflect my last patch. Of course, feel free to review the patch and send comments/fixes as usual ;) Cheers, Mauro. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-05-06 15:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <E1QHwSm-0006hA-A9@www.linuxtv.org>
2011-05-06 10:01 ` [git:v4l-dvb/for_v2.6.40] [media] Sony CXD2820R DVB-T/T2/C demodulator driver Andreas Oberritter
2011-05-06 10:42 ` Steve Kerrison
2011-05-06 12:23 ` Mauro Carvalho Chehab
2011-05-06 12:36 ` Andreas Oberritter
2011-05-06 14:35 ` Antti Palosaari
[not found] <20110506125542.ADA1D162E7@stevekerrison.com>
2011-05-06 13:46 ` Antti Palosaari
2011-05-06 15:30 ` Mauro Carvalho Chehab
2011-05-06 15:46 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox