Skip to contentSkip to conversation

increase MPEG encoder timout

PATCH In review Dmitri Belimov 2 participants 2 messages Apr 28, 2009 – May 11, 2009 Media (V4L/DVB)

lore
Share & tools
lore

Dmitri Belimov

author
#0 lore raw

PATCH 1/3 increase MPEG encoder timout

Hi All.

If video has a lot of changes in frame, MPEG encoder need more time for coding process. Add new bigger timeout for encoder.

This is patch from our customer. I checked this.

linux/drivers/media/video/saa7134/saa7134-ts.c +1 -1
diff -r b40d628f830d linux/drivers/media/video/saa7134/saa7134-ts.c
@@ -65,7 +65,7 @@
 	/* start DMA */
 	saa7134_set_dmabits(dev);
 
-	mod_timer(&dev->ts_q.timeout, jiffies+BUFFER_TIMEOUT);
+	mod_timer(&dev->ts_q.timeout, jiffies+TS_BUFFER_TIMEOUT);
 
 	if (dev->ts_state == SAA7134_TS_BUFF_DONE) {
 		/* Clear TS cache */
linux/drivers/media/video/saa7134/saa7134.h +1 -0
diff -r b40d628f830d linux/drivers/media/video/saa7134/saa7134.h
@@ -367,6 +367,7 @@
 #define INTERLACE_OFF          2
 
 #define BUFFER_TIMEOUT     msecs_to_jiffies(500)  /* 0.5 seconds */
+#define TS_BUFFER_TIMEOUT  msecs_to_jiffies(1000)  /* 1 second */
 
 struct saa7134_dev;
 struct saa7134_dma;
Signed-off-by Alexey Osipov <[email protected]>

With my best regards, Dmitry.

linux/drivers/media/video/saa7134/saa7134-ts.c +1 -1
diff -r b40d628f830d linux/drivers/media/video/saa7134/saa7134-ts.c
@@ -65,7 +65,7 @@
 	/* start DMA */
 	saa7134_set_dmabits(dev);
 
-	mod_timer(&dev->ts_q.timeout, jiffies+BUFFER_TIMEOUT);
+	mod_timer(&dev->ts_q.timeout, jiffies+TS_BUFFER_TIMEOUT);
 
 	if (dev->ts_state == SAA7134_TS_BUFF_DONE) {
 		/* Clear TS cache */
linux/drivers/media/video/saa7134/saa7134.h +1 -0
diff -r b40d628f830d linux/drivers/media/video/saa7134/saa7134.h
@@ -367,6 +367,7 @@
 #define INTERLACE_OFF          2
 
 #define BUFFER_TIMEOUT     msecs_to_jiffies(500)  /* 0.5 seconds */
+#define TS_BUFFER_TIMEOUT  msecs_to_jiffies(1000)  /* 1 second */
 
 struct saa7134_dev;
 struct saa7134_dma;
Signed-off-by Alexey Osipov <[email protected]>
#1 lore raw

Em Tue, 28 Apr 2009 19:41:08 +1000
Dmitri Belimov <[email protected]> escreveu:

Quoted 8 lines

Hi All.

If video has a lot of changes in frame, MPEG encoder need more time for coding process. Add new bigger timeout for encoder.

This is patch from our customer. I checked this.

Signed-off-by Alexey Osipov <[email protected]>

Next time, please sign the patch, since you're forwarding it.

Cheers,
Mauro