mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Paolo Abeni <pabeni@redhat.com>
To: Slark Xiao <slark_xiao@163.com>
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	Muhammad Nuzaihan <zaihan@unrealasia.net>,
	Qiang Yu <quic_qianyu@quicinc.com>,
	Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
	Johan Hovold <johan@kernel.org>,
	ryazanov.s.a@gmail.com, johannes@sipsolutions.net,
	loic.poulain@oss.qualcomm.com, andrew+netdev@lunn.ch,
	davem@davemloft.net, edumazet@google.com, mani@kernel.org,
	kuba@kernel.org
Subject: Re: [net-next v3 0/8] net: wwan: add NMEA port type support
Date: Wed, 31 Dec 2025 10:06:29 +0100	[thread overview]
Message-ID: <75e23b2c-67bf-41c8-8970-627d22af3147@redhat.com> (raw)
In-Reply-To: <20251231065109.43378-1-slark_xiao@163.com>

On 12/31/25 7:51 AM, Slark Xiao wrote:
> The series introduces a long discussed NMEA port type support for the
> WWAN subsystem. There are two goals. From the WWAN driver perspective,
> NMEA exported as any other port type (e.g. AT, MBIM, QMI, etc.). From
> user space software perspective, the exported chardev belongs to the
> GNSS class what makes it easy to distinguish desired port and the WWAN
> device common to both NMEA and control (AT, MBIM, etc.) ports makes it
> easy to locate a control port for the GNSS receiver activation.
> 
> Done by exporting the NMEA port via the GNSS subsystem with the WWAN
> core acting as proxy between the WWAN modem driver and the GNSS
> subsystem.
> 
> The series starts from a cleanup patch. Then two patches prepares the
> WWAN core for the proxy style operation. Followed by a patch introding a
> new WWNA port type, integration with the GNSS subsystem and demux. The
> series ends with a couple of patches that introduce emulated EMEA port
> to the WWAN HW simulator.
> 
> The series is the product of the discussion with Loic about the pros and
> cons of possible models and implementation. Also Muhammad and Slark did
> a great job defining the problem, sharing the code and pushing me to
> finish the implementation. Many thanks.
> 
> Comments are welcomed.
> 
> Changes since V1:
> Uniformly use put_device() to release port memory. This made code less
> weird and way more clear. Thank you, Loic, for noticing and the fix
> discussion!
> 
> Changes since V2:
> Add supplement of Loic and Slark about some fix
> 
> CC: Slark Xiao <slark_xiao@163.com>
> CC: Muhammad Nuzaihan <zaihan@unrealasia.net>
> CC: Qiang Yu <quic_qianyu@quicinc.com>
> CC: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> CC: Johan Hovold <johan@kernel.org>
> CC: Sergey Ryazanov <ryazanov.s.a@gmail.com>

Uhm... I thingh I already shared the following with you ?!?

## Form letter - net-next-closed

The net-next tree is closed for new drivers, features, code refactoring
and optimizations due to the merge window and the winter break. We are
currently accepting bug fixes only.

Please repost when net-next reopens after Jan 2nd.

RFC patches sent for review only are obviously welcome at any time.


  parent reply	other threads:[~2025-12-31  9:06 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-31  6:51 Slark Xiao
2025-12-31  6:51 ` [net-next v3 1/8] net: wwan: core: remove unused port_id field Slark Xiao
2025-12-31  6:51 ` [net-next v3 2/8] net: wwan: core: split port creation and registration Slark Xiao
2025-12-31  6:51 ` [net-next v3 3/8] net: wwan: core: split port unregister and stop Slark Xiao
2025-12-31  6:51 ` [net-next v3 4/8] net: wwan: add NMEA port support Slark Xiao
2025-12-31 20:13   ` kernel test robot
2025-12-31  6:51 ` [net-next v3 5/8] net: wwan: hwsim: refactor to support more port types Slark Xiao
2025-12-31  6:51 ` [net-next v3 6/8] net: wwan: hwsim: support NMEA port emulation Slark Xiao
2025-12-31  6:51 ` [net-next v3 7/8] net: wwan: prevent premature device unregister when NMEA port is present Slark Xiao
2025-12-31  6:51 ` [net-next v3 8/8] net: wwan: mhi_wwan_ctrl: Add NMEA channel support Slark Xiao
2025-12-31  9:06 ` Paolo Abeni [this message]
2025-12-31  9:52   ` Re:Re: [net-next v3 0/8] net: wwan: add NMEA port type support Slark Xiao

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=75e23b2c-67bf-41c8-8970-627d22af3147@redhat.com \
    --to=pabeni@redhat.com \
    --cc=andrew+netdev@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=johan@kernel.org \
    --cc=johannes@sipsolutions.net \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=loic.poulain@oss.qualcomm.com \
    --cc=mani@kernel.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=netdev@vger.kernel.org \
    --cc=quic_qianyu@quicinc.com \
    --cc=ryazanov.s.a@gmail.com \
    --cc=slark_xiao@163.com \
    --cc=zaihan@unrealasia.net \
    /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

all inboxes | Powered by JetHome®