On Wed, 21 Jan 2009, Jean-Francois Moine wrote:
(previous material covered a different topic)
Now, the request: some guys asked for support of their webcams based on
sq930x chips. A SANE backend driver exists, written by Gerard Klaver
(http://gkall.hobby.nl/sq930x.html).
May you have a look and say if handling these chips may be done in your
driver?
I am sorry to have been rather brief about this one in my previous
message, but I am quite serious in saying "no." Clearly it is always
possible somehow to jam-fit the support for two or more very different
devices into one block of code. One would require some kind of branching
at every logical step in the code, and nobody would be able to read it and
make sense of that code, or to maintain it.
In this present case of the sq930x, I did look at Gerard's information.
Indeed, the very structure of the commands is different from the commands
for the sq905 and for that matter from the sq905c cameras, with both of
which I am intimately familiar. A mere glance at the file
creative protocol I420.txt
at Gerard's web page suffices to show that the command sequences are
completely different. One can add to this that the frames are JPEG,
whereas the frames for the sq905 are uncompressed bitmap data.
I would be glad to help with the sq930x if there is any help I can give
and if the help is wanted. Anyone who wants me to do that should just ask.
But that is a totally different matter from trying to glue together the
support for two very unlike devices in one driver module, apparently for
the reason that the two devices share the same Vendor number.
As a very relevant analogy, permit me to say that I wrote the driver in
libgphoto2 for the Logitech Clicksmart 310 camera. When it became clear
that this is yet another spca50x camera, I contacted the person who had
written the unified spca50x driver for libgphoto2 and asked him if,
because of this, he wanted this new spca5x camera to be included there. I
sent him my code. He looked at it, and he said to leave it as a separate
driver. In spite of using the same chipset, the camera was just too
different. He further added that he was sorry he had tried to put so many
cameras into one driver in the first place, had learned a lesson from the
experience, and if he had it to do over again he would not.
Theodore Kilgore
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
