Browse Source

[media] dvb_frontend.h: get rid of unused tuner params/states

There are several tuner_param values that aren't by any driver or core:
	DVBFE_TUNER_TUNERSTEP
	DVBFE_TUNER_IFFREQ
	DVBFE_TUNER_REFCLOCK
	DVBFE_TUNER_IQSENSE
	DVBFE_TUNER_DUMMY

Several of those correspond to the values at the tuner_state
struct with is also only initialized by not used anyware:
	u32 tunerstep;
	u32 ifreq;
	u32 refclock;

It doesn't make sense to keep anything at the kABI that it is
not used. So, get rid of them.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Mauro Carvalho Chehab 9 years ago
parent
commit
2184e2530c
2 changed files with 7 additions and 27 deletions
  1. 1 10
      drivers/media/dvb-core/dvb_frontend.h
  2. 6 17
      drivers/media/dvb-frontends/stb6100.c

+ 1 - 10
drivers/media/dvb-core/dvb_frontend.h

@@ -137,12 +137,7 @@ struct analog_parameters {
 
 
 enum tuner_param {
 enum tuner_param {
 	DVBFE_TUNER_FREQUENCY		= (1 <<  0),
 	DVBFE_TUNER_FREQUENCY		= (1 <<  0),
-	DVBFE_TUNER_TUNERSTEP		= (1 <<  1),
-	DVBFE_TUNER_IFFREQ		= (1 <<  2),
-	DVBFE_TUNER_BANDWIDTH		= (1 <<  3),
-	DVBFE_TUNER_REFCLOCK		= (1 <<  4),
-	DVBFE_TUNER_IQSENSE		= (1 <<  5),
-	DVBFE_TUNER_DUMMY		= (1 << 31)
+	DVBFE_TUNER_BANDWIDTH		= (1 <<  1),
 };
 };
 
 
 /**
 /**
@@ -177,11 +172,7 @@ enum dvbfe_algo {
 
 
 struct tuner_state {
 struct tuner_state {
 	u32 frequency;
 	u32 frequency;
-	u32 tunerstep;
-	u32 ifreq;
 	u32 bandwidth;
 	u32 bandwidth;
-	u32 iqsense;
-	u32 refclock;
 };
 };
 
 
 /**
 /**

+ 6 - 17
drivers/media/dvb-frontends/stb6100.c

@@ -496,14 +496,15 @@ static int stb6100_init(struct dvb_frontend *fe)
 {
 {
 	struct stb6100_state *state = fe->tuner_priv;
 	struct stb6100_state *state = fe->tuner_priv;
 	struct tuner_state *status = &state->status;
 	struct tuner_state *status = &state->status;
+	int refclk = 27000000; /* Hz */
 
 
-	status->tunerstep	= 125000;
-	status->ifreq		= 0;
-	status->refclock	= 27000000;	/* Hz	*/
-	status->iqsense		= 1;
+	/*
+	 * iqsense = 1
+	 * tunerstep = 125000
+	 */
 	status->bandwidth	= 36000;	/* kHz	*/
 	status->bandwidth	= 36000;	/* kHz	*/
 	state->bandwidth	= status->bandwidth * 1000;	/* Hz	*/
 	state->bandwidth	= status->bandwidth * 1000;	/* Hz	*/
-	state->reference	= status->refclock / 1000;	/* kHz	*/
+	state->reference	= refclk / 1000;	/* kHz	*/
 
 
 	/* Set default bandwidth. Modified, PN 13-May-10	*/
 	/* Set default bandwidth. Modified, PN 13-May-10	*/
 	return 0;
 	return 0;
@@ -517,15 +518,9 @@ static int stb6100_get_state(struct dvb_frontend *fe,
 	case DVBFE_TUNER_FREQUENCY:
 	case DVBFE_TUNER_FREQUENCY:
 		stb6100_get_frequency(fe, &state->frequency);
 		stb6100_get_frequency(fe, &state->frequency);
 		break;
 		break;
-	case DVBFE_TUNER_TUNERSTEP:
-		break;
-	case DVBFE_TUNER_IFFREQ:
-		break;
 	case DVBFE_TUNER_BANDWIDTH:
 	case DVBFE_TUNER_BANDWIDTH:
 		stb6100_get_bandwidth(fe, &state->bandwidth);
 		stb6100_get_bandwidth(fe, &state->bandwidth);
 		break;
 		break;
-	case DVBFE_TUNER_REFCLOCK:
-		break;
 	default:
 	default:
 		break;
 		break;
 	}
 	}
@@ -544,16 +539,10 @@ static int stb6100_set_state(struct dvb_frontend *fe,
 		stb6100_set_frequency(fe, state->frequency);
 		stb6100_set_frequency(fe, state->frequency);
 		tstate->frequency = state->frequency;
 		tstate->frequency = state->frequency;
 		break;
 		break;
-	case DVBFE_TUNER_TUNERSTEP:
-		break;
-	case DVBFE_TUNER_IFFREQ:
-		break;
 	case DVBFE_TUNER_BANDWIDTH:
 	case DVBFE_TUNER_BANDWIDTH:
 		stb6100_set_bandwidth(fe, state->bandwidth);
 		stb6100_set_bandwidth(fe, state->bandwidth);
 		tstate->bandwidth = state->bandwidth;
 		tstate->bandwidth = state->bandwidth;
 		break;
 		break;
-	case DVBFE_TUNER_REFCLOCK:
-		break;
 	default:
 	default:
 		break;
 		break;
 	}
 	}