| Message ID | 20130113193133.GA5907@elgon.mountain (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Em Sun, 13 Jan 2013 22:31:33 +0300 Dan Carpenter <dan.carpenter@oracle.com> escreveu: > There is a missing break so we use XC_RF_MODE_CABLE instead of > XC_RF_MODE_AIR. > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > Static checker stuff. Untested. > > diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c > index dc93cf3..d6be1b6 100644 > --- a/drivers/media/tuners/xc5000.c > +++ b/drivers/media/tuners/xc5000.c > @@ -785,6 +785,7 @@ static int xc5000_set_params(struct dvb_frontend *fe) > return -EINVAL; > } > priv->rf_mode = XC_RF_MODE_AIR; > + break; There one small change with this patch. On this part of the code: static int xc_SetSignalSource(struct xc5000_priv *priv, u16 rf_mode) { dprintk(1, "%s(%d) Source = %s\n", __func__, rf_mode, rf_mode == XC_RF_MODE_AIR ? "ANTENNA" : "CABLE"); if ((rf_mode != XC_RF_MODE_AIR) && (rf_mode != XC_RF_MODE_CABLE)) { rf_mode = XC_RF_MODE_CABLE; printk(KERN_ERR "%s(), Invalid mode, defaulting to CABLE", __func__); } return xc_write_reg(priv, XREG_SIGNALSOURCE, rf_mode); } It will set the value for XREG_SIGNALSOURCE with a different value. While I didn't test it, such change makes sense, by looking at xc5000 "open source" datasheet. So, it looks correct on my eyes. While the datasheet doesn't give any glue, my guess is that changing from "cable" to "air" will just optimize the tuner's sensibility for either air (where signals can be weaker) or cable, so I bet that the effects of a change like that won't be easily noticed. I'll apply it, in order to give people some chance to test it. Cheers, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c index dc93cf3..d6be1b6 100644 --- a/drivers/media/tuners/xc5000.c +++ b/drivers/media/tuners/xc5000.c @@ -785,6 +785,7 @@ static int xc5000_set_params(struct dvb_frontend *fe) return -EINVAL; } priv->rf_mode = XC_RF_MODE_AIR; + break; case SYS_DVBC_ANNEX_A: case SYS_DVBC_ANNEX_C: dprintk(1, "%s() QAM modulation\n", __func__);
There is a missing break so we use XC_RF_MODE_CABLE instead of XC_RF_MODE_AIR. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- Static checker stuff. Untested. -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html