This is an automatic generated email to let you know that the following patch 
were queued:

Subject: [media] lgdt3306a: remove 20*50 msec unnecessary timeout
Author:  Abylay Ospan <aos...@netup.ru>
Date:    Mon Jul 25 15:38:59 2016 -0300

inside lgdt3306a_search we reading demod status 20 times with 50 msec sleep 
after each read.
This gives us more than 1 sec of delay. Removing this delay should not affect 
demod functionality.

Signed-off-by: Abylay Ospan <aos...@netup.ru>
Acked-by: Michael Ira Krufky <mkru...@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

 drivers/media/dvb-frontends/lgdt3306a.c | 16 ++++------------
 1 file changed, 4 insertions(+), 12 deletions(-)

---

diff --git a/drivers/media/dvb-frontends/lgdt3306a.c 
b/drivers/media/dvb-frontends/lgdt3306a.c
index afb9d731e60e..0ca4e810e9d8 100644
--- a/drivers/media/dvb-frontends/lgdt3306a.c
+++ b/drivers/media/dvb-frontends/lgdt3306a.c
@@ -1737,24 +1737,16 @@ static int lgdt3306a_get_tune_settings(struct 
dvb_frontend *fe,
 static int lgdt3306a_search(struct dvb_frontend *fe)
 {
        enum fe_status status = 0;
-       int i, ret;
+       int ret;
 
        /* set frontend */
        ret = lgdt3306a_set_parameters(fe);
        if (ret)
                goto error;
 
-       /* wait frontend lock */
-       for (i = 20; i > 0; i--) {
-               dbg_info(": loop=%d\n", i);
-               msleep(50);
-               ret = lgdt3306a_read_status(fe, &status);
-               if (ret)
-                       goto error;
-
-               if (status & FE_HAS_LOCK)
-                       break;
-       }
+       ret = lgdt3306a_read_status(fe, &status);
+       if (ret)
+               goto error;
 
        /* check if we have a valid signal */
        if (status & FE_HAS_LOCK)

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to