[linux-dvb] DVB-S2 / Multiproto and future modulation support
obi at linuxtv.org
Sat Aug 30 16:16:19 CEST 2008
Steven Toth wrote:
> If you feel that you want to support our movement then please help us by
> acking this email.
In general, I like your proposal.
Acked-by: Andreas Oberritter <obi at linuxtv.org>
Regarding the code:
1) What's TV_SEQ_CONTINUE good for? It seems to be unused.
2) Like Christophe I'd prefer to use DTV_ and dtv_ prefixes.
3) Did you mean p.u.qam.modulation below? Also, p.u.qam.fec_inner is
+ printk("%s() Preparing QAM req\n", __FUNCTION__);
+ /* TODO: Insert sanity code to validate a little. */
+ p.frequency = c->frequency;
+ p.inversion = c->inversion;
+ p.u.qam.symbol_rate = c->symbol_rate;
+ p.u.vsb.modulation = c->modulation;
4) About enum tv_cmd_types:
SYMBOLRATE -> SYMBOL_RATE?
INNERFEC -> INNER_FEC (or FEC)?
The Tone Burst command got lost (FE_DISEQC_SEND_BURST). How about
FE_ENABLE_HIGH_LNB_VOLTAGE got lost, too.
Which old ioctls should be considered as obsolete? Do you plan to add a
tv_cmd for every old ioctl?
More information about the linux-dvb