Skip to content

Commit

Permalink
[media] dvb_frontend: get rid of set_state ops & related data
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Mauro Carvalho Chehab committed Nov 17, 2015
1 parent 65f0f68 commit 45346e0
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 24 deletions.
23 changes: 0 additions & 23 deletions drivers/media/dvb-core/dvb_frontend.h
Original file line number Diff line number Diff line change
Expand Up @@ -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
*
Expand Down Expand Up @@ -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
*
Expand Down Expand Up @@ -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.
Expand Down Expand Up @@ -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);
};

/**
Expand Down
1 change: 0 additions & 1 deletion drivers/media/tuners/mt2063.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 45346e0

Please sign in to comment.