From: Matthew Dharm <mdharm-kernel@one-eyed-alien.net>
To: Jan Niehusmann <jan@gondor.com>
Cc: Harald Schreiber <harald@harald-schreiber.de>,
linux-kernel@vger.kernel.org,
linux-usb-devel@lists.sourceforge.net
Subject: Re: [PATCH] USB Support for recent Casio Digital Still Cameras, linux-2.4.11
Date: Wed, 10 Oct 2001 17:41:17 -0700 [thread overview]
Message-ID: <20011010174117.B10273@one-eyed-alien.net> (raw)
In-Reply-To: <m3elobqblf.fsf@harald-schreiber.de> <20011011023647.A14615@gondor.com>
In-Reply-To: <20011011023647.A14615@gondor.com>; from jan@gondor.com on Thu, Oct 11, 2001 at 02:36:47AM +0200
[-- Attachment #1: Type: text/plain, Size: 2268 bytes --]
In my experience, if you've tested the endpoints on these things, you've
covered the middle. I'd rather have the single entry.
Matt
On Thu, Oct 11, 2001 at 02:36:47AM +0200, Jan Niehusmann wrote:
> On Thu, Oct 11, 2001 at 02:11:56AM +0200, Harald Schreiber wrote:
> > Recent Casio QV digital still cameras (for example the QV 4000)
> > show up as
> > VendorID: 0x07cf ProductID: 0x1001 Revision: 10.00
> >
> > So the following patch is necessary to make these devices working
> > with the USB storage driver. The patch is against linux-2.4.11,
> > but it should work with any kernel since 2.4.8-pre3.
>
> Confirmed. I sent the same patch (without the comments) to linux-kernel
> three days ago.
>
> BTW, perhaps it would be better to patch the thing the following way,
> because I don't know what's in the revision number range between
> 1000 and 9009, perhaps there are other devices which don't like the
> workaround?
>
>
> --- linux-2.4.9-ac10/drivers/usb/storage/unusual_devs.h Sat Sep 8 18:18:51 2001
> +++ linux-2.4.10-ac11/drivers/usb/storage/unusual_devs.h Thu Oct 11 02:11:29 2001
> @@ -394,6 +394,12 @@
> * - They don't like the INQUIRY command. So we must handle this command
> * of the SCSI layer ourselves.
> */
> +UNUSUAL_DEV( 0x07cf, 0x1001, 0x1000, 0x1000,
> + "Casio",
> + "QV DigitalCamera",
> + US_SC_8070, US_PR_CB, NULL,
> + US_FL_FIX_INQUIRY ),
> +
> UNUSUAL_DEV( 0x07cf, 0x1001, 0x9009, 0x9009,
> "Casio",
> "QV DigitalCamera",
>
> > --------------------------------------------------------------------
> > Dipl.-Math. Harald Schreiber, Nizzaallee 26, D-52072 Aachen, Germany
> > Phone/Fax: +49-241-9108015/6 mailto:harald@harald-schreiber.de
> > --------------------------------------------------------------------
>
> Interesting, twice the same patch, and then from nearly the same location
> in the world. (Roermonder Str. 112a - distance several 100m )
>
> Jan
--
Matthew Dharm Home: mdharm-usb@one-eyed-alien.net
Maintainer, Linux USB Mass Storage Driver
God, root, what is difference?
-- Pitr
User Friendly, 11/11/1999
[-- Attachment #2: Type: application/pgp-signature, Size: 232 bytes --]
prev parent reply other threads:[~2001-10-11 0:41 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-11 0:11 Harald Schreiber
2001-10-11 0:36 ` Jan Niehusmann
2001-10-11 0:41 ` Matthew Dharm [this message]
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=20011010174117.B10273@one-eyed-alien.net \
--to=mdharm-kernel@one-eyed-alien.net \
--cc=harald@harald-schreiber.de \
--cc=jan@gondor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb-devel@lists.sourceforge.net \
/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
all inboxes | Powered by JetHome®