From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from mail.kernel.org ([198.145.29.99]:50396 "EHLO mail.kernel.org"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1729693AbeLTPUC (ORCPT <rfc822;stable@vger.kernel.org>);
        Thu, 20 Dec 2018 10:20:02 -0500
Date: Thu, 20 Dec 2018 16:19:59 +0100
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Oliver Neukum <oneukum@suse.com>
Cc: Macpaul Lin <macpaul.lin@mediatek.com>,
        Lars Melin <larsm17@gmail.com>,
        Andrey Arapov <andrey.arapov@gmail.com>,
        Johan Hovold <johan@kernel.org>,
        Mediatek WSD Upstream <wsd_upstream@mediatek.com>,
        linux-usb@vger.kernel.org, stable@vger.kernel.org
Subject: Re: [PATCH v5] cdc-acm: fix abnormal DATA RX issue for Mediatek
 Preloader.
Message-ID: <20181220151959.GA17408@kroah.com>
References: <1545140229.18594.3.camel@suse.com>
 <1545143195.10193.6.camel@mtkswgap22>
 <1545146389.10193.12.camel@mtkswgap22>
 <5b0e5bf4-0411-0c58-ccae-fe66c937a36b@gmail.com>
 <1545155303.16619.11.camel@mtkswgap22>
 <1545186137.1821.5.camel@mtkswgap22>
 <1545189395.3262.2.camel@mtkswgap22>
 <3465233a-1afd-8112-ebd6-78cceb426c9d@gmail.com>
 <1545192203.4134.11.camel@mtkswgap22>
 <1545209782.31763.2.camel@suse.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1545209782.31763.2.camel@suse.com>
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On Wed, Dec 19, 2018 at 09:56:22AM +0100, Oliver Neukum wrote:
> On Mi, 2018-12-19 at 12:03 +0800, Macpaul Lin wrote:
> > On Wed, 2018-12-19 at 10:31 +0700, Lars Melin wrote:
> > > On 12/19/2018 10:16, Macpaul Lin wrote: 
> > > 
> > > Hi Macpaul,
> > > your verbose usb listing show me that Mediatek has made two different 
> > > 0e8d:003 devices, see my verbose lsusb listing below.
> > > (Notice also the reverse order for cmd and data interfaces in it 
> > > compared to yours).
> > > USB id's are intended to identify a device and its needs so there should
> > > never be more than one unique device per id.
> > > 
> > > 
> > > Fairphone FP-1, MT6227  (no CDC union !!!)
> > > 
> > 
> > Hi Lars,
> > 
> > Ha ha ha, it is a little bit embarrassing.
> > What I've used to capture verbose log is MT6765 platform.
> > Then I've checked Fairphone FP-1, which is MT6589 a pretty old platform.
> > The BROM (boot ROM) has been maintained by other teams and will vary by
> > different SoC project in Mediatek. I'm not sure why they changed the
> > descriptors.
> > 
> > For the consistency of BROM's behavior, I'll update a new patch keeps
> > PID:0003 remain untouched. I'll trying to report it to BROM team and see
> > if they have any action on this issue.
> 
> Thank you all for taking care of these important issues.
> Please submit that new patch.

Can you review v7 please?

thanks,

greg k-h