This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/media_tree.git tree:

Subject: [media] si2165: only write agc registers after reset before 
start_syncro
Author:  Matthias Schwarzott <[email protected]>
Date:    Thu Nov 19 18:03:56 2015 -0200

Datasheet says they must be rewritten after reset.
But it only makes sense to write them when trying to tune afterwards.

Signed-off-by: Matthias Schwarzott <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/dvb-frontends/si2165.c | 32 +++++++++++++++-----------------
 1 file changed, 15 insertions(+), 17 deletions(-)

---

http://git.linuxtv.org/cgit.cgi/media_tree.git/commit/?id=eae56684a000f2be0f62a7cfd427376b1fb02538
diff --git a/drivers/media/dvb-frontends/si2165.c 
b/drivers/media/dvb-frontends/si2165.c
index 222d7750f98d..07247e307f55 100644
--- a/drivers/media/dvb-frontends/si2165.c
+++ b/drivers/media/dvb-frontends/si2165.c
@@ -690,23 +690,6 @@ static int si2165_init(struct dvb_frontend *fe)
                        goto error;
        }
 
-       /* write adc values after each reset*/
-       ret = si2165_writereg8(state, 0x012a, 0x46);
-       if (ret < 0)
-               goto error;
-       ret = si2165_writereg8(state, 0x012c, 0x00);
-       if (ret < 0)
-               goto error;
-       ret = si2165_writereg8(state, 0x012e, 0x0a);
-       if (ret < 0)
-               goto error;
-       ret = si2165_writereg8(state, 0x012f, 0xff);
-       if (ret < 0)
-               goto error;
-       ret = si2165_writereg8(state, 0x0123, 0x70);
-       if (ret < 0)
-               goto error;
-
        return 0;
 error:
        return ret;
@@ -788,6 +771,14 @@ static int si2165_set_if_freq_shift(struct si2165_state 
*state, u32 IF)
        return si2165_writereg32(state, 0x00e8, reg_value);
 }
 
+static const struct si2165_reg_value_pair agc_rewrite[] = {
+       { 0x012a, 0x46 },
+       { 0x012c, 0x00 },
+       { 0x012e, 0x0a },
+       { 0x012f, 0xff },
+       { 0x0123, 0x70 }
+};
+
 static int si2165_set_frontend(struct dvb_frontend *fe)
 {
        int ret;
@@ -924,6 +915,13 @@ static int si2165_set_frontend(struct dvb_frontend *fe)
        ret = si2165_writereg32(state, 0x0384, 0x00000000);
        if (ret < 0)
                return ret;
+
+       /* write adc values after each reset*/
+       ret = si2165_write_reg_list(state, agc_rewrite,
+                                   ARRAY_SIZE(agc_rewrite));
+       if (ret < 0)
+               return ret;
+
        /* start_synchro */
        ret = si2165_writereg8(state, 0x02e0, 0x01);
        if (ret < 0)

_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to