linux-kernel - Re: [PATCH v2] USB: usb-storage: unusual_devs update for TrekStor DataStation maxi g.u external hard drive enclosure

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4D270213.7000104@t-online.de>
Date:	Fri, 07 Jan 2011 13:07:47 +0100
From:	Richard Schütz <r.schtz@...nline.de>
To:	linux-usb@...r.kernel.org
CC:	linux-kernel@...r.kernel.org,
	Alan Stern <stern@...land.harvard.edu>,
	Sergey Vlasov <vsu@...linux.ru>, Greg KH <gregkh@...e.de>,
	Matthew Dharm <mdharm-usb@...-eyed-alien.net>
Subject: Re: [PATCH v2] USB: usb-storage: unusual_devs update for TrekStor
 DataStation maxi g.u external hard drive enclosure

> The TrekStor DataStation maxi g.u external hard drive enclosure uses a
> JMicron USB to SATA chip which needs the US_FL_IGNORE_RESIDUE flag to work
> properly.
>
> Signed-off-by: Richard Schütz <r.schtz@...nline.de>
> ---
>
> Applies to 2.6.37-rc7 now.
>
> --- a/drivers/usb/storage/unusual_devs.h 2010-12-21 20:26:40.000000000 +0100
> +++ b/drivers/usb/storage/unusual_devs.h 2010-12-22 13:37:58.438818902 +0100
> @@ -1872,6 +1872,15 @@ UNUSUAL_DEV( 0x1908, 0x3335, 0x0200, 0x0
> USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> US_FL_NO_READ_DISC_INFO ),
>
> +/* Patch by Richard Schütz <r.schtz@...nline.de>
> + * This external hard drive enclosure uses a JMicron chip which
> + * needs the US_FL_IGNORE_RESIDUE flag to work properly. */
> +UNUSUAL_DEV( 0x1e68, 0x001b, 0x0000, 0x0000,
> + "TrekStor GmbH & Co. KG",
> + "DataStation maxi g.u",
> + USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> + US_FL_IGNORE_RESIDUE | US_FL_SANE_SENSE ),
> +
> UNUSUAL_DEV( 0x2116, 0x0320, 0x0001, 0x0001,
> "ST",
> "2A",
>

Why was this simple patch not accepted (yet)? Is something wrong with it?

Discussion was here:
http://thread.gmane.org/gmane.linux.usb.general/40545
http://thread.gmane.org/gmane.linux.usb.general/40550

-- 
Richard Schütz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/