From: Jean-Francois Moine <moinejf@free.fr>
To: Adam Baker <linux@baker-net.org.uk>
Cc: video4linux-list <video4linux-list@redhat.com>,
sqcam-devel@lists.sourceforge.net,
kilgota@banach.math.auburn.edu
Subject: Re: [REVIEW] Driver for SQ-905 based cameras
Date: Thu, 01 Jan 2009 18:38:18 +0100 [thread overview]
Message-ID: <1230831498.1702.40.camel@localhost> (raw)
In-Reply-To: <200901010033.58093.linux@baker-net.org.uk>
On Thu, 2009-01-01 at 00:33 +0000, Adam Baker wrote:
> Theodore Kilgore and I now have a driver for cameras based on the
> SQ 905 chipset that is capable of producing images. It is based on gspca
[snip]
Fine, but...
+ * This driver has used as a base the finepix driver and other gspca
+ * based drivers and may still contain code fragments taken from those
+ * drivers.
You did not look carefully at the finepix subdriver. Its webcams work
quite the same as yours, i.e. they ask for a control message to start
the bulk image transfer and an other control message to ack the
reception of the image. For that, the finepix implements a state machine
running at interrupt level or in the system work queue. All USB
exchanges are asynchronous, so that the system thread is not blocked.
Instead, you do a loop in this thread: then, this one cannot be used for
any other purpose!
I see only one alternative to do the image transfer:
- either implement a state machine as it is done in finepix,
- or have a specific work queue to handle the USB exchanges.
Best regards.
--
Ken ar c'hentañ | ** Breizh ha Linux atav! **
Jef | http://moinejf.free.fr/
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2009-01-01 17:38 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-01 0:33 [REVIEW] Driver for SQ-905 based cameras Adam Baker
2009-01-01 6:11 ` Alexey Klimov
2009-01-01 12:28 ` Hans de Goede
2009-01-01 21:19 ` [sqcam-devel] " Adam Baker
[not found] ` <Pine.LNX.4.64.0901011539120.19217@banach.math.auburn.edu>
2009-01-02 7:55 ` Hans de Goede
2009-01-01 17:38 ` Jean-Francois Moine [this message]
[not found] ` <Pine.LNX.4.64.0901011220230.18838@banach.math.auburn.edu>
2009-01-01 20:48 ` Adam Baker
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=1230831498.1702.40.camel@localhost \
--to=moinejf@free.fr \
--cc=kilgota@banach.math.auburn.edu \
--cc=linux@baker-net.org.uk \
--cc=sqcam-devel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.