Em Wed, 12 Nov 2014 06:23:04 +0200
Antti Palosaari <crope@xxxxxx> escreveu:
> Implement PIP mode to stream from slave demodulator. PIP mode is
> enabled when .set_frontend is called with RF frequency 0, otherwise
> normal demod mode is enabled.
This would be an API change, so, a DocBook patch is required.
Anyway, using frequency=0 for PIP doesn't seem to be a good idea,
as a read from GET_PROPERTY should override the cache with the real
frequency.
Also, someone came with me with a case where auto-frequency would
be interesting, and proposed frequency=0. I was not convinced
(and patches weren't sent), but using 0 for AUTO seems more
appropriate, as we do the same for bandwidth (and may do the same
for symbol_rate).
So, the best seems to add a new property to enable PIP mode.
Regards,
Mauro
>
> Signed-off-by: Antti Palosaari <crope@xxxxxx>
> ---
> drivers/media/dvb-frontends/rtl2832.c | 42 ++++++++++++++++++++++++++++++++---
> 1 file changed, 39 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/dvb-frontends/rtl2832.c b/drivers/media/dvb-frontends/rtl2832.c
> index eb737cf..a58b456 100644
> --- a/drivers/media/dvb-frontends/rtl2832.c
> +++ b/drivers/media/dvb-frontends/rtl2832.c
> @@ -258,13 +258,11 @@ static int rtl2832_rd_regs(struct rtl2832_priv *priv, u8 reg, u8 page, u8 *val,
> return rtl2832_rd(priv, reg, val, len);
> }
>
> -#if 0 /* currently not used */
> /* write single register */
> static int rtl2832_wr_reg(struct rtl2832_priv *priv, u8 reg, u8 page, u8 val)
> {
> return rtl2832_wr_regs(priv, reg, page, &val, 1);
> }
> -#endif
>
> /* read single register */
> static int rtl2832_rd_reg(struct rtl2832_priv *priv, u8 reg, u8 page, u8 *val)
> @@ -595,6 +593,44 @@ static int rtl2832_set_frontend(struct dvb_frontend *fe)
> "%s: frequency=%d bandwidth_hz=%d inversion=%d\n",
> __func__, c->frequency, c->bandwidth_hz, c->inversion);
>
> + /* PIP mode */
> + if (c->frequency == 0) {
> + dev_dbg(&priv->i2c->dev, "%s: setting PIP mode\n", __func__);
> + ret = rtl2832_wr_regs(priv, 0x0c, 1, "\x5f\xff", 2);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_demod_reg(priv, DVBT_PIP_ON, 0x1);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_reg(priv, 0xbc, 0, 0x18);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_reg(priv, 0x22, 0, 0x01);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_reg(priv, 0x26, 0, 0x1f);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_reg(priv, 0x27, 0, 0xff);
> + if (ret)
> + goto err;
> +
> + ret = rtl2832_wr_regs(priv, 0x92, 1, "\x7f\xf7\xff", 3);
> + if (ret)
> + goto err;
> +
> + goto exit_soft_reset;
> + } else {
> + ret = rtl2832_wr_regs(priv, 0x92, 1, "\x00\x0f\xff", 3);
> + if (ret)
> + goto err;
> + }
> +
> /* program tuner */
> if (fe->ops.tuner_ops.set_params)
> fe->ops.tuner_ops.set_params(fe);
> @@ -661,7 +697,7 @@ static int rtl2832_set_frontend(struct dvb_frontend *fe)
> if (ret)
> goto err;
>
> -
> +exit_soft_reset:
> /* soft reset */
> ret = rtl2832_wr_demod_reg(priv, DVBT_SOFT_RST, 0x1);
> if (ret)
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
