[PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225

[PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225

[Oliver Neukum]

This device automatically switches itself to another mode (0x1405)
unless the specific access pattern of Windows is followed in its
initial mode. That makes a dirty unmount of the internal storage
devices inevitable if they are mounted. So the card reader of
such a device should be ignored, lest an unclean removal become
inevitable.

This replaces an earlier patch that ignored all LUNs of this device.
That patch was overly broad.

Signed-off-by: Oliver Neukum <oneukum@suse.com>
CC: stable@vger.kernel.org
---
 drivers/usb/storage/unusual_devs.h | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
index caf1888..87898ca 100644
--- a/drivers/usb/storage/unusual_devs.h
+++ b/drivers/usb/storage/unusual_devs.h
@@ -2065,6 +2065,18 @@ UNUSUAL_DEV( 0x1908, 0x3335, 0x0200, 0x0200,
 		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
 		US_FL_NO_READ_DISC_INFO ),
 
+/* Reported by Oliver Neukum <oneukum@suse.com>
+ * This device morphes spontaneously into another device if the access
+ * pattern of Windows isn't followed. Thus writable media would be dirty
+ * if the initial instance is used. So the device is limited to its
+ * virtual CD.
+ * And yes, the concept that BCD goes up to 9 is not heeded */
+UNUSUAL_DEV( 0x19d2, 0x1225, 0x0000, 0xffff,
+		"ZTE,Incorporated",
+		"ZTE WCDMA Technologies MSM",
+		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
+		US_FL_SINGLE_LUN ),
+
 /* Reported by Sven Geggus <sven-usbst@geggus.net>
  * This encrypted pen drive returns bogus data for the initial READ(10).
  */
-- 
2.1.4

Re: [PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225

[Lars Melin]

On 2015-07-06 18:12, Oliver Neukum wrote:
> This device automatically switches itself to another mode (0x1405)
> unless the specific access pattern of Windows is followed in its
> initial mode. That makes a dirty unmount of the internal storage
> devices inevitable if they are mounted. So the card reader of
> such a device should be ignored, lest an unclean removal become
> inevitable.
>
> This replaces an earlier patch that ignored all LUNs of this device.
> That patch was overly broad.
>
> Signed-off-by: Oliver Neukum <oneukum@suse.com>
> CC: stable@vger.kernel.org
> ---
>   drivers/usb/storage/unusual_devs.h | 12 ++++++++++++
>   1 file changed, 12 insertions(+)
>
> diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
> index caf1888..87898ca 100644
> --- a/drivers/usb/storage/unusual_devs.h
> +++ b/drivers/usb/storage/unusual_devs.h
> @@ -2065,6 +2065,18 @@ UNUSUAL_DEV( 0x1908, 0x3335, 0x0200, 0x0200,
>   		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
>   		US_FL_NO_READ_DISC_INFO ),
>
> +/* Reported by Oliver Neukum <oneukum@suse.com>
> + * This device morphes spontaneously into another device if the access
> + * pattern of Windows isn't followed. Thus writable media would be dirty
> + * if the initial instance is used. So the device is limited to its
> + * virtual CD.
> + * And yes, the concept that BCD goes up to 9 is not heeded */
> +UNUSUAL_DEV( 0x19d2, 0x1225, 0x0000, 0xffff,
> +		"ZTE,Incorporated",
> +		"ZTE WCDMA Technologies MSM",
> +		USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> +		US_FL_SINGLE_LUN ),
> +
>   /* Reported by Sven Geggus <sven-usbst@geggus.net>
>    * This encrypted pen drive returns bogus data for the initial READ(10).
>    */
>
Looks good now

Reviewed-by: Lars Melin <larsm17@gmail.com>

rgds
Lars