linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Aleksander Morgado <aleksander@aleksander.es>
To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: Jakub Kicinski <kuba@kernel.org>,
	Loic Poulain <loic.poulain@linaro.org>,
	Greg KH <gregkh@linuxfoundation.org>,
	David Miller <davem@davemloft.net>,
	linux-arm-msm <linux-arm-msm@vger.kernel.org>,
	open list <linux-kernel@vger.kernel.org>,
	Jeffrey Hugo <jhugo@codeaurora.org>,
	Bhaumik Bhatt <bbhatt@codeaurora.org>,
	Network Development <netdev@vger.kernel.org>
Subject: Re: [RESEND PATCH v18 0/3] userspace MHI client interface driver
Date: Sun, 28 Feb 2021 15:12:42 +0100	[thread overview]
Message-ID: <CAAP7uc+Q=ToKVNz4wDv0JWHK4NTniSLE1QwMbP0eXEqVMTUwwQ@mail.gmail.com> (raw)
In-Reply-To: <20210210062531.GA13668@work>

Hey Manivannan, Jakub & all,

>
> So please let us know the path forward on this series. We are open to
> any suggestions but you haven't provided one till now.
>

I just found out that Sierra Wireless also provides their own version
of mhi-net and mhi-uci in precompiled binaries for several Ubuntu
kernel versions and other setups; and that made me extremely unhappy.
They're not the only manufacturer doing that; most of them are doing
it, because we don't have yet a common solution in upstream Linux. Not
the first time we've seen this either, see the per-vendor GobiNet
implementations vs the upstream qmi_wwan one. I was hoping we could
avoid that mess again with the newer Qualcomm modules! :)

In ModemManager we've always *forced* all manufacturers we interact
with to first do the work in upstream Linux, and then we integrate
support in MM for those drivers. We've never accepted support for
vendor-specific proprietary kernel drivers, and that's something I
would personally like to keep on doing. The sad status right now is
that any user that wants to use the newer 5G modules with Qualcomm
chipsets, they need to go look for manufacturer-built precompiled
drivers for their specific kernel, and also then patch ModemManager
and the tools themselves. Obviously almost no one is doing all that,
except for some company with resources or a lot of interest. Some of
these new 5G modules are PCIe-only by default, unless some pin in the
chipset is brought up and then some of them may switch to USB support.
No one is really doing that either, as tampering with the hardware
voids warranty.

The iosm driver is also stalled in the mailing list and there doesn't
seem to be a lot of real need for a new common wwan subsystem to
rework everything...

I'm not involved with the mhi-uci driver development at all, and I
also don't have anything to say on what goes in the upstream kernel
and what doesn't. But as one of the ModemManager/libqmi/libmbim
maintainers I would like to represent all the users of these modules
that are right now forced to look for shady binary precompiled drivers
out there... that is no better solution than this proposed mhi-uci
common driver.

Manivannan, are you attempting to rework the mhi-uci driver in a
different way, or have you given up? Is there anything I could help
with?

Jakub, is there really no way you can be convinced that this mhi-uci
driver isn't that bad after all? :) All the mhi-net bits are already
integrated I think, even the MBIM support over MHI in the net device,
but all that is truly useless without a way to control the modem
sending and receiving messages.

Sorry for being back again with this discussion :)

Cheers!


--
Aleksander
https://aleksander.es

  parent reply	other threads:[~2021-02-28 14:13 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-06 18:44 [RESEND PATCH v18 0/3] userspace MHI client interface driver Hemant Kumar
2021-01-06 18:44 ` [RESEND PATCH v18 1/3] bus: mhi: core: Move MHI_MAX_MTU to external header file Hemant Kumar
2021-01-06 18:44 ` [RESEND PATCH v18 2/3] docs: Add documentation for userspace client interface Hemant Kumar
2021-01-06 18:44 ` [RESEND PATCH v18 3/3] bus: mhi: Add userspace client interface driver Hemant Kumar
2021-01-13 15:26 ` [RESEND PATCH v18 0/3] userspace MHI " Manivannan Sadhasivam
2021-01-19  9:42   ` Manivannan Sadhasivam
2021-01-19 10:28     ` Greg KH
2021-01-27 15:15   ` Greg KH
2021-01-27 16:24     ` Bjorn Andersson
2021-02-01 10:55     ` Manivannan Sadhasivam
2021-02-01 11:15       ` Greg KH
2021-02-01 12:13         ` Manivannan Sadhasivam
2021-02-02  4:22           ` Manivannan Sadhasivam
2021-02-03  4:10             ` Jakub Kicinski
2021-02-03  4:15               ` Manivannan Sadhasivam
2021-02-03 18:05                 ` Jakub Kicinski
2021-02-03 18:28                   ` Loic Poulain
2021-02-03 18:40                     ` Jakub Kicinski
2021-02-04  4:07                       ` Manivannan Sadhasivam
2021-02-04  5:53                       ` Bjorn Andersson
2021-02-09  9:20                       ` Aleksander Morgado
2021-02-09 16:17                         ` Jakub Kicinski
2021-02-09 16:49                           ` Aleksander Morgado
2021-02-10  6:25                           ` Manivannan Sadhasivam
2021-02-10 18:41                             ` Jakub Kicinski
2021-02-10 19:18                               ` Jeffrey Hugo
2021-02-10 22:08                               ` Bjorn Andersson
2021-02-11  9:26                               ` Aleksander Morgado
2021-02-28 14:12                             ` Aleksander Morgado [this message]
2021-02-28 15:52                               ` Manivannan Sadhasivam
2021-02-03 18:34                   ` Bjorn Andersson

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='CAAP7uc+Q=ToKVNz4wDv0JWHK4NTniSLE1QwMbP0eXEqVMTUwwQ@mail.gmail.com' \
    --to=aleksander@aleksander.es \
    --cc=bbhatt@codeaurora.org \
    --cc=davem@davemloft.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=jhugo@codeaurora.org \
    --cc=kuba@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=loic.poulain@linaro.org \
    --cc=manivannan.sadhasivam@linaro.org \
    --cc=netdev@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).