public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
From: Lars Melin <larsm17@gmail.com>
To: Ivan Shapovalov <intelfx@intelfx.name>, linux-kernel@vger.kernel.org
Cc: "Bjørn Mork" <bjorn@mork.no>,
	"Andrew Lunn" <andrew+netdev@lunn.ch>,
	"David S. Miller" <davem@davemloft.net>,
	"Eric Dumazet" <edumazet@google.com>,
	"Jakub Kicinski" <kuba@kernel.org>,
	"Paolo Abeni" <pabeni@redhat.com>,
	"Johan Hovold" <johan@kernel.org>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	netdev@vger.kernel.org, linux-usb@vger.kernel.org
Subject: Re: [PATCH 1/5] net: usb: qmi_wwan: add IDs for EM7565 (9X50) in "MBIM USBIF" config
Date: Tue, 19 Nov 2024 11:55:10 +0700	[thread overview]
Message-ID: <26889f86-82bd-4bf2-808e-7476ad6671f7@gmail.com> (raw)
In-Reply-To: <20241117083204.57738-1-intelfx@intelfx.name>


On 2024-11-17 15:31, Ivan Shapovalov wrote:
> This change was discovered and tested using author's own EM7565 device.
> 
> This "MBIM USBIF" configuration corresponds to the `AT!USBCOMP=1,1,xxx`
> on-device USB composition setting. When activated, the VID:PID resets
> to 1199:90b1 ("application" mode) + 1199:90b0 ("boot" mode, i.e. QDL).

Hi Ivan,
for the whole series 1-5, please include an lsusb -v or usb-devices 
listing for any device that you add support for.
What you find in device firmware or in Qualcomm SDK drivers does not 
necessarily reflect what has actually been manufactured and consequently 
  how those Id's could be used for a future different product.

The 1199:90d3 is already supported by the option serial driver and that 
is where devices with interfaces supported by their unique interface 
attributes belong instead of being supported by interface number in 
qcserial.

thanks
Lars




      parent reply	other threads:[~2024-11-19  4:55 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-11-17  8:31 [PATCH 1/5] net: usb: qmi_wwan: add IDs for EM7565 (9X50) in "MBIM USBIF" config Ivan Shapovalov
2024-11-17  8:31 ` [PATCH 2/5] USB: serial: qcserial: add EM7565 (9X50) IDs and configurations Ivan Shapovalov
2024-11-17  8:31 ` [PATCH 3/5] USB: serial: qcserial: add EM9xxx (SDX55) " Ivan Shapovalov
2024-11-17  8:31 ` [PATCH 4/5] USB: serial: qcserial: disable autosuspend for QDL configurations Ivan Shapovalov
2024-11-17  8:31 ` [PATCH 5/5] USB: serial: qcserial: enable ZLP for non-QDL interfaces Ivan Shapovalov
2024-11-19  4:55 ` Lars Melin [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=26889f86-82bd-4bf2-808e-7476ad6671f7@gmail.com \
    --to=larsm17@gmail.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=bjorn@mork.no \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=intelfx@intelfx.name \
    --cc=johan@kernel.org \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox