Re: [PATCH] Multi protocol support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Manu,

Manu Abraham wrote:
> +typedef enum {
> +       DMX_PROTO_DVB_S,
> +       DMX_PROTO_DVB_C,
> +       DMX_PROTO_DVB_T,
> +       DMX_PROTO_DSS,
> +       DMX_PROTO_ATSC,
> +       DMX_PROTO_DVB_S2
> +} dmx_protocol_t;

Why should a demux care how the TS was delivered? All DVB variants and
ATSC use MPEG-2 TS, right? I don't know about DSS though.

> +typedef enum {
> +       DMX_TRANSPORT,
> +       DMX_GENERIC_PACKET,
> +       DMX_GENERIC_CONTINUOUS,
> +       DMX_RESERVED
> +} dmx_bbheader_t;

What is this data type good for? It seems to be unused.

> +typedef enum {
> +       FE_PROTO_DVB_S,
> +       FE_PROTO_DVB_C,
> +       FE_PROTO_DVB_T,
> +       FE_PROTO_DSS,
> +       FE_PROTO_ATSC,
> +       FE_PROTO_DVB_S2
> +} fe_protocol_t;

I don't think that protocol is a proper name. In DVB terms S, C and T
are called delivery systems.

> +typedef enum {
> +       FE_TRANSPORT,
> +       FE_GENERIC_PACKET,
> +       FE_GENERIC_CONTINUOUS,
> +       FE_RESERVED
> +} fe_bbheader_t;

See dmx_bbheader_t.

IIRC it was discouraged to introduce new typedefs.

How does an program know which delivery systems are supported by a frontend?

Best regards,
Andreas

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________

linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux