Re: [PATCH 01/12] dvb-frontends: Support for DVB-C2 to DVB frontends

[Maik Broemme <mbroemme@parallels.com>]

Hi Ralph,

Ralph Metzler <rjkm@metzlerbros.de> wrote:
> Mauro Carvalho Chehab writes:
>  > Em Sun, 3 Nov 2013 01:24:25 +0100
>  > Maik Broemme <mbroemme@parallels.com> escreveu:
>  > 
>  > > Added support for DVB-C2 to DVB frontends. It will be required
>  > > by cxd2843 and tda18212dd (Digital Devices) frontends.
>  > > 
>  > > Signed-off-by: Maik Broemme <mbroemme@parallels.com>
>  > > ---
>  > >  include/uapi/linux/dvb/frontend.h | 1 +
>  > >  1 file changed, 1 insertion(+)
>  > > 
>  > > diff --git a/include/uapi/linux/dvb/frontend.h b/include/uapi/linux/dvb/frontend.h
>  > > index c56d77c..98648eb 100644
>  > > --- a/include/uapi/linux/dvb/frontend.h
>  > > +++ b/include/uapi/linux/dvb/frontend.h
>  > > @@ -410,6 +410,7 @@ typedef enum fe_delivery_system {
>  > >  	SYS_DVBT2,
>  > >  	SYS_TURBO,
>  > >  	SYS_DVBC_ANNEX_C,
>  > > +	SYS_DVBC2,
>  > >  } fe_delivery_system_t;
>  > >  
>  > >  /* backward compatibility */
>  > 
>  > Please update also the documentation, at Documentation/DocBook/media/dvb.
>  > 
>  > Doesn't DVB-C2 provide any newer property? If so, please add it there as
>  > well, and at frontend.h.
>  > 
> 
> I asked about this on linux-media a week or so ago. The main question was
> concerning STREAM_ID. I asked if it would be fine to combine PLP and
> slice id (each 8 bit) into stream_id or if there should be a separate 
> new property. And for which one, PLP or slice id? 
> Probably slice id, because stream_id is also used for PLP in T2?
> I combined them into stream_id for now (but that was after the 0.9.10 version
> of the dddvb package).
> 

Do you have a patch ready at this time? I'm asking because CXD2843ER
depends on DVB-C2 changes.

> There are also many new qam types, etc. but, as I said back then, it was not  
> urgent for me to add those because the Sony demod does not allow setting those.
> At least it is not documented how to do it.
> 
> 
> Regards,
> Ralph
> --
> 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

--Maik