linux-usb.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <gregkh@linuxfoundation.org>
To: "庞苏荣 (Surong Pang)" <surong.pang@unisoc.com>
Cc: "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"翟京 (Orson Zhai)" <Orson.Zhai@unisoc.com>,
	"刘智勇 (Zhiyong Liu)" <Zhiyong.Liu@unisoc.com>,
	"Surong.Pang@gmail.com" <Surong.Pang@gmail.com>
Subject: Re: [PATCH] usb: gadget: rndis: add multi packages support for rndis
Date: Tue, 26 Mar 2024 15:07:27 +0100	[thread overview]
Message-ID: <2024032616-dumping-blustery-aeb2@gregkh> (raw)
In-Reply-To: <9d39b91505c4449f98e8431e2f257f8b@shmbx05.spreadtrum.com>

On Tue, Mar 26, 2024 at 10:47:16AM +0000, 庞苏荣 (Surong Pang) wrote:
> Dear Greg,
> Share Android Phone internet to Windows PC via USB still need the rndis feature.

What supported Windows PC still only has rndis and not cdc-ncm support?
Or the other USB networking driver support built in?  Windows has
deprecated RNDIS for a good reason, let's not continue to insist on
using insecure protocols please.

> OK, if rndis will be deleted entirely, this patch can be abandoned.

Eventually, yes, but as is, your patch can't be taken given the quick
review I already provided.

If you can show that there are Windows systems that only have RNDIS
support, and need this increased speed (i.e. that they can also support
the speedup), and fix up the other issues, then we can review your new
submission.  But please provide that proof when you do so.

thanks,

greg k-h

  reply	other threads:[~2024-03-26 14:07 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-26 10:47 [PATCH] usb: gadget: rndis: add multi packages support for rndis 庞苏荣 (Surong Pang)
2024-03-26 14:07 ` Greg KH [this message]
  -- strict thread matches above, loose matches on Subject: below --
2024-03-26 10:52 庞苏荣 (Surong Pang)
2024-03-06  5:49 Surong Pang
2024-03-26  9:34 ` Greg KH
2024-03-26  9:36 ` Greg KH

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=2024032616-dumping-blustery-aeb2@gregkh \
    --to=gregkh@linuxfoundation.org \
    --cc=Orson.Zhai@unisoc.com \
    --cc=Surong.Pang@gmail.com \
    --cc=Zhiyong.Liu@unisoc.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=surong.pang@unisoc.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;
as well as URLs for NNTP newsgroup(s).