Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 3 Jan 2019 10:06:43 -0500 (EST)
From:   Alan Stern <stern@rowland.harvard.edu>
To:     Icenowy Zheng <icenowy@aosc.io>
cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        <linux-usb@vger.kernel.org>,
        <usb-storage@lists.one-eyed-alien.net>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2 0/2] USB Storage quirk for SMI SM3350
In-Reply-To: <20190103032618.9237-1-icenowy@aosc.io>
Message-ID: <Pine.LNX.4.44L0.1901031006220.1460-100000@iolanthe.rowland.org>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, 3 Jan 2019, Icenowy Zheng wrote:

> SMI SM3350 UFS-USB bridge controller cannot handle REQUEST SENSE command
> with long sense (96-bytes) well, and will even trap the controller into
> a state that refuses to do read/write command.
> 
> Currently Linux uncondintionally set US_FL_SANE_SENSE for devices
> claiming SPC3+, which makes simply add US_FL_BAD_SENSE for SM3350 fail
> (as it claims SPC4).
> 
> Fix this conflicting quirk issue, and add the quirk for SM3350.
> 
> Icenowy Zheng (2):
>   USB: storage: don't insert sane sense for SPC3+ when bad sense
>     specified
>   USB: storage: add quirk for SMI SM3350
> 
>  drivers/usb/storage/scsiglue.c     |  8 ++++++--
>  drivers/usb/storage/unusual_devs.h | 12 ++++++++++++
>  2 files changed, 18 insertions(+), 2 deletions(-)

For both patches:

Acked-by: Alan Stern <stern@rowland.harvard.edu>