Hi, On 09/13/2014 10:27 PM, Sergei Shtylyov wrote: > Hello. > > On 9/13/2014 10:21 PM, Thomas Backlund wrote: > >>> And set this quirk for the Seagate Expansion Desk (0bc2:2312), as that one >>> seems to hang upon receiving an ATA_12 or ATA_16 command. > >>> https://bugzilla.kernel.org/show_bug.cgi?id=79511 > >>> While at it also add missing documentation for the u value for usb-storage >>> quirks. > >>> Cc: stable@xxxxxxxxxxxxxxx # 3.16 >>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > >>> -- >>> Changes in v2: Add documentation for new t and u usb-storage.quirks flags >>> --- >>> Documentation/kernel-parameters.txt | 3 +++ >>> drivers/usb/storage/uas.c | 13 +++++++++++++ >>> drivers/usb/storage/unusual_uas.h | 16 ++++++---------- >>> drivers/usb/storage/usb.c | 6 +++++- >>> include/linux/usb_usual.h | 2 ++ >>> 5 files changed, 29 insertions(+), 11 deletions(-) > >>> diff --git a/Documentation/kernel-parameters.txt >>> b/Documentation/kernel-parameters.txt >>> index 5ae8608..7c32053 100644 >>> --- a/Documentation/kernel-parameters.txt >>> +++ b/Documentation/kernel-parameters.txt >>> @@ -3541,6 +3541,9 @@ bytes respectively. Such letter suffixes can also be >>> entirely omitted. >>> bogus residue values); >>> s = SINGLE_LUN (the device has only one >>> Logical Unit); >>> + t = NO_ATA_1X (don't allow ATA12 and ATA12 >>> + commands, uas only); > >> I guess you meant ATA12 and *ATA16* > > ... or even ATA(12) and ATA(16). As far as I remember SCSI, it has CDB length in parens for the command names. Right, ugh. v3 coming up ... Thanks, Hans -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html