Bug 188521 - I/O error when reading from Nikon D80 V1.01 camera
Bug#: 188521 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: kernel@gentoo.org Reported By: kscherer@primus.ca
Component: Core system
URL:  http://marc.info/?l=linux-usb-devel&m=118724455113424&w=2
Summary: I/O error when reading from Nikon D80 V1.01 camera
Keywords:  
Status Whiteboard: InSVN, InCVS
Opened: 2007-08-12 00:12 0000
Description:   Opened: 2007-08-12 00:12 0000 
This is closely related to bug #186326
Running kernel 2.6.22-gentoo-r2 and udev 114.
The Nikon D80 is listed in drivers/usb/storage/unusual_devs.h but the max
version is 0x0100. I recently upgraded my camera to firmware version 1.01 and
in order to be able to mount my camera I had to make the following small
update.
338c338
< UNUSUAL_DEV(  0x04b0, 0x0411, 0x0100, 0x0100,
---
> UNUSUAL_DEV(  0x04b0, 0x0411, 0x0100, 0x0101,

Perhaps bumping the max version number to 0x0200 for this and other nikon
cameras will avoid future patches like this one. 


Reproducible: Always

Steps to Reproduce:
1. Attach Nikon D80 and turn it on.
2. 
3.

Actual Results:  
Exact same error message as bug #186326

Expected Results:  
Should be able to mount the filesystem.

------- Comment #1 From Jakub Moc (RETIRED) 2007-08-12 04:00:53 0000 ------- 
Folks, do we really need *3* bugs for the very same issue? Why can't this be
fixed for all affected devices at once?

------- Comment #2 From Mike Pagano 2007-08-13 23:43:03 0000 ------- 
Created an attachment (id=128003) [details]
Formatted patch for potential upstream submission

This is your change formatted as a proper patch. Can you test this? If this
works, we can send this upstream for eventually inclusion into the mainline
kernel.

You can apply it to 2.6.22-r2 as
follows:usb-storage-nikon-d80-firmware-upgrade.patch

cd /usr/src/linux (or proper directory where the source is located)
patch -p1 < usb-storage-nikon-d80-firmware-upgrade.patch

------- Comment #3 From Konrad Scherer 2007-08-14 22:30:01 0000 ------- 
I have verified that the patch works.
Thanks.

------- Comment #4 From Mike Pagano 2007-08-15 00:42:09 0000 ------- 
Konrad,

Thank-you very much.

Can you do one more thing?

Can you attach the contents of /proc/bus/usb/devices with your Nikon plugged
in?

------- Comment #5 From Konrad Scherer 2007-08-15 02:00:31 0000 ------- 
Here is the output from /proc/bus/usb/devices for the Nikon.

T:  Bus=01 Lev=01 Prnt=01 Port=07 Cnt=01 Dev#=  5 Spd=480 MxCh= 0
D:  Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs=  1
P:  Vendor=04b0 ProdID=0411 Rev= 1.01
S:  Manufacturer=NIKON
S:  Product=NIKON DSC D80
S:  SerialNumber=5007206
C:* #Ifs= 1 Cfg#= 1 Atr=c0 MxPwr=  2mA
I:* If#= 0 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50 Driver=usb-storage
E:  Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E:  Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms

Thank you taking care of this.

------- Comment #6 From Mike Pagano 2007-08-15 14:56:53 0000 ------- 
OK, thanks Konrad for your assistance. I have submitted the patch upstream so I
am going to close this bug as such.

I'll keep an eye on the patch to see if gets traction but we shouldn't expect
anything before 2.6.24 since the .23 window has been closed for a few weeks.

------- Comment #7 From Mike Pagano 2007-08-16 20:21:01 0000 ------- 
ok, the patch was accepted upstream, so hopefully 2.6.24 will see the change.

------- Comment #8 From Mike Pagano 2007-08-29 23:03:39 0000 ------- 
Reopened in anticipation of this patch eventually getting into genpatches for
2.6.22

Bug will be closed once the gentoo-sources revision containing the patch is
released.

------- Comment #9 From Christian Heim (RETIRED) 2007-09-01 09:13:17 0000 ------- 
genpatches-2.6.22-7 has been released yesterday, gentoo-sources-2.6.22-r6 is
using it.