From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f173.google.com ([209.85.192.173]:35713 "EHLO mail-pd0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753857AbbGFPnX (ORCPT ); Mon, 6 Jul 2015 11:43:23 -0400 Subject: Re: [PATCH] usb-storage: ignore ZTE MF 823 card reader in mode 0x1225 To: Oliver Neukum , mdharm-usb@one-eyed-alien.net, gregkh@linuxfoundation.org, linux-usb@vger.kernel.org References: <1436181152-4206-1-git-send-email-oneukum@suse.com> Cc: stable@vger.kernel.org From: Lars Melin Message-ID: <559AA216.4030801@gmail.com> Date: Mon, 6 Jul 2015 22:43:18 +0700 MIME-Version: 1.0 In-Reply-To: <1436181152-4206-1-git-send-email-oneukum@suse.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: stable-owner@vger.kernel.org List-ID: 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 > 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 > + * 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 > * This encrypted pen drive returns bogus data for the initial READ(10). > */ > Looks good now Reviewed-by: Lars Melin rgds Lars