diff options
author | Hans de Goede <hdegoede@redhat.com> | 2010-10-01 17:20:11 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-10-22 13:22:05 -0400 |
commit | 00914025cc4e783d4703b4db1d47b41f389e50c8 (patch) | |
tree | 0cd74cdf181c1bb3a6dbfbecea7d5e6ed8a3d028 | |
parent | 5ce524bdff367b4abda20bcfd4dafd9d30c773df (diff) |
usb-storage: add new no_read_capacity_16 quirk
Some Rockbox based mp4 players will crash when ever they see a
read_capacity_16 scsi command. So add a new US_FL which tells the scsi sd
driver to not issue any read_capacity_16 scsi commands.
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>
-rw-r--r-- | drivers/usb/storage/scsiglue.c | 4 | ||||
-rw-r--r-- | drivers/usb/storage/unusual_devs.h | 3 | ||||
-rw-r--r-- | include/linux/usb_usual.h | 4 |
3 files changed, 9 insertions, 2 deletions
diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c index a1128ff5cc2c..a688b1e686ea 100644 --- a/drivers/usb/storage/scsiglue.c +++ b/drivers/usb/storage/scsiglue.c | |||
@@ -209,6 +209,10 @@ static int slave_configure(struct scsi_device *sdev) | |||
209 | if (us->fflags & US_FL_CAPACITY_HEURISTICS) | 209 | if (us->fflags & US_FL_CAPACITY_HEURISTICS) |
210 | sdev->guess_capacity = 1; | 210 | sdev->guess_capacity = 1; |
211 | 211 | ||
212 | /* Some devices cannot handle READ_CAPACITY_16 */ | ||
213 | if (us->fflags & US_FL_NO_READ_CAPACITY_16) | ||
214 | sdev->no_read_capacity_16 = 1; | ||
215 | |||
212 | /* assume SPC3 or latter devices support sense size > 18 */ | 216 | /* assume SPC3 or latter devices support sense size > 18 */ |
213 | if (sdev->scsi_level > SCSI_SPC_2) | 217 | if (sdev->scsi_level > SCSI_SPC_2) |
214 | us->fflags |= US_FL_SANE_SENSE; | 218 | us->fflags |= US_FL_SANE_SENSE; |
diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h index c8264ff5457e..6ccdd3dd5259 100644 --- a/drivers/usb/storage/unusual_devs.h +++ b/drivers/usb/storage/unusual_devs.h | |||
@@ -877,7 +877,8 @@ UNUSUAL_DEV( 0x071b, 0x3203, 0x0000, 0x0000, | |||
877 | "RockChip", | 877 | "RockChip", |
878 | "MP3", | 878 | "MP3", |
879 | USB_SC_DEVICE, USB_PR_DEVICE, NULL, | 879 | USB_SC_DEVICE, USB_PR_DEVICE, NULL, |
880 | US_FL_NO_WP_DETECT | US_FL_MAX_SECTORS_64), | 880 | US_FL_NO_WP_DETECT | US_FL_MAX_SECTORS_64 | |
881 | US_FL_NO_READ_CAPACITY_16), | ||
881 | 882 | ||
882 | /* Reported by Jean-Baptiste Onofre <jb@nanthrax.net> | 883 | /* Reported by Jean-Baptiste Onofre <jb@nanthrax.net> |
883 | * Support the following product : | 884 | * Support the following product : |
diff --git a/include/linux/usb_usual.h b/include/linux/usb_usual.h index e62e9fe08883..71693d4a4fe1 100644 --- a/include/linux/usb_usual.h +++ b/include/linux/usb_usual.h | |||
@@ -60,7 +60,9 @@ | |||
60 | US_FLAG(BAD_SENSE, 0x00020000) \ | 60 | US_FLAG(BAD_SENSE, 0x00020000) \ |
61 | /* Bad Sense (never more than 18 bytes) */ \ | 61 | /* Bad Sense (never more than 18 bytes) */ \ |
62 | US_FLAG(NO_READ_DISC_INFO, 0x00040000) \ | 62 | US_FLAG(NO_READ_DISC_INFO, 0x00040000) \ |
63 | /* cannot handle READ_DISC_INFO */ | 63 | /* cannot handle READ_DISC_INFO */ \ |
64 | US_FLAG(NO_READ_CAPACITY_16, 0x00080000) \ | ||
65 | /* cannot handle READ_CAPACITY_16 */ | ||
64 | 66 | ||
65 | #define US_FLAG(name, value) US_FL_##name = value , | 67 | #define US_FLAG(name, value) US_FL_##name = value , |
66 | enum { US_DO_ALL_FLAGS }; | 68 | enum { US_DO_ALL_FLAGS }; |