mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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 --]

      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®