[media] dvb_frontend: get rid of set_state ops & related data
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 11 Nov 2015 20:57:16 +0000 (18:57 -0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 17 Nov 2015 08:46:02 +0000 (06:46 -0200)
The get_state()/set_state and the corresponding data types
(struct tuner_state and enum tuner_param) are old DVB interfaces
that came from the DVBv3 time.

Nowadays, set_params() provide a better way to set the tuner
and demod parameters. So, no need to keep those legacy stuff,
as all drivers that were using it got converted.

With this patch, all kABI elements at dvb_frontend.h are now
documented.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/dvb-core/dvb_frontend.h
drivers/media/tuners/mt2063.c

index 48564115bd59b8283585ae7a2e860d317889d203..032e125a5f1538b5b0b257d548187585fdce874a 100644 (file)
@@ -135,11 +135,6 @@ struct analog_parameters {
        u64 std;
 };
 
-enum tuner_param {
-       DVBFE_TUNER_FREQUENCY           = (1 <<  0),
-       DVBFE_TUNER_BANDWIDTH           = (1 <<  1),
-};
-
 /**
  * enum dvbfe_algo - defines the algorithm used to tune into a channel
  *
@@ -170,11 +165,6 @@ enum dvbfe_algo {
        DVBFE_ALGO_RECOVERY             = (1 << 31)
 };
 
-struct tuner_state {
-       u32 frequency;
-       u32 bandwidth;
-};
-
 /**
  * enum dvbfe_search - search callback possible return status
  *
@@ -245,12 +235,6 @@ enum dvbfe_search {
  *                     set_params is preferred.
  * @set_bandwidth:     Set a new frequency. Please notice that using
  *                     set_params is preferred.
- * @set_state:         callback function used on some legacy drivers that
- *                     don't implement set_params in order to set properties.
- *                     Shouldn't be used on new drivers.
- * @get_state:         callback function used to get properties by some
- *                     legacy drivers that don't implement set_params.
- *                     Shouldn't be used on new drivers.
  *
  * NOTE: frequencies used on get_frequency and set_frequency are in Hz for
  * terrestrial/cable or kHz for satellite.
@@ -290,13 +274,6 @@ struct dvb_tuner_ops {
         * tuners which require sophisticated tuning loops, controlling each parameter separately. */
        int (*set_frequency)(struct dvb_frontend *fe, u32 frequency);
        int (*set_bandwidth)(struct dvb_frontend *fe, u32 bandwidth);
-
-       /*
-        * These are provided separately from set_params in order to facilitate silicon
-        * tuners which require sophisticated tuning loops, controlling each parameter separately.
-        */
-       int (*set_state)(struct dvb_frontend *fe, enum tuner_param param, struct tuner_state *state);
-       int (*get_state)(struct dvb_frontend *fe, enum tuner_param param, struct tuner_state *state);
 };
 
 /**
index 9e9c5eb4cb66949a7e84a9789a9829f6f1f913d9..6457ac91ef09567b8b3986d3c6bf3aa55771cfaa 100644 (file)
@@ -225,7 +225,6 @@ struct mt2063_state {
        const struct mt2063_config *config;
        struct dvb_tuner_ops ops;
        struct dvb_frontend *frontend;
-       struct tuner_state status;
 
        u32 frequency;
        u32 srate;