Manu Abraham wrote:
> On 8/6/07, Michael Krufky <[EMAIL PROTECTED]> wrote:
> >
> > Now I'm beginning to have doubts about Oliver's original patch:
> >
> > dvb_frontend: Range check of frequency and symbol rate
> > http://linuxtv.org/hg/v4l-dvb/rev/8186a34dd0a6
> >
> > Should we be checking fe->ops.tuner_ops.info.frequency_min|max , instead of
> > fe->ops.info.frequency_min|max ???
>
>
> Ideally, what's provided by the demod and not the tuner max/min. The
> tuners max/min should be checked by the demod on setting params.
>
> The upper/lower limits in the demodulator drivers, came from the
> concept of a frontend as a whole. Independant bounds do not make sense
> (except internally -- It is the demod driver that which sets
> parameters for the tuner. The external world doesn't need to know
> what's the limit of the tuner, but only of the frontend as a whole).
>
> Ideally, the demodulator should just demodulate only. There are some
> cases, there are some cases which are not.
Ok, I'm trying to put all pieces together:
There might be cases where demod and tuner have different limits.
So FE_GET_INFO and dvb_frontend_check_parameters() should use the
'smallest common bandwidth':
freq_min = max(fe->ops.info.frequency_min,
fe->ops.tuner_ops.info.frequency_min);
if (fe->ops.info.frequency_max == 0)
freq_max = fe->ops.tuner_ops.info.frequency_max;
else if (fe->ops.tuner_ops.info.frequency_max == 0)
freq_max = fe->ops.info.frequency_max;
else
freq_max = min(fe->ops.info.frequency_max,
fe->ops.tuner_ops.info.frequency_max);
if (freq_min == 0 || freq_max == 0)
printk(KERN_WARNING "frequency limits undefined - please fix the
driver\n");
Conclusions:
- A tuner-only driver must set fe->ops.tuner_ops.info.
- Monolithic drivers must set fe->ops.tuner_ops.info or fe->ops.info
(or both).
Ok?
CU
Oliver
--
----------------------------------------------------------------
VDR Remote Plugin 0.3.9: http://www.escape-edv.de/endriss/vdr/
----------------------------------------------------------------
_______________________________________________
linux-dvb mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
