Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: odB3g+RU3IH2RLqLKfzXgbP2VWd9rKntlQjm+X0O3/IFZ7ylLkiQvvOCazVA6Zzzw8+1geo+5B
 7UfZRo40egFg==
IronPort-HdrOrdr: A9a23:wN6WBaopHbSmywuwTiolrGUaV5oueYIsimQD101hICG8cqSj+/
 xG/c5rsCMc5wx+ZJhNo7q90ey7IU80lqQFg7X5X43CYOCOggLBR72Kr7GSoAEIcxeOktK1vp
 0OT0ERMrDN5Q8Qt7ed3ODuKadG/OW6
From:   Stefan =?ISO-8859-1?Q?Br=FCns?= <stefan.bruens@rwth-aachen.de>
To:     Johan Hovold <johan@kernel.org>
CC:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        <linux-usb@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Daniele Palmas <dnlplm@gmail.com>,
        =?ISO-8859-1?Q?Bj=F8rn?= Mork <bjorn@mork.no>
Subject: Re: [PATCH v2] USB: serial: qcserial: Support for SDX55 based Sierra Wireless 5G modules
Date:   Thu, 1 Jul 2021 18:41:26 +0200
Message-ID: <3614092.7mX7SIBJgt@pebbles>
In-Reply-To: <YNQ0O0vhtpStp0n/@hovoldconsulting.com>
References: <20210611134507.8780-1-stefan.bruens@rwth-aachen.de> <20210611135842.14415-1-stefan.bruens@rwth-aachen.de> <YNQ0O0vhtpStp0n/@hovoldconsulting.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="nextPart16585872.0GYKUT4djp";
        micalg=pgp-sha1; protocol="application/pgp-signature"
Precedence: bulk

--nextPart16585872.0GYKUT4djp
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="iso-8859-1"; protected-headers="v1"
From: Stefan =?ISO-8859-1?Q?Br=FCns?= <stefan.bruens@rwth-aachen.de>
To: Johan Hovold <johan@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Daniele Palmas <dnlplm@gmail.com>, =?ISO-8859-1?Q?Bj=F8rn?= Mork <bjorn@mork.no>
Subject: Re: [PATCH v2] USB: serial: qcserial: Support for SDX55 based Sierra Wireless 5G modules
Date: Thu, 01 Jul 2021 18:41:26 +0200
Message-ID: <3614092.7mX7SIBJgt@pebbles>
In-Reply-To: <YNQ0O0vhtpStp0n/@hovoldconsulting.com>
References: <20210611134507.8780-1-stefan.bruens@rwth-aachen.de> <20210611135842.14415-1-stefan.bruens@rwth-aachen.de> <YNQ0O0vhtpStp0n/@hovoldconsulting.com>

On Donnerstag, 24. Juni 2021 09:28:59 CEST Johan Hovold wrote:
> [ +CC: Daniele and Bj=F8rn ]
>=20
> On Fri, Jun 11, 2021 at 03:58:41PM +0200, Stefan Br=FCns wrote:
> > The devices exposes two different interface compositions:
> > - QDL mode, single interface
> > - MBIM mode, MBIM class compliant plus AT/DM(/ADB)
> >=20
> > Current firmware versions (up to 01.07.19) do not expose an NMEA port.
>=20
> We already have at least one SDX55 based modem (FN980) supported by the
> option driver. Any particular reason why you chose to add it to qcserial
> instead of option?
>=20
Support for qualcomm based modems are spread over option and qcserial. All=
=20
other Sierra devices are supported by qcserial.
=20
> Note that the FN980 also needs the ZLP flag set in QDL (flashing) mode,
> I'd assume this one needs it too.

It depends if you implement the Firehose protocol in accordance to the=20
specification or not. 80-NG319-1 (Firehose specification) explicitly states=
 to=20
pad any XML command packet which is an exact multiple of 512 bytes with an=
=20
extra newline or other whitespace character.

Regards,

Stefan

=2D-=20
Stefan Br=FCns  /  Bergstra=DFe 21  /  52062 Aachen
home: +49 241 53809034     mobile: +49 151 50412019
--nextPart16585872.0GYKUT4djp
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part.
Content-Transfer-Encoding: 7Bit

-----BEGIN PGP SIGNATURE-----

iF0EABECAB0WIQSwWRWIpJbl0W4DemNvf0o9jP6qUwUCYN3wNgAKCRBvf0o9jP6q
U+TBAJ0QV69j/+RbvDCsiyexV8sI0ngz5ACffbHYfFMNzDVGim05566P0eAQLjk=
=Uihm
-----END PGP SIGNATURE-----

--nextPart16585872.0GYKUT4djp--