V4L/DVB (8198): gspca: Frame decoding errors when PAC207 in full daylight.
authorHans de Goede <j.w.r.degoede@hhs.nl>
Fri, 4 Jul 2008 21:39:08 +0000 (18:39 -0300)
committerMauro Carvalho Chehab <mchehab@infradead.org>
Sun, 20 Jul 2008 10:17:06 +0000 (07:17 -0300)
Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Signed-off-by: Jean-Francois Moine <moinejf@free.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
drivers/media/video/gspca/pac207.c

index 0135ba599e81cb8890d23fd857469a5c27e25502..ab83d7b6934848824893d726c364e5fb8b5a765e 100644 (file)
@@ -40,9 +40,17 @@ MODULE_LICENSE("GPL");
 #define PAC207_BRIGHTNESS_MAX          255
 #define PAC207_BRIGHTNESS_DEFAULT      4 /* power on default: 4 */
 
-#define PAC207_EXPOSURE_MIN            4
+/* An exposure value of 4 also works (3 does not) but then we need to lower
+   the compression balance setting when in 352x288 mode, otherwise the usb
+   bandwidth is not enough and packets get dropped resulting in corrupt
+   frames. The problem with this is that when the compression balance gets
+   lowered below 0x80, the pac207 starts using a different compression
+   algorithm for some lines, these lines get prefixed with a 0x2dd2 prefix
+   and currently we do not know how to decompress these lines, so for now
+   we use a minimum exposure value of 5 */
+#define PAC207_EXPOSURE_MIN            5
 #define PAC207_EXPOSURE_MAX            26
-#define PAC207_EXPOSURE_DEFAULT                4 /* power on default: 3 ?? */
+#define PAC207_EXPOSURE_DEFAULT                5 /* power on default: 3 ?? */
 #define PAC207_EXPOSURE_KNEE           11 /* 4 = 30 fps, 11 = 8, 15 = 6 */
 
 #define PAC207_GAIN_MIN                        0