diff options
author | Hans de Goede <hdegoede@redhat.com> | 2010-10-01 17:20:08 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-22 13:22:04 -0400 |
commit | 8e04d8056c1ea0e0aab730994b74756f0526cda8 (patch) | |
tree | b81a3e2f5993c69559d120fa96840e1ef70b42d4 /drivers/scsi | |
parent | 319feaabb6c7ccd90da6e3207563c265da7d21ae (diff) |
scsi/sr: add no_read_disc_info scsi_device flag
Some USB devices emulate a usb-mass-storage attached (scsi) cdrom device,
usually this fake cdrom contains the windows software for the device.
While working on supporting Appotech ax3003 based photoframes, which do
this I discovered that they will go of into lala land when ever they see a
READ_DISC_INFO scsi command.
Thus this patch adds a scsi_device flag (which can then be set by the
usb-storage driver through an unsual-devs entry), to indicate this, and
makes the sr driver honor this flag.
I know this sucks, but as discussed on linux-scsi list there is no other
way to make this device work properly.
Looking at usb traces made under windows, windows never sends a
READ_DISC_INFO during normal interactions with a usb cdrom device. So as
this cdrom emulation thingie becomes more common we might see more of this
problem.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: Matthew Dharm <mdharm-usb@one-eyed-alien.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/sr.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/scsi/sr.c b/drivers/scsi/sr.c index ba9c3e0387ce..b811dd0eb240 100644 --- a/drivers/scsi/sr.c +++ b/drivers/scsi/sr.c | |||
@@ -862,10 +862,16 @@ static void get_capabilities(struct scsi_cd *cd) | |||
862 | static int sr_packet(struct cdrom_device_info *cdi, | 862 | static int sr_packet(struct cdrom_device_info *cdi, |
863 | struct packet_command *cgc) | 863 | struct packet_command *cgc) |
864 | { | 864 | { |
865 | struct scsi_cd *cd = cdi->handle; | ||
866 | struct scsi_device *sdev = cd->device; | ||
867 | |||
868 | if (cgc->cmd[0] == GPCMD_READ_DISC_INFO && sdev->no_read_disc_info) | ||
869 | return -EDRIVE_CANT_DO_THIS; | ||
870 | |||
865 | if (cgc->timeout <= 0) | 871 | if (cgc->timeout <= 0) |
866 | cgc->timeout = IOCTL_TIMEOUT; | 872 | cgc->timeout = IOCTL_TIMEOUT; |
867 | 873 | ||
868 | sr_do_ioctl(cdi->handle, cgc); | 874 | sr_do_ioctl(cd, cgc); |
869 | 875 | ||
870 | return cgc->stat; | 876 | return cgc->stat; |
871 | } | 877 | } |