Re: [linux-dvb] DVB-S2 / Multiproto and future modulation support

[Steven Toth <stoth@linuxtv.org>]

Andreas Oberritter wrote:
> 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@linuxtv.org>

Andreas, thank you for your support.

> 
> 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
> missing.
> 
> +		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
> TV_SET_TONE_BURST?
> 
> 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?

I'm collecting all of the feedback, we have lots of comments and change 
suggests - but largely we're heading in a good direction.

You've pointed out some obvious missing pieces (the new s2 patch was 
written in 12 hours - so it hasn't had the time multiproto had to be 
developers), so we're going to have to fill in some missing pieces.

When the mailing list settles down I'm going to publish an email to all 
interested parties about all of the comments, and we can respond to each 
comment until we feels it's resolved.

Again, thank you for your support.

Regards,

Steve

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb