* [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
@ 2023-09-01 9:36 Puliang Lu
2023-10-13 9:23 ` Johan Hovold
0 siblings, 1 reply; 10+ messages in thread
From: Puliang Lu @ 2023-09-01 9:36 UTC (permalink / raw)
To: johan, gregkh; +Cc: linux-usb, linux-kernel, Puliang Lu
Update the USB serial option driver support for the Fibocom
FM101R-GL
LTE modules as there are actually several different variants.
- VID:PID 413C:8213, FM101R-GL are laptop M.2 cards (with
MBIM interfaces for Linux)
- VID:PID 413C:8215, FM101R-GL ESIM are laptop M.2 cards(with
MBIM interface for Linux)
T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 2 Spd=5000 MxCh= 0
D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
P: Vendor=413c ProdID=8213 Rev= 5.04
S: Manufacturer=Fibocom Wireless Inc.
S: Product=Fibocom FM101-GL Module
S: SerialNumber=a3b7cbf0
C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 3 Spd=5000 MxCh= 0
D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
P: Vendor=413c ProdID=8215 Rev= 5.04
S: Manufacturer=Fibocom Wireless Inc.
S: Product=Fibocom FM101-GL Module
S: SerialNumber=a3b7cbf0
C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
Signed-off-by: Puliang Lu <puliang.lu@fibocom.com>
---
drivers/usb/serial/option.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index 8ac98e60fff5..1dcd1f068f23 100644
--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -203,6 +203,9 @@ static void option_instat_callback(struct urb *urb);
#define DELL_PRODUCT_5829E_ESIM 0x81e4
#define DELL_PRODUCT_5829E 0x81e6
+#define DELL_PRODUCT_FM101R 0x8213
+#define DELL_PRODUCT_FM101R_ESIM 0x8215
+
#define KYOCERA_VENDOR_ID 0x0c88
#define KYOCERA_PRODUCT_KPC650 0x17da
#define KYOCERA_PRODUCT_KPC680 0x180a
@@ -1107,6 +1110,8 @@ static const struct usb_device_id option_ids[] = {
.driver_info = RSVD(0) | RSVD(6) },
{ USB_DEVICE(DELL_VENDOR_ID, DELL_PRODUCT_5829E_ESIM),
.driver_info = RSVD(0) | RSVD(6) },
+ { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
+ { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_E100A) }, /* ADU-E100, ADU-310 */
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_500A) },
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_620UW) },
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
2023-09-01 9:36 Puliang Lu
@ 2023-10-13 9:23 ` Johan Hovold
0 siblings, 0 replies; 10+ messages in thread
From: Johan Hovold @ 2023-10-13 9:23 UTC (permalink / raw)
To: Puliang Lu; +Cc: gregkh, linux-usb, linux-kernel, Puliang Lu
On Fri, Sep 01, 2023 at 05:36:10PM +0800, Puliang Lu wrote:
> Update the USB serial option driver support for the Fibocom
> FM101R-GL
> LTE modules as there are actually several different variants.
> - VID:PID 413C:8213, FM101R-GL are laptop M.2 cards (with
> MBIM interfaces for Linux)
> - VID:PID 413C:8215, FM101R-GL ESIM are laptop M.2 cards(with
> MBIM interface for Linux)
>
> T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 2 Spd=5000 MxCh= 0
> D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
> P: Vendor=413c ProdID=8213 Rev= 5.04
> S: Manufacturer=Fibocom Wireless Inc.
> S: Product=Fibocom FM101-GL Module
> S: SerialNumber=a3b7cbf0
> C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
> A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
> I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
> E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
> I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
> E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
> E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
Could there ever be further vendor class interfaces (e.g. adb) so that
you should be using a more specific match entry?
> Signed-off-by: Puliang Lu <puliang.lu@fibocom.com>
The SoB email does not match the sender address (e.g. as checkpatch
complains about). Could you resend this one from you fibocom account so
that it matches?
> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
@ 2023-10-16 7:36 L-鲁璞亮
2023-10-16 8:23 ` Johan Hovold
0 siblings, 1 reply; 10+ messages in thread
From: L-鲁璞亮 @ 2023-10-16 7:36 UTC (permalink / raw)
To: johan, gregkh; +Cc: linux-usb, linux-kernel, Puliang Lu
From: Puliang Lu <puliang.lu@fibocom.com>
Update the USB serial option driver support for the Fibocom
FM101R-GL
LTE modules as there are actually several different variants.
- VID:PID 413C:8213, FM101R-GL are laptop M.2 cards (with
MBIM interfaces for Linux)
- VID:PID 413C:8215, FM101R-GL ESIM are laptop M.2 cards(with
MBIM interface for Linux)
0x8213: mbim, tty
0x8215: mbim, tty
T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 2 Spd=5000 MxCh= 0
D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
P: Vendor=413c ProdID=8213 Rev= 5.04
S: Manufacturer=Fibocom Wireless Inc.
S: Product=Fibocom FM101-GL Module
S: SerialNumber=a3b7cbf0
C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 3 Spd=5000 MxCh= 0
D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
P: Vendor=413c ProdID=8215 Rev= 5.04
S: Manufacturer=Fibocom Wireless Inc.
S: Product=Fibocom FM101-GL Module
S: SerialNumber=a3b7cbf0
C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
Signed-off-by: Puliang Lu <puliang.lu@fibocom.com>
---
drivers/usb/serial/option.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c
index 8ac98e60fff5..1dcd1f068f23 100644
--- a/drivers/usb/serial/option.c
+++ b/drivers/usb/serial/option.c
@@ -203,6 +203,9 @@ static void option_instat_callback(struct urb *urb);
#define DELL_PRODUCT_5829E_ESIM 0x81e4
#define DELL_PRODUCT_5829E 0x81e6
+#define DELL_PRODUCT_FM101R 0x8213
+#define DELL_PRODUCT_FM101R_ESIM 0x8215
+
#define KYOCERA_VENDOR_ID 0x0c88
#define KYOCERA_PRODUCT_KPC650 0x17da
#define KYOCERA_PRODUCT_KPC680 0x180a
@@ -1107,6 +1110,8 @@ static const struct usb_device_id option_ids[] = {
.driver_info = RSVD(0) | RSVD(6) },
{ USB_DEVICE(DELL_VENDOR_ID, DELL_PRODUCT_5829E_ESIM),
.driver_info = RSVD(0) | RSVD(6) },
+ { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
+ { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_E100A) }, /* ADU-E100, ADU-310 */
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_500A) },
{ USB_DEVICE(ANYDATA_VENDOR_ID, ANYDATA_PRODUCT_ADU_620UW) },
--
2.34.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
2023-10-16 7:36 [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL L-鲁璞亮
@ 2023-10-16 8:23 ` Johan Hovold
0 siblings, 0 replies; 10+ messages in thread
From: Johan Hovold @ 2023-10-16 8:23 UTC (permalink / raw)
To: L-鲁璞亮; +Cc: gregkh, linux-usb, linux-kernel
On Mon, Oct 16, 2023 at 03:36:16PM +0800, L-鲁璞亮 wrote:
> From: Puliang Lu <puliang.lu@fibocom.com>
>
> Update the USB serial option driver support for the Fibocom
> FM101R-GL
> T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 2 Spd=5000 MxCh= 0
> D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
> P: Vendor=413c ProdID=8213 Rev= 5.04
> S: Manufacturer=Fibocom Wireless Inc.
> S: Product=Fibocom FM101-GL Module
> S: SerialNumber=a3b7cbf0
> C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
> A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
> I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
> E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
> I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
> E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
> E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
> E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
> E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
Thanks for resending with the author address fixed.
You did however not answer my question whether there could ever be more
vendor class interfaces (e.g. for adb) so that you should be using a
more specific match for the entries (i.e. USB_DEVICE_AND_INTERFACE_INFO())?
> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
@ 2023-10-16 8:36 Puliang Lu(Puliang)
2023-10-16 8:40 ` Johan Hovold
0 siblings, 1 reply; 10+ messages in thread
From: Puliang Lu(Puliang) @ 2023-10-16 8:36 UTC (permalink / raw)
To: Johan Hovold
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Mon, Oct 16, 2023 at 03:36:16PM +0800, L-鲁璞亮 wrote:
>> From: Puliang Lu <puliang.lu@fibocom.com>
>>
>> Update the USB serial option driver support for the Fibocom
>> FM101R-GL
>
>> T: Bus=04 Lev=01 Prnt=01 Port=01 Cnt=01 Dev#= 2 Spd=5000 MxCh= 0
>> D: Ver= 3.20 Cls=00(>ifc ) Sub=00 Prot=00 MxPS= 9 #Cfgs= 1
>> P: Vendor=413c ProdID=8213 Rev= 5.04
>> S: Manufacturer=Fibocom Wireless Inc.
>> S: Product=Fibocom FM101-GL Module
>> S: SerialNumber=a3b7cbf0
>> C:* #Ifs= 3 Cfg#= 1 Atr=a0 MxPwr=896mA
>> A: FirstIf#= 0 IfCount= 2 Cls=02(comm.) Sub=0e Prot=00
>> I:* If#= 0 Alt= 0 #EPs= 1 Cls=02(comm.) Sub=0e Prot=00 Driver=cdc_mbim
>> E: Ad=81(I) Atr=03(Int.) MxPS= 64 Ivl=32ms
>> I: If#= 1 Alt= 0 #EPs= 0 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
>> I:* If#= 1 Alt= 1 #EPs= 2 Cls=0a(data ) Sub=00 Prot=02 Driver=cdc_mbim
>> E: Ad=8e(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> E: Ad=0f(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> I:* If#= 2 Alt= 0 #EPs= 3 Cls=ff(vend.) Sub=00 Prot=40 Driver=(none)
>> E: Ad=83(I) Atr=03(Int.) MxPS= 10 Ivl=32ms
>> E: Ad=82(I) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>> E: Ad=01(O) Atr=02(Bulk) MxPS=1024 Ivl=0ms
>
> Thanks for resending with the author address fixed.
>
> You did however not answer my question whether there could ever be more
> vendor class interfaces (e.g. for adb) so that you should be using a
> more specific match for the entries (i.e. USB_DEVICE_AND_INTERFACE_INFO())?
0x8213 and 0x8215 have only one mbim and one tty, Thanks.
>
>> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R, 0xff) },
>> + { USB_DEVICE_INTERFACE_CLASS(DELL_VENDOR_ID, DELL_PRODUCT_FM101R_ESIM, 0xff) },
>
> Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
2023-10-16 8:36 Puliang Lu(Puliang)
@ 2023-10-16 8:40 ` Johan Hovold
0 siblings, 0 replies; 10+ messages in thread
From: Johan Hovold @ 2023-10-16 8:40 UTC (permalink / raw)
To: Puliang Lu(Puliang)
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Mon, Oct 16, 2023 at 08:36:30AM +0000, Puliang Lu(Puliang) wrote:
> > You did however not answer my question whether there could ever be more
> > vendor class interfaces (e.g. for adb) so that you should be using a
> > more specific match for the entries (i.e. USB_DEVICE_AND_INTERFACE_INFO())?
>
> 0x8213 and 0x8215 have only one mbim and one tty, Thanks.
Ok, thanks for confirming.
I've now applied the patch.
Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
@ 2023-10-24 6:36 Puliang Lu(Puliang)
2023-10-24 7:24 ` Johan Hovold
0 siblings, 1 reply; 10+ messages in thread
From: Puliang Lu(Puliang) @ 2023-10-24 6:36 UTC (permalink / raw)
To: Johan Hovold
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Mon, 16 Oct 2023 10:40:05 +0200, Johan Hovold wrote:
> Ok, thanks for confirming.
>
> I've now applied the patch.
Can I still modify my patch now? 0x8213 is an ESIM device, 0x8215 is a NO ESIM
device, but it is written backwards in the commit message, and the define is
also reversed.
Thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
2023-10-24 6:36 Puliang Lu(Puliang)
@ 2023-10-24 7:24 ` Johan Hovold
0 siblings, 0 replies; 10+ messages in thread
From: Johan Hovold @ 2023-10-24 7:24 UTC (permalink / raw)
To: Puliang Lu(Puliang)
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Tue, Oct 24, 2023 at 06:36:58AM +0000, Puliang Lu(Puliang) wrote:
> On Mon, 16 Oct 2023 10:40:05 +0200, Johan Hovold wrote:
> > I've now applied the patch.
>
> Can I still modify my patch now? 0x8213 is an ESIM device, 0x8215 is a NO ESIM
> device, but it is written backwards in the commit message, and the define is
> also reversed.
No, this patch is already in Linus's tree so that's not possible. But
please do send a follow-up incremental fix.
Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
@ 2023-10-26 11:44 Puliang Lu(Puliang)
2023-10-26 12:06 ` Johan Hovold
0 siblings, 1 reply; 10+ messages in thread
From: Puliang Lu(Puliang) @ 2023-10-26 11:44 UTC (permalink / raw)
To: Johan Hovold
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Tue, 24 Oct 2023 09:24:20 +0200, Johan Hovold wrote:
> No, this patch is already in Linus's tree so that's not possible. But
> please do send a follow-up incremental fix.
>
> Johan
Thank you for your reply.
We have decided not to address this issue because it does not
affect the operation of modem.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL
2023-10-26 11:44 Puliang Lu(Puliang)
@ 2023-10-26 12:06 ` Johan Hovold
0 siblings, 0 replies; 10+ messages in thread
From: Johan Hovold @ 2023-10-26 12:06 UTC (permalink / raw)
To: Puliang Lu(Puliang)
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel@vger.kernel.org
On Thu, Oct 26, 2023 at 11:44:37AM +0000, Puliang Lu(Puliang) wrote:
> On Tue, 24 Oct 2023 09:24:20 +0200, Johan Hovold wrote:
>
> > No, this patch is already in Linus's tree so that's not possible. But
> > please do send a follow-up incremental fix.
> Thank you for your reply.
> We have decided not to address this issue because it does not
> affect the operation of modem.
No, that's not the way things work.
Please do send an incremental fix so that this is corrected. Otherwise
I'll just revert your patch.
Johan
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-10-26 12:06 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-16 7:36 [PATCH] USB: serial: option: add Fibocom to DELL custom modem FM101R-GL L-鲁璞亮
2023-10-16 8:23 ` Johan Hovold
-- strict thread matches above, loose matches on Subject: below --
2023-10-26 11:44 Puliang Lu(Puliang)
2023-10-26 12:06 ` Johan Hovold
2023-10-24 6:36 Puliang Lu(Puliang)
2023-10-24 7:24 ` Johan Hovold
2023-10-16 8:36 Puliang Lu(Puliang)
2023-10-16 8:40 ` Johan Hovold
2023-09-01 9:36 Puliang Lu
2023-10-13 9:23 ` Johan Hovold
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox