4 files changed, 16 insertions, 8 deletions

diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c
index 5e8bb41a088b..59ac35ddd51e 100644
--- a/drivers/media/dvb/dvb-core/dvb_frontend.c
+++ b/drivers/media/dvb/dvb-core/dvb_frontend.c
@@ -570,7 +570,8 @@ static int dvb_frontend_thread(void *data)
                                         dvb_frontend_add_event(fe, s);
                                         fepriv->status = s;
                                 }
-                        }
+                        } else
+                                dvb_frontend_swzigzag(fe);
                 } else
                         dvb_frontend_swzigzag(fe);
         }
@@ -975,6 +976,7 @@ static int dvb_frontend_ioctl(struct inode *inode, struct file *file,
 
         case FE_SET_FRONTEND_TUNE_MODE:
                 fepriv->tune_mode_flags = (unsigned long) parg;
+                err = 0;
                 break;
         };
 
diff --git a/drivers/media/dvb/frontends/cx22700.c b/drivers/media/dvb/frontends/cx22700.c
index 3c7c09a362b2..13ad1bfae663 100644
--- a/drivers/media/dvb/frontends/cx22700.c
+++ b/drivers/media/dvb/frontends/cx22700.c
@@ -134,6 +134,7 @@ static int cx22700_set_tps (struct cx22700_state *state, struct dvb_ofdm_paramet
                 return -EINVAL;
 
         if (p->code_rate_LP < FEC_1_2 || p->code_rate_LP > FEC_7_8)
+                return -EINVAL;
 
         if (p->code_rate_HP == FEC_4_5 || p->code_rate_LP == FEC_4_5)
                 return -EINVAL;
diff --git a/drivers/media/dvb/frontends/cx24123.c b/drivers/media/dvb/frontends/cx24123.c
index f2f795cba56a..274a87b7a5d5 100644
--- a/drivers/media/dvb/frontends/cx24123.c
+++ b/drivers/media/dvb/frontends/cx24123.c
@@ -670,10 +670,10 @@ static int cx24123_set_voltage(struct dvb_frontend* fe, fe_sec_voltage_t voltage
         switch (voltage) {
         case SEC_VOLTAGE_13:
                 dprintk("%s: setting voltage 13V\n", __FUNCTION__);
-                return cx24123_writereg(state, 0x29, val | 0x80);
+                return cx24123_writereg(state, 0x29, val & 0x7f);
         case SEC_VOLTAGE_18:
                 dprintk("%s: setting voltage 18V\n", __FUNCTION__);
-                return cx24123_writereg(state, 0x29, val & 0x7f);
+                return cx24123_writereg(state, 0x29, val | 0x80);
         default:
                 return -EINVAL;
         };
diff --git a/drivers/media/dvb/ttpci/budget-av.c b/drivers/media/dvb/ttpci/budget-av.c
index 6163cb03b8f4..5f111d407730 100644
--- a/drivers/media/dvb/ttpci/budget-av.c
+++ b/drivers/media/dvb/ttpci/budget-av.c
@@ -1141,6 +1141,15 @@ static void frontend_init(struct budget_av *budget_av)
                 break;
 
         case SUBID_DVBC_KNC1:
+                budget_av->reinitialise_demod = 1;
+                fe = tda10021_attach(&philips_cu1216_config,
+                                     &budget_av->budget.i2c_adap,
+                                     read_pwm(budget_av));
+                if (fe) {
+                        fe->ops.tuner_ops.set_params = philips_cu1216_tuner_set_params;
+                }
+                break;
+
         case SUBID_DVBC_KNC1_PLUS:
         case SUBID_DVBC_CINERGY1200:
                 budget_av->reinitialise_demod = 1;
@@ -1293,11 +1302,7 @@ static int budget_av_attach(struct saa7146_dev *dev, struct saa7146_pci_extensio
 
         budget_av->budget.dvb_adapter.priv = budget_av;
         frontend_init(budget_av);
-
-        if (!budget_av->has_saa7113) {
-                ciintf_init(budget_av);
-        }
-
+        ciintf_init(budget_av);
         return 0;
 }