Re: [PATCH] Nikon D40 Quirks

Ortwin Glück <[email protected]>
Newsgroups gmane.linux.usb.devel
Message-ID <[email protected]>
Alan Stern wrote:
> On Wed, 5 Sep 2007, Ortwin Glück wrote:
> 
>> The D40 needs the same quirks as the other (semi-)professional Nikon cameras.
>> The patch is against 2.6.23-rc5.
>>
>> Details:
>> http://bugs.gentoo.org/show_bug.cgi?id=191431
> 
> Please follow the instructions about keeping the entries in numerical 
> order by Vendor and Product ID.  Your patch adds an entry with Product 
> ID 0x0413 before an entry with Product ID 0x0409, both for the same 
> vendor.
> 
> Alan Stern

Alan,

I wasn't aware of an "order" as there is 0x409 before 0x405 as well. But well,
here is the trivially changed patch. I was under the impression maintainers
would naturally handle such details on the fly.

Cheers

Ortwin

PS: As this is a one time fix, I am no longer subscribed to the list. I hope my
email got through.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/

_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
nikon-d40.patch (text/plain, 634 B)
diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
index 2d92ce3..23c83f3 100644
--- a/drivers/usb/storage/unusual_devs.h
+++ b/drivers/usb/storage/unusual_devs.h
@@ -348,6 +348,13 @@ UNUSUAL_DEV(  0x04b0, 0x0411, 0x0100, 0x0101,
 		US_SC_DEVICE, US_PR_DEVICE, NULL,
 		US_FL_FIX_CAPACITY),
 
+/* Reported by Ortwin Glueck <[email protected]> */
+UNUSUAL_DEV(  0x04b0, 0x0413, 0x0110, 0x0110,
+		"NIKON",
+		"NIKON DSC D40",
+		US_SC_DEVICE, US_PR_DEVICE, NULL,
+		US_FL_FIX_CAPACITY),
+
 /* BENQ DC5330
  * Reported by Manuel Fombuena <[email protected]> and
  * Frank Copeland <[email protected]> */
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.