From: Hans de Goede <j.w.r.degoede@hhs.nl>
To: Jean-Francois Moine <moinejf@free.fr>
Cc: Linux and Kernel Video <video4linux-list@redhat.com>,
SPCA50x Linux Device Driver Development
<spca50x-devs@lists.sourceforge.net>
Subject: Patch: gspca-zc3xx-pas106b-detect.patch
Date: Thu, 07 Aug 2008 12:21:57 +0200 [thread overview]
Message-ID: <489ACCC5.1050109@hhs.nl> (raw)
[-- Attachment #1: Type: text/plain, Size: 493 bytes --]
Hi,
This patch moves the detection of the 2wr SIF pas106b sensor to before detecing
other 2 wire sensors (as it was done with gspcav1) without this the pas106b
gets misdetected as an HDCS2020b.
This makes my Philips SPC 200NC cam work, but unfortunately the picture is
upside down (same with gspcav1). While experimenting to try to fix this I've
found the meaning of one pas106b register and a comment now documents this.
Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>
Regards,
Hans
[-- Attachment #2: gspca-zc3xx-pas106b-detect.patch --]
[-- Type: text/x-patch, Size: 1321 bytes --]
This patch moves the detection of the 2wr SIF pas106b sensor to before detecing
other 2 wire sensors (as it was done with gspcav1) without this the pas106b
gets misdetected as an HDCS2020b.
This makes my Philips SPC 200NC cam work, but unfortunately the picture is
upside down (same with gspcav1). While experimenting to try to fix this I've
found the meaning of one pas106b register and a comment now documents this.
Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>
diff -r 380c5715fd90 linux/drivers/media/video/gspca/zc3xx.c
--- a/linux/drivers/media/video/gspca/zc3xx.c Wed Aug 06 10:49:16 2008 +0200
+++ b/linux/drivers/media/video/gspca/zc3xx.c Thu Aug 07 12:18:01 2008 +0200
@@ -4134,6 +4134,7 @@
{0xaa, 0x0c, 0x0005},
{0xaa, 0x0d, 0x0000},
{0xaa, 0x0e, 0x0002},
+/* {0xaa, 0x11, 0x0000}, seems to control exposure (0 = max exp) */
{0xaa, 0x14, 0x0081},
/* Other registors */
@@ -6972,6 +6973,10 @@
/* may probe but with write in reg 0x0010 */
return -1; /* don't probe */
}
+ sensor = sif_probe(gspca_dev);
+ if (sensor >= 0)
+ return sensor;
+
sensor = vga_2wr_probe(gspca_dev);
if (sensor >= 0) {
if (sensor < 0x7600)
@@ -6984,7 +6989,7 @@
return sensor;
return sensor2;
}
- return sif_probe(gspca_dev);
+ return -1;
}
/* this function is called at probe time */
[-- Attachment #3: Type: text/plain, Size: 164 bytes --]
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
reply other threads:[~2008-08-07 10:12 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=489ACCC5.1050109@hhs.nl \
--to=j.w.r.degoede@hhs.nl \
--cc=moinejf@free.fr \
--cc=spca50x-devs@lists.sourceforge.net \
--cc=video4linux-list@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox