[media] cx88: increase API command timeout
authorHans Verkuil <hans.verkuil@cisco.com>
Fri, 29 Aug 2014 08:45:59 +0000 (05:45 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Mon, 3 Nov 2014 10:30:26 +0000 (08:30 -0200)
The timeout is way too small. Especially complicated command like
CX2341X_ENC_STOP_CAPTURE takes much more time than 10 ms. Increase the
timeout to 1 second, just as ivtv does (the cx88-blackbird has the
same IP core for MPEG compression as ivtv).

This solves a nasty issue where STOP_CAPTURE would timeout and the
mailbox is left in a busy state, making it impossible to start streaming
a second time without reloading the driver first.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/pci/cx88/cx88-blackbird.c

index f27a3f134025fc5edca7847605ac27ff64802d5f..32fb9355c9f6a8bc6014fb98b06281e4689c3955 100644 (file)
@@ -348,7 +348,7 @@ static int blackbird_mbox_func(void *priv, u32 command, int in, int out, u32 dat
        memory_write(dev->core, dev->mailbox, flag);
 
        /* wait for firmware to handle the API command */
-       timeout = jiffies + msecs_to_jiffies(10);
+       timeout = jiffies + msecs_to_jiffies(1000);
        for (;;) {
                memory_read(dev->core, dev->mailbox, &flag);
                if (0 != (flag & 4))