Linux USB
 help / color / mirror / Atom feed
From: "Wu. JackBB (GSM)" <JackBB_Wu@compal.com>
To: Johan Hovold <johan@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"wojackbb@gmail.com" <wojackbb@gmail.com>
Subject: RE: [External Mail] Re: [PATCH v3] USB: serial: option: add new usb-id for Dell Wireless DW5826e-m
Date: Fri, 5 Jun 2026 01:36:43 +0000	[thread overview]
Message-ID: <50432b9f12eb4d26aa17eba2bce9c059@compal.com> (raw)
In-Reply-To: <aiFlImelxdae6g4s@hovoldconsulting.com>



> On Thu, Jun 04, 2026 at 06:30:10PM +0800, Jack Wu via B4 Relay wrote:
>
> > From: Jack Wu <jackbb_wu@compal.com>
> >
> > Add support for Dell DW5826e-m with USB-id 0x413c:0x81ea
> >
> > T: Bus=03 Lev=01 Prnt=01 Port=05 Cnt=01 Dev#= 15 Spd=480 MxCh= 0
> > D: Ver= 2.10 Cls=ef(misc ) Sub=02 Prot=01 MxPS=64 #Cfgs= 1
> > P: Vendor=413c ProdID=81ea Rev= 5.04
> > S: Manufacturer=DELL
> > S: Product=DW5826e-m Qualcomm Snapdragon X12 Global LTE-A
> > S: SerialNumber=358988870177734
> > C:* #Ifs= 7 Cfg#= 1 Atr=a0 MxPwr=500mA
> > A: FirstIf#=12 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
> > I:* If#= 0 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=ff Prot=30 Driver=option
> > E: Ad=01(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > E: Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > I:* If#= 1 Alt= 0 #EPs= 2 Cls=ff(vend.) Sub=42 Prot=01 Driver=usbfs
> > E: Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > E: Ad=82(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=60 Driver=option
> > E: Ad=84(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
> > E: Ad=83(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > E: Ad=03(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > I:* If#= 3 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=ff Prot=40 Driver=option
> > E: Ad=86(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
> > E: Ad=85(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > E: Ad=04(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > I:* If#= 4 Alt= 0 #EPs= 1 Cls=ff(vend.) Sub=ff Prot=ff Driver=(none)
> > E: Ad=87(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
> > I:* If#=12 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
> > E: Ad=88(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
> > I: If#=13 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> > I:* If#=13 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> > E: Ad=8e(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> > E: Ad=0f(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> >
> > Signed-off-by: Jack Wu <jackbb_wu@compal.com>
>
> > ---
> > v3:
> > - Blacklist interface #4 (ADB) using RSVD(4)
> > - Change DELL_PRODUCT_DW5826E_ESIM to DELL_PRODUCT_5826E_ESIM
>
> You forgot to mention the missing comma that caused a build failure...
>
> I added the missing RSVD(4) when applying v2 and noticed that it failed
> to compile but assumed that I had messed when amending the patch (and
> fixed it up).
>
> End result is here:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/johan/usb-serial.git/commit/?h=usb-linus&id=1938fb9fe38c4f04a3f30bea44f8071c80a63be4
>
> Johan

I'm sorry for the inconvenience.
Do I need to upload the v4 version to fix this commit message error?

> On Thu, Jun 04, 2026 at 10:31:43AM +0000, Wu. JackBB (GSM) wrote:
>
> > ================================================================================================================================================================
> > This message may contain information which is private, privileged or
> > confidential of Compal Electronics, Inc. If you are not the intended
> > recipient of this message, please notify the sender and destroy/delete the
> > message. Any review, retransmission, dissemination or other use of, or
> > taking of any action in reliance upon this information, by persons or
> > entities other than the intended recipient is prohibited.
> > ================================================================================================================================================================
>
> Make sure not to include footers like this when replying.
>
> Johan

I apologize for the automated disclaimer.
It is forced by our corporate mail server whenever sending emails externally.

Due to internal policies, we are required to use the company email address to submit the driver.
However, to avoid this footer issue,
is it acceptable if I use my personal email (wojackbb@gmail.com) to reply to the community from now on?
I can CC my personal email on the patches so I can jump into the thread using that address.






================================================================================================================================================================
This message may contain information which is private, privileged or confidential of Compal Electronics, Inc. If you are not the intended recipient of this message, please notify the sender and destroy/delete the message. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon this information, by persons or entities other than the intended recipient is prohibited.
================================================================================================================================================================

      reply	other threads:[~2026-06-05  1:36 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-04 10:30 [PATCH v3] USB: serial: option: add new usb-id for Dell Wireless DW5826e-m Jack Wu via B4 Relay
2026-06-04 11:44 ` Johan Hovold
2026-06-05  1:36   ` Wu. JackBB (GSM) [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=50432b9f12eb4d26aa17eba2bce9c059@compal.com \
    --to=jackbb_wu@compal.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=johan@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=wojackbb@gmail.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