mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jan Niehusmann <jan@gondor.com>
To: Harald Schreiber <harald@harald-schreiber.de>
Cc: mdharm-usb@one-eyed-alien.net, 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: Thu, 11 Oct 2001 02:36:47 +0200	[thread overview]
Message-ID: <20011011023647.A14615@gondor.com> (raw)
In-Reply-To: <m3elobqblf.fsf@harald-schreiber.de>
In-Reply-To: <m3elobqblf.fsf@harald-schreiber.de>

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


  reply	other threads:[~2001-10-11  0:36 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 [this message]
2001-10-11  0:41   ` Matthew Dharm

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=20011011023647.A14615@gondor.com \
    --to=jan@gondor.com \
    --cc=harald@harald-schreiber.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=mdharm-usb@one-eyed-alien.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®