Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 24 Jul 2022 16:26:05 +0200
From:   Johan Hovold <johan@kernel.org>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Reinhard Speyerer <rspmn@arcor.de>, sdlyyxy <sdlyyxy@bupt.edu.cn>,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] USB: usb-serial-simple: add new device id for OPPO R11
Message-ID: <Yt1WfSZk03Plpnan@hovoldconsulting.com>
References: <20220715142444.4173681-1-gregkh@linuxfoundation.org>
 <119D7B0F-7809-464A-AFF1-DF72FFF9E63F@bupt.edu.cn>
 <YtKrbucYNulPEKUp@arcor.de>
 <YtRtswctFMLxeglu@kroah.com>
 <YtXG3EVrRKAG7WVx@arcor.de>
 <YtwjiWG5ZFBvCZ1M@hovoldconsulting.com>
 <Yt1QhCIiUf97Dl3D@kroah.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Yt1QhCIiUf97Dl3D@kroah.com>
Precedence: bulk

On Sun, Jul 24, 2022 at 04:00:36PM +0200, Greg Kroah-Hartman wrote:
> On Sat, Jul 23, 2022 at 06:36:25PM +0200, Johan Hovold wrote:
> > On Mon, Jul 18, 2022 at 10:47:24PM +0200, Reinhard Speyerer wrote:

> > > Please don't give the OPPO R11 diag port on Linux a bad name by letting
> > > the usb-serial-simple driver handle it.
> > 
> > So while I'm not sure bandwidth is really a problem, I still tend to
> > agree that we should add this one to the option driver for now as that
> > is how we handle (non-GOBI) Qualcomm modems and their QCDM ports.
> 
> If you want it to stay on the option driver, that's fine, but I still
> think it feels odd as it obviously does not follow the vendor-specific
> protocol that the option driver supports.

But we've been dumping modem device-id entries in there since forever.

The entries added to option have been for devices whose interfaces did
not follow any particular pattern (e.g. unlike the old GOBI modems).

And as Reinhard mentioned, the line-control requests (which follow CDC)
are actually required by some Qualcomm modems so moving things out would
need to be done carefully.

On the other hand, that request likely isn't needed for any QCDM/DIAG
ports, but who knows for sure.

> To be fair, loads of the ids in that driver could move to the simple
> driver as they probably do not also support the line setting protocol
> that this driver was originally written for, so what's a few more ids
> added :)

Not sure about the simple driver, unless we want to handle say NMEA and
MODEM ports using different drivers. But either way, for now I think we
can add a few more to option.

If we start seeing patterns regarding the Qualcomm interface
descriptors, like the QCDM interface using 0xff/0xff/0x30, we could
implement something more generic in qcserial too.

But the lack of documentation and unification is just a pain to deal
with. So someone would need to be motivated enough to try to organise
this mess.

Johan