Merge branch 'upstream'

[linux-2.6] / include / media / tuner-types.h

diff --git a/include/media/tuner-types.h b/include/media/tuner-types.h
index 756693188991c65210983b049fe7ad11b1b26202..ad9c171bfa07177e0716e34a6d0c8be4e84b547d 100644 (file)
--- a/include/media/tuner-types.h
+++ b/include/media/tuner-types.h
@@ -14,13 +14,31 @@ enum param_type {
 
 struct tuner_range {
        unsigned short limit;
+       unsigned char config;
        unsigned char cb;
 };
 
 struct tuner_params {
        enum param_type type;
-       unsigned int tda988x;
-       unsigned char config; /* to be moved into struct tuner_range for dvb-pll merge */
+       /* 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 int count;
        struct tuner_range *ranges;
@@ -28,6 +46,7 @@ struct tuner_params {
 
 struct tunertype {
        char *name;
+       unsigned int count;
        struct tuner_params *params;
 };