The switch/case was setting clock_setting that is not being used. Both 
switch/case and the variable definition were removed.

Currently clock is being calculated by the formula:
(state->fclk/1000000 - 8)

Tested by compilation only.

Signed-off-by: Peter Senna Tschudin <peter.se...@gmail.com>
---
 drivers/media/dvb/frontends/s5h1420.c |   20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/drivers/media/dvb/frontends/s5h1420.c 
b/drivers/media/dvb/frontends/s5h1420.c
index 2322257..e2fec9e 100644
--- a/drivers/media/dvb/frontends/s5h1420.c
+++ b/drivers/media/dvb/frontends/s5h1420.c
@@ -634,7 +634,6 @@ static int s5h1420_set_frontend(struct dvb_frontend *fe)
        struct s5h1420_state* state = fe->demodulator_priv;
        int frequency_delta;
        struct dvb_frontend_tune_settings fesettings;
-       uint8_t clock_setting;
 
        dprintk("enter %s\n", __func__);
 
@@ -679,25 +678,6 @@ static int s5h1420_set_frontend(struct dvb_frontend *fe)
        else
                state->fclk = 44000000;
 
-       /* Clock */
-       switch (state->fclk) {
-       default:
-       case 88000000:
-               clock_setting = 80;
-               break;
-       case 86000000:
-               clock_setting = 78;
-               break;
-       case 80000000:
-               clock_setting = 72;
-               break;
-       case 59000000:
-               clock_setting = 51;
-               break;
-       case 44000000:
-               clock_setting = 36;
-               break;
-       }
        dprintk("pll01: %d, ToneFreq: %d\n", state->fclk/1000000 - 8, 
(state->fclk + (TONE_FREQ * 32) - 1) / (TONE_FREQ * 32));
        s5h1420_writereg(state, PLL01, state->fclk/1000000 - 8);
        s5h1420_writereg(state, PLL02, 0x40);
-- 
1.7.10.2

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to