aboutsummaryrefslogtreecommitdiffstats
path: root/include/media/tuner-types.h
diff options
context:
space:
mode:
Diffstat (limited to 'include/media/tuner-types.h')
-rw-r--r--include/media/tuner-types.h20
1 files changed, 19 insertions, 1 deletions
diff --git a/include/media/tuner-types.h b/include/media/tuner-types.h
index 64b16b1f1926..15821ab14a9e 100644
--- a/include/media/tuner-types.h
+++ b/include/media/tuner-types.h
@@ -19,6 +19,25 @@ struct tuner_range {
struct tuner_params {
enum param_type type;
+ /* Many Philips based tuners have a comment like this in their
+ * datasheet:
+ *
+ * For channel selection involving band switching, and to ensure
+ * smooth tuning to the desired channel without causing
+ * unnecessary charge pump action, it is recommended to consider
+ * the difference between wanted channel frequency and the
+ * current channel frequency. Unnecessary charge pump action
+ * will result in very low tuning voltage which may drive the
+ * oscillator to extreme conditions.
+ *
+ * Set cb_first_if_lower_freq to 1, if this check is
+ * required for this tuner.
+ *
+ * I tested this for PAL by first setting the TV frequency to
+ * 203 MHz and then switching to 96.6 MHz FM radio. The result was
+ * static unless the control byte was sent first.
+ */
+ unsigned int cb_first_if_lower_freq:1;
unsigned char config; /* to be moved into struct tuner_range for dvb-pll merge */
unsigned int count;
@@ -27,7 +46,6 @@ struct tuner_params {
struct tunertype {
char *name;
- unsigned int has_tda988x:1;
struct tuner_params *params;
};